Seven's primary button focus styling sets a border colour that is really similar to the background gradient. It looks like there is no border at all.

The offending CSS is on line 83 of buttons.theme.css:

.button--primary:focus {
  border-color: #0087db;
}
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

anksy’s picture

So which colour should be set as border colour??

anksy’s picture

Assigned: Unassigned » anksy
LewisNyman’s picture

The border is set to be #1e5c90 in the selector before it. I think that's correct. We can just delete that extra css

andrei.dincu’s picture

andrei.dincu’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 4: primary-button-issue-2207631-4.patch, failed testing.

anksy’s picture

Please ignore this patch.

anksy’s picture

anksy’s picture

Status: Needs work » Needs review

The last submitted patch, 7: primary-button-border-not-visible-on-focus-2207631-7.patch, failed testing.

andrei.dincu’s picture

LewisNyman’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
8.63 KB

Much better :) Thanks!

Before:

After:

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed a02ae4a and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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