[a11y] Focus state not visible for "Home" icon in toolbar.

The changes in the MR makes it so the focus state matches that of the other items in the toolbar.

Hi,

Tested the MR !272 to address the issue where the focus outline was not appearing on the home link. it's now works perfectly fine in both vertical and horizontal views.

Testing Steps:
1. Install the Drupal 10.0.x version and enable gin 8.x.
2. switch to MR and verified the changes.

attached the screen for references.

Looks Good for RTBC

+1 LGTM, moving to RTBC

Nice work @bnjmnm!

Moving this back to needs work as the focus cuts off when using the vertical toolbar.

Ooops, didn't spot that. Sorry @saschaeggi. Will take a look for a fix this afternoon.

Hey @saschaeggi,

I've added a small change to the existing MR which corrects the previous issue with the focus being cut off.

I've checked this on both the horizontal and vertical toolbar.

Please could you take another look? Thanks!

Reviewing this issue.

I have tested the recent MR #7 to verify the issue mentioned in #6 and it is working fine without getting cut off while on focus on the vertical toolbar. The MR works totally fine for the horizontal toolbar as well. I am attaching a screenshot for reference after applying the MR.
Looks Good for RTBC++

Hey @saschaeggi,

Would you mind taking another look at this now it's been fixed and tested by the community?

Thanks

Luke

Great work, thanks 👏

Automatically closed - issue fixed for 2 weeks with no activity.