Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

User Favorites Do Not Appear & Overwritten #1

Closed
jnozzi opened this issue Mar 4, 2017 · 1 comment
Closed

User Favorites Do Not Appear & Overwritten #1

jnozzi opened this issue Mar 4, 2017 · 1 comment

Comments

@jnozzi
Copy link
Contributor

jnozzi commented Mar 4, 2017

Like BFColorPicker, this clean new implementation also suffers a bug where user favorites (the favorite color palette at the bottom) are not loaded when the picker is displayed. These favorites are shared across applications. If you then add favorites through the picker, the user's favorite colors are replaced with the newly-created list (that is, a list of 10 colors is immediately reset to a list with the only first color you add via the picker). Because the list is "empty" when presented, then overwritten when changed, this leads to data loss.

Steps to recreate:

  1. Launch another application and create or observe favorite colors in the color picker panel.
  2. Launch your picker demo app.
  3. Note the empty favorites.
  4. Drag a new color into the favorites.
  5. Switch back to other app and observe reset favorites with new color only.
  6. Quit your demo app and relaunch.
  7. Observe once again the most recent favorites are empty.

After spending some time investigating BFColorPicker, I was unable to determine the cause. I have a theory transferring the panel's contents disconnects something that wasn't considered. I welcome a discussion. :-)

@OskarGroth
Copy link
Owner

Thank you! Fixed in 75d4f5d.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants