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

feat(ui5-flexible-column-layout): add start/mid/endColumnAccessibleName properties #2483

Merged
merged 5 commits into from
Nov 30, 2020

Conversation

fifoosid
Copy link
Contributor

@fifoosid fifoosid commented Nov 16, 2020

Add startColumnAccessibleText, midColumnAccessibleText and endColumnAccessibleText properties

Fixes #2464

…, midColumnAccessibleText and endColumnAccessibleText properties
* @public
* @since 1.0.0-rc.11
*/
startColumnAccessibleText: {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think accessibleName is the correct term, here it is accessibleText. But, I will have to discuss it with responsible team.

@@ -90,7 +90,7 @@ const metadata = {
* @public
* @since 1.0.0-rc.11
*/
startColumnAccessibleText: {
startColumnAccessibleName: {
Copy link
Member

@ilhan007 ilhan007 Nov 24, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and the properties' descriptions - "accessibility text" to "accessible name"

@ilhan007 ilhan007 changed the title feat(ui5-flexible-column-layout): implement startColumnAccessibleText... feat(ui5-flexible-column-layout): add start/middle/endAccessibleName properties Nov 24, 2020
@ilhan007 ilhan007 changed the title feat(ui5-flexible-column-layout): add start/middle/endAccessibleName properties feat(ui5-flexible-column-layout): add start/middle/endColumnAccessibleName properties Nov 24, 2020
@ilhan007 ilhan007 changed the title feat(ui5-flexible-column-layout): add start/middle/endColumnAccessibleName properties feat(ui5-flexible-column-layout): add start/mid/endColumnAccessibleName properties Nov 24, 2020
Copy link
Member

@ilhan007 ilhan007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything thing is fine, just update the jsdoc descriptions - "accessibility text" to "accessible name"

@fifoosid fifoosid merged commit fb0b24c into master Nov 30, 2020
@fifoosid fifoosid deleted the fcl-region-acc branch November 30, 2020 10:07
ilhan007 pushed a commit that referenced this pull request Dec 7, 2020
fifoosid added a commit that referenced this pull request Dec 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

a11y issue in Flexible column layout: No way to pass land mark text
2 participants