WordPress.org

Make WordPress Core

Opened 5 months ago

Last modified 5 months ago

#45904 new defect (bug)

Twenty Nineteen: .button doesn't override link color

Reported by: laurelfulford Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 5.0.2
Component: Bundled Theme Keywords: needs-patch has-screenshots
Focuses: Cc:

Description

Originally reported by @crunnells in Twenty Nineteen's GitHub repo:

I noticed this one the theme demo site when I tried to change the "Get in touch" button into a link. If you put a link inside of an element with the .button class, the link color is the same color blue as the background, so the text can't be read. The same thing occurs when you try <a href="#" class="button">, so we'll need to have an override on the link color so that it's readable.

Moved over from: https://github.com/WordPress/twentynineteen/issues/746

Attachments (2)

Screen Shot 2019-01-23 at 1.35.52 PM.png (19.4 KB) - added by laurelfulford 5 months ago.
Twenty Nineteen - links with the class .button
Screen Shot 2019-01-23 at 1.36.19 PM.png (18.7 KB) - added by laurelfulford 5 months ago.
Twenty Nineteen - visited link with the class .button

Download all attachments as: .zip

Change History (4)

#1 @laurelfulford
5 months ago

  • Keywords needs-screenshots added

@laurelfulford
5 months ago

Twenty Nineteen - links with the class .button

@laurelfulford
5 months ago

Twenty Nineteen - visited link with the class .button

#2 @laurelfulford
5 months ago

  • Keywords has-screenshots added; needs-screenshots removed
  • Milestone changed from Awaiting Review to Future Release

I could recreate this in the widget area, when non-visited links had the .button class added. The theme's .widget a styles is overriding the white text colour on the .button class.

:visited links display correctly initially. Both :visited and non-visited links display low contrast on hover (blue text on a black background).

Note: See TracTickets for help on using tickets.