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

Feature Request: Update Images? #11

Closed
justinwhite2 opened this issue Oct 19, 2017 · 3 comments
Closed

Feature Request: Update Images? #11

justinwhite2 opened this issue Oct 19, 2017 · 3 comments
Assignees

Comments

@justinwhite2
Copy link

Great control, thank you for it!

How would one go about allowing for a button image to be updated? Imagine a state: on/off - off having a different image or a just an image with some opacity. Or even a button with a series of states.

If you can point me in the right direction, I will happily code it up and submit a pull request.

Thank you again and cheers!

@zapletnev
Copy link
Contributor

Hi Justin,
Thank you for choosing fan-menu. I will check the easiest way how to achieve it and will back to you.

@zapletnev zapletnev self-assigned this Oct 19, 2017
@zapletnev
Copy link
Contributor

Hi @justinwhite2,

Thanks again for using fan-menu!

I checked the code and I realized that there is no "selected" state for the button in the current implementation. If you select some button from the menu, the menu is closing.

But you can quickly extend fan-menu. Fan-menu based on the Macaw library. If you didn't hear about that here is a good place to start.

Technical details about fan-menu that can be helpful for you:

  • FanMenuScene class is responsible how the menu is looking and behaves
  • createFanButtonNode describes how each button of the menu looks like. It is a circle and image for now.
  • updateState is starting close or open animation

Feel free to contact me if you will have any questions about Fan-menu or Macaw. If your changes would be helpful for other users, we can include your changes in the main project. Good luck!

@zapletnev
Copy link
Contributor

I hope this solution helped you. Closing this issue for now. Please let me know if any questions remain.

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