Problem/Motivation

dark mode showing a light button

The button on the cards does not look right in dark mode in Gin. (Also need to check the modal popup button)

Steps to reproduce

Install latest Drupal and Gin, enable the Gin theme and set it as your admin theme. Go to Gin settings and enable Dark Mode. Visit Project Browser pages and note the button color.

Proposed resolution

Apply CSS to make the button look correct in BOTH light mode and dark mode with Gin.

CommentFileSizeAuthor
#7 lightmode.png74.34 KBlibbna
#7 after(dark).png72.36 KBlibbna
#7 before (dark).png80.14 KBlibbna
Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

chrisfromredfin created an issue. See original summary.

anirudhsingh19’s picture

Assigned: Unassigned » anirudhsingh19

working on it.

anirudhsingh19’s picture

Assigned: anirudhsingh19 » Unassigned
Status: Active » Needs review

Improved the colors of the select button. When using the gin admin theme for both in light and dark mode.

anirudhsingh19’s picture

Assigned: Unassigned » anirudhsingh19
Status: Needs review » Active

@phenaproxima Thanks for the feedback. I'll look into it and try to accomplish it using classes. But, in the gin.css file there is already a huge amount of styling done using CSS IDs (#project-browser), I'm not sure why it has been used.

anirudhsingh19’s picture

Assigned: anirudhsingh19 » Unassigned
Status: Active » Needs review

Used CSS classes instead of IDs, also resolved the stylelint errors which were leading to pipeline failure. Moving to Needs review.

libbna’s picture

StatusFileSize
new80.14 KB
new72.36 KB
new74.34 KB

I have reviewed MR !833. Earlier, the buttons appeared white, but after applying the changes from this MR, it now displays correctly in dark mode. I also checked in light mode, and the buttons look good there as well.

phenaproxima’s picture

Screenshots look good to me, no objections to the CSS (although I'm not really the one to make the determination about that).

chrisfromredfin’s picture

Status: Needs review » Fixed

Tested and works great. Thanks everyone! Merged to 2.1.x and cherry-picked clean to 2.0.x

Status: Fixed » Closed (fixed)

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