ducky / play-midnight

(DEPRECATED) Enjoy your music at night. All the time.
86 stars 92 forks source link

Zero text/bg contrast in contextual menus #34

Closed L-P closed 7 years ago

L-P commented 8 years ago

Dark text on dark background and light text on light background when highlighting.

black

Version: Play Midnight for Google Play Music™ 2.4.0

ducky commented 8 years ago

Ugh, this is becoming so hard to diagnose. You're like the 4th person to report this and yet I can't see the issue on my end.

I might need someone with html skill to look into this and give me some html markup/class names.

L-P commented 8 years ago

The original color #dcdcdc comes from a .body, .body.material selector in Play Midnight and gets overriden to #212121 in the selector .material .goog-menu .goog-menuitem .goog-menuitem-content from listenstyles.css.

The menu hierarchy is as follows:

<div class="goog-menu goog-menu-vertical album-menu">
    <div class="goog-menuheader goog-menuheader-disabled"></div>

    <!-- Repeat block per menu item {{{ -->
    <div class="goog-menuitem">
        <div class="goog-menuitem-content">
            …
        </div>
    </div>
    <!-- }}} -->
</div>

The .goog-menuitem-content get the wrong style here, their parent and children seem fine.

ducky commented 8 years ago

I seem to have a selector matching that in the Play Midnight styles, but for some reason it's being overridden. I've added an important flag on it for now since I'm not actually able to diagnose it myself. I'm hoping it's a specificity thing that can be fixed lazily for now.

I've heard from someone else who was originally having the issue that it went away, so I'm hoping it's just a slow rollout on Google's end.

ducky commented 8 years ago

I've pushed up v2.4.1, hopefully that fixes this (I just guessed), or Google fixes it on their end soon.

L-P commented 8 years ago

v2.4.1 fixed it, thanks.

~The :hover is still white on white, I'll wait and see if this is because of a slow rollout on Google's end.~

Fixed in 2.4.2.

L-P commented 8 years ago

Same bug on the queue: uv

Should I also wait on this one or do you have a repro @chrisxclash ?

ducky commented 7 years ago

Holy buns, sorry I didn't see the second part to this issue. Preferably if I can get a PR for the fixes to the menu and queue that'd be helpful for me. I still don't have this issue on my end, so I don't want to just start firing at random and hoping I can fix it. I'm going to put a post on Reddit too to see if someone can submit a PR for this that is also getting the issue.

Thanks!

L-P commented 7 years ago

I'll try fixing it this week.

ducky commented 7 years ago

I appreciate it!