search

justinmeiners / ios-color-wheel

A fully scalable, dynamically rendered color wheel for iOS.
82 stars 15 forks source link

Added Key Value Observing support for currentColor. #3

Closed dimensionerror closed 8 years ago

dimensionerror commented 10 years ago

I added willChangeValueForKey and didChangeValueForKey method calls to wrap changes to the currentColor property so that outside classes can use KVO to monitor the color.

justinmeiners commented 8 years ago

I am interested in this change, however I would like to avoid an additional file for the string category. Is it possible to add KVO without that category?

dimensionerror commented 8 years ago

Yes, it is. I removed the category but have not tested the changes.

justinmeiners commented 8 years ago

If you were to create a new pull request, or just send me the relevant code I would be happy to integrate and test it for you. I am familiar with KVO just not how to implement it for properties.

justinmeiners commented 8 years ago

Oh thank you! I just saw you changed that. I will go ahead and integrate this later today.