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

@readonly secondary tag would be useful for @property primary tag #123

Open
zpapp opened this issue Feb 23, 2013 · 7 comments
Open

@readonly secondary tag would be useful for @property primary tag #123

zpapp opened this issue Feb 23, 2013 · 7 comments

Comments

@zpapp
Copy link

zpapp commented Feb 23, 2013

Currently the @readonly secondary tag is ignored if the primary tag is @Property. I would like to be able to add the "readonly" flag to properties as well, not just to @Attribute and @config elements.

@cedx
Copy link

cedx commented Sep 11, 2014

@readOnly is intended for YUI attributes. For properties, you can use the @final tag.

@lannymcnie
Copy link

I agree that this is an issue. In JavaScript, we typically treat properties as read-only. Marking them as final doesn't reflect the same meaning. I would like this support as well.

(thanks for an awesome tool)

@okuryu
Copy link
Member

okuryu commented Mar 8, 2015

OK, I'll take a look.

@lannymcnie
Copy link

We modified the source locally to make it work - I can submit a pull request tomorrow.

@okuryu
Copy link
Member

okuryu commented Mar 8, 2015

Sure, patch welcome. :)

@andyearnshaw
Copy link
Contributor

@lannymcnie are you still able to submit a pull request for this?

@okuryu
Copy link
Member

okuryu commented May 22, 2015

Maybe not yet.

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

5 participants