fundamental-ngx
fundamental-ngx copied to clipboard
[Sourcing-2][Accessibility: ACC-253.1][WIP] The search field, within Table does not have visible label defining the purpose.
Is this a bug, enhancement, or feature request?
Bug
Describe your proposal.
Search label in Platform Table should be customisable by user. As per accessibility Team bases on what kind of Table field we are searching its should be that not by default 'Search' always. Like for Employee table we should show 'Search for Employee'
Can you handle that on the application side
No
Which versions of Angular and Fundamental Library for Angular are affected? Please, specify the exact version. (If this is a feature request, use current version.)
All latest version
If this is a bug, please provide steps for reproducing it; the exact components you are using;
NGX Platform Table toolbar menu search textbox
Please provide relevant source code (if applicable).
Please provide stackblitz example(s).
In case this is Accessibility related topic, did you consult with an accessibility expert? If not, please do so and share their recommendations.
Yes we have got this bug from Accessibility expert only.
Did you check the documentation and the API?
Yes
Did you search for similar issues?
Yes
Is there anything else we should know?
Attaching screenshot for references
@JyotiKumari1507 you can customize the toolbar and the content of it with core table like this example
@nikolay-kolarov do you think we have to offer more than that?
@JyotiKumari1507 you can customize the toolbar and the content of it with core table like this example
@nikolay-kolarov do you think we have to offer more than that?
@droshev Accessibility Team expecting Placeholder label of Search should be more customise where user can decide based on Table. Currently its fixed 'Search'. We can discussed about this more in next week call.
Hi @droshev ,
In the context of the SearchField the placeholder, together with the search icon give enough context that this is a search field and as an exception, a real label in front can be skipped (this is approved by UX and ACC).
Best Regards, Nikolay
Going through the report once again, I see that the focus is more on the placeholder and not on external label. I think the reporter needs an API to check the placeholder from application side, which is a feature request. In UI5 such an API exist, so I do not see issues to introduce it here as well.
Best Regards, Nikolay
You can control the search field(setting a custom placeholder - link) if you use Core table. there you have full control of all attributes and inputs.
@droshev pls let me know the NGX version where this fix available?
@JyotiKumari1507 there wasn't a fix. you can check the API for everything you need
Hi @droshev, we are not using the search component directly. That is coming as part of platform table component. We don't have control of Search label over there. Based on accessibility team suggestion, either the label should be able to customise or we should have tooltip on the Search icon.
@divyamalhotra488 the component TableToolbarComponent
offers an input searchFieldAriaLabel
which you can use to pass what you need I think