Fix #1592 Miniplayer bugged controls

CarlosBor commented 2 months ago

Fixes some miniplayer dropdown buttons being unclickable

nuki-chan[bot] commented 2 months ago

Lines 18-21

Ohayou, Code Adventurer~! ✨ Nuki is here to sprinkle some advice on this... "neumorphic" piece of art! 🖌️

First things first, what's with the z-index: 1000; joining the party out of nowhere? 🎉 Going overboard with z-index values is like using a katana to slice a single strawberry—cool but unnecessary! ✂️ Try using lower incremental values. This makes it easier to manage layers and prevents future "z-index wars". 💥
Also, Nuki thinks hardcoded values are like yesterday's sushi... not the best idea! 😸 Maybe this z-index needs to be dynamic or based on a variable? Think it through like a puzzle game where the high score is maintainable code! 🧩💕

Here's a cute and maintainable suggestion, just for you:

$z-index-small: 10; // You can adjust and place it with other z-index values at the top!

&.small {
  box-shadow: $smallDisplacement $smallDisplacement $smallBlur darken($background, $contrast), 
    (-$smallDisplacement) (-$smallDisplacement) $smallBlur lighten($background, $contrast);
  z-index: $z-index-small;
}

Remember, following Nuki’s advice isn't mandatory to get your PR merged; she's still powering up her skill levels! 🌟

nukeop commented 2 months ago

I think it's fine, it's a single screen after all. Thanks for fixing this.

nukeop / nuclear

Fix #1592 Miniplayer bugged controls #1595

Lines 18-21