Nodejs.dev: Remove dotted border from download cards and download icon

Created on 16 Nov 2020  路  4Comments  路  Source: nodejs/nodejs.dev

Description

We should remove the dashed border around the buttons LTS/Current, Download cards section and download icon

Screen Shot 2020-11-15 at 7 45 38 PM
Screen Shot 2020-11-15 at 7 45 46 PM

Screen Shot 2020-11-15 at 7 45 54 PM

accessibility bug good first issue help wanted

Most helpful comment

i'd like to work on it :)

All 4 comments

i'd like to work on it :)

This css rule seems to be causing this to happen
https://github.com/nodejs/nodejs.dev/blob/ca46400b2b63180f447201f213f01923e5b146cd/src/styles/layout.scss#L247-L249

Do we want to specifically remove this dotted-border from the component mentioned above or remove it at all?

@talarviv the outline is there to improve accessibility but they are not correct for this component.

Screenshot 1: The outline should only be shown for each button when tabbed through, not the entire group.

Screenshot 2: The first focus (the whole group) is wrong, that should be removed. The outline should only be kept for the individual download options.

Screenshot 3: The padding and outline are not the same.

Try to adjust HTML before CSS.

And thanks for picking this up, I am very picky when it comes to accessibility 馃槅馃憤

I decided to split the change into two parts. so #1063 is about download cards.
I will open a new pull request for download button (LTS/Current screenshot 1).

Was this page helpful?
0 / 5 - 0 ratings