You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Reasoning: i was developing with the library and found that
the method within the interface was incorrectly named (setOnFastChangeListener instead of onFastChange)
It was extremely frustrating when creating similar code to have to rename the method and the interface for the exact same code (for setting color variables)
An option IF you really dont want to remove the OnFastChangeListener is to make it the same method names as both interfaces were made to provide the same data for the same output just via a different path.
Wanted to create an issue for this before i make a pull request.
The text was updated successfully, but these errors were encountered:
Don't see the point in it, i created a fork located https://github.com/KlutzyBubbles/colorpicker/tree/master/colorpicker-library/src/main/java/petrov/kristiyan/colorpicker which contains the changes to remove the interface.
Reasoning: i was developing with the library and found that
An option IF you really dont want to remove the OnFastChangeListener is to make it the same method names as both interfaces were made to provide the same data for the same output just via a different path.
Wanted to create an issue for this before i make a pull request.
The text was updated successfully, but these errors were encountered: