The Tab component lacks a11y about ui HOT 9 CLOSED

@cooper-joe this is not a breaking change, so there's nothing really preventing us from doing this now. However, since we are migrating this repo to @dhis2/ui, I think it'd be better to work on it there.

from ui.

Thanks for reporting. @cooper-joe do you have any suggestions with regards to this? Perhaps activating the hover styles on focus as well? And making sure this gets overridden by the selected style? IMO an outline wouldn't look very good in a TabBar....

from ui.

Yes, we should definitely activate the hover styles on focus , good idea. Although I don't think that just the hover style is clear enough for keyboard navigation.

Is it possible to, when the tab has focus, outline the text only?

Here is a rough illustration of what I mean:

from ui.

Great idea! I'll work on this once we get next released 👍

from ui.

@cooper-joe I've given this a little try and it's definitely possible. Could you let me know:

• padding around the text
• border color/thickness/style

from ui.

Thanks @HendrikThePendric. Let's go for:

border: 1px dotted var(--grey900);
padding: 0px 2px;

(based on an application directly on the text)

from ui.

Hi! Due to a lack of activity on this issue over time it seems to be stale. If still relevant, please provide information that moves it forward: e.g. more information, reason to keep it open, supply a pull request, etc.
Any activity will keep it open, otherwise it will be closed automatically. Thanks! 🤖

from ui.

Still relevant. @HendrikThePendric I just tested the latest ui-core and these styles weren't present. Do you think we can add them now, or is it better to wait until ui 5?

from ui.

https://jira.dhis2.org/browse/LIBS-77

from ui.