WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 6 years ago

Last modified 3 months ago

#7995 closed enhancement (wontfix)

Remove Dotted Borders in Firefox

Reported by: ydekproductions Owned by: Nicholas91
Milestone: Priority: normal
Severity: normal Version: 2.7
Component: UI Keywords: has-patch needs-testing
Focuses: Cc:

Description

I would just like to propose removing the dotted borders that show up in Firefox for active links and buttons. I think they come across as ugly, especially in cases like the Screen Options button.

I know their purpose is for usability's sake, but that's alleviated by normal CSS states.

Attachments (1)

firefox-css-fix.diff (445 bytes) - added by ydekproductions 7 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 @azaozz7 years ago

That would make them look better, but what about navigating all links/buttons with the Tab key?

For this to work properly all links and buttons will have to have :active or :focus styling too. Don't think this can be done globally, will need at least a few separate styles.

comment:2 follow-up: @ydekproductions7 years ago

Is that really a big concern for every link on the page? Maybe just leave the dotted borders on the input fields and buttons. I can't imagine people Tabbing around through the left hand navigation.

Also, Firefox is the only browser that renders those, so I just tend to like to neaten it up to look the same all around.

comment:3 in reply to: ↑ 2 @azaozz7 years ago

Replying to ydekproductions:

Is that really a big concern for every link on the page? Maybe just leave the dotted borders on the input fields and buttons. I can't imagine people Tabbing around through the left hand navigation.

Yes, it is a big accessibility concern, not everyone can use a mouse. Can remove the dotted box (don't like it either) but the patch has to have alternate method of highlighting the links, buttons, etc.

comment:4 @ydekproductions7 years ago

Then for the navigation items, can we just mirror the :hover state to the :active states as well? That can just be done with a little bit of tedious find and replace.

Actually, we can probably do that for every single rule that has a :hover state, duplicate it for :active.

Does that sound feasible? If so, I can take some time and go though the rules and do that.

comment:5 @markjaquith7 years ago

  • Milestone changed from 2.7 to 2.8

comment:6 @Nicholas916 years ago

  • Cc dragos.nicholas@… added
  • Keywords has-patch needs-testing added
  • Owner changed from anonymous to Nicholas91
  • Status changed from new to assigned

comment:7 @ryan6 years ago

  • Component changed from General to UI

comment:8 @ryan6 years ago

  • Resolution set to wontfix
  • Status changed from assigned to closed

No traction. Reopen with fresh patch.

comment:9 @DrewAPicture3 months ago

  • Milestone 2.8 deleted
Note: See TracTickets for help on using tickets.