2fa-notifier icon indicating copy to clipboard operation
2fa-notifier copied to clipboard

Remove the outline from the custom buttons on the dropdown

Open ravenac95 opened this issue 7 years ago • 5 comments

@conorgil @designedbinary you should get rid of the blue outline:

screen shot 2018-05-04 at 10 48 35 pm

ravenac95 avatar May 05 '18 05:05 ravenac95

Here's the solution:

.someclass:focus {
  outline: 0;
}

// can't remember if you need this next section
.someclass:active {
  outline: 0; 
}

ravenac95 avatar May 05 '18 05:05 ravenac95

@ravenac95, Does this show up for you by default?

designedbinary avatar May 13 '18 04:05 designedbinary

@conorgil, I don't think this should be part of the "initial release." From an accessibility standpoint, we'll need some focus indicators. Its just a matter for making them better incorporate with the overall design.

So, I want to address this issue, but don't think its worth blocking release.

designedbinary avatar May 13 '18 16:05 designedbinary

@designedbinary Agreed. This is a minor bug at best. There are more important things to work on right now.

conorgil avatar May 13 '18 17:05 conorgil

@ravenac95 I don't think we addressed this specifically, but I also don't see the blue box around the menu anymore. Does it still show for you? If no, can you close this issue :)

conorgil avatar Sep 26 '18 05:09 conorgil