As the snippet is color highlighted, it makes sense to force a light background color on it with some custom CSS.

CommentFileSizeAuthor
#5 add_css-3087035-5.patch7.83 KBlolandese
#3 add_css-3087035-3.patch6.05 KBlolandese
#2 Convert YAML to PHP Drush Site-Install.jpg91.49 KBlolandese

Comments

lolandese created an issue. See original summary.

lolandese’s picture

lolandese commented
StatusFileSize
new Convert YAML to PHP Drush Site-Install.jpg91.49 KB

This is how it looks on Claro:

Screenshot

lolandese’s picture

lolandese commented
Status: Active » Needs review
StatusFileSize
new add_css-3087035-3.patch6.05 KB

The attached patch:

  • adds an extra light grey background color to the snippet area with some vertical margin and horizontal padding
  • sets green, orange and red text colors (status, warning, error) that render well both on light as dark backgrounds.

Status: Needs review » Needs work

The last submitted patch, 3: add_css-3087035-3.patch, failed testing. View results

lolandese’s picture

lolandese commented
Status: Needs work » Needs review
StatusFileSize
new add_css-3087035-5.patch7.83 KB

Some texts changed so we updated the test accordingly.

  • lolandese committed f7f26f7 on 8.x-1.x 
    Issue #3087035 by lolandese: Hard to read on dark background
lolandese’s picture

lolandese commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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