-
Notifications
You must be signed in to change notification settings - Fork 117
Have date chooser dispatch month change events #53
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
Have date chooser dispatch month change events #53
Conversation
This reverts commit 2ec5304.
Note as other pull request have not been accepted yet this pull request contains some extra changes and my need to be modified. I can't see any easy way of fixing this as those changes were made in my develop branch. |
I'm looking over the changes. Why does "displayedMonthChanged" need to change to "monthChanged"? I know that adding the [Event] metadata to DateChooser is necessary, but I don't see why the event names had to change. Was there a conflict? |
To make the event handler name more in line with other event handler names. See my email on the dev list can change the event name back if you think it's a better fit. |
Reopens as PR #100 to remove commit history. |
Example code:
<js:Application xmlns:fx="http://ns.adobe.com/mxml/2009"
xmlns:js="library://ns.apache.org/royale/basic">
</js:Application>