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

property naming issue: 'readOnly' or 'readonly' #412

Closed
ilhan007 opened this issue May 20, 2019 · 0 comments · Fixed by #413
Closed

property naming issue: 'readOnly' or 'readonly' #412

ilhan007 opened this issue May 20, 2019 · 0 comments · Fixed by #413
Labels
API documentation This issue is about wrong documentation

Comments

@ilhan007
Copy link
Member

ilhan007 commented May 20, 2019

Describe the documentation issue
Several components have the property "readonly", but it is written differently.
It i called "readonly" in ui5-input and ui5-datepicker.
It i called "readOnly" in ui5-checkbox and ui5-radiobutton.

Expectation
As there is no difference in the purpose of the property,
we should stick to single name, either "readonly", or "readOnly"

Side notes
There is native "readonly" property already (and aria-readonly).

Affected components
ui5-input, ui5-datepicker, ui5-checkbox, ui5-radiobutton

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API documentation This issue is about wrong documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant