Make WordPress Core

Opened 4 years ago

Last modified 34 hours ago

#45916 new enhancement

Twenty Nineteen: Consider setting background color and foreground color together

Reported by: allancole's profile allancole Owned by:
Milestone: 6.3 Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: good-first-issue good-first-bug has-patch changes-requested
Focuses: css Cc:

Description

Originally reported by @joyously in the Twenty Nineteen GitHub repo:

On the <body> tag, as a best practice the background-color and text color should always be set together in the main stylesheet. This will make it easier to quickly understand the underlying color scheme.

See here: https://core.trac.wordpress.org/browser/trunk/src/wp-content/themes/twentynineteen/style.css#L765

Original ticket here: https://github.com/WordPress/twentynineteen/issues/64

Attachments (1)

45916.diff (2.1 KB) - added by samful 3 years ago.

Download all attachments as: .zip

Change History (4)

#1 @samful
3 years ago

  • Focuses css added
  • Keywords good-first-bug has-patch added; needs-patch removed

This ticket is from 2018 on github, I submitted a patch to push this along and changed keywords. If this patch is accepted, then the 2017 theme also has the same issues and needs to be patched in the same way. (The 2020 theme does set these together btw)

@samful
3 years ago

#2 @sabernhardt
4 weeks ago

  • Milestone changed from Awaiting Review to 6.3

The patch still applies. We could also consider moving the background-color to _typography.scss so the two body colors are defined earlier in the stylesheet and before the text color of form elements (button, input, select, optgroup, textarea).

#3 @oglekler
34 hours ago

  • Keywords changes-requested added

Hi @samful,
can you please make a bit of an improvement, which is mentioned in #comment:2? If you do not have the opportunity, please write if for another developer can make it further.

Thank you 🙏

Note: See TracTickets for help on using tickets.