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

Surface the automatic events from notify properties more clearly #55

Open
rictic opened this issue Aug 24, 2015 · 3 comments
Open

Surface the automatic events from notify properties more clearly #55

rictic opened this issue Aug 24, 2015 · 3 comments

Comments

@rictic
Copy link
Contributor

rictic commented Aug 24, 2015

Currently we add a little text annotation on notifyproperties. This is useful for power users who've used Polymer for a while, but I often see folks that want to listen for the automatically generated events but don't know what notify implies.

It'd be helpful for them if we added a tooltip or similar on the notify annotation and had a way to show auto generated events for users curious about them.

@jmstriegel
Copy link

+1
It would be super clear for auto generated events to be explicitly listed under the "Events" section of each element's API doc.

@notwaldorf
Copy link
Contributor

We're already saying something like Default: false – readOnly – notifies for these properties, maybe it could be notifies foo-changed instead?

@arthurevans
Copy link
Contributor

I'm with @jmstriegel on this... People look in the Events section for events. The current system works fine for people who know the conventions, but for everyone else, it's just one more unnecessary mystery.

If there's no technical limitation, I'd go with adding them to Events and adding a "Hide generated events" button.

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

No branches or pull requests

4 participants