ClearVision / ClearVision-v6

Highly customizable theme for BetterDiscord.
https://clearvision.github.io/
Apache License 2.0
385 stars 1.89k forks source link

More bugs squashed #63

Closed NyxIsBad closed 3 years ago

NyxIsBad commented 3 years ago

-> Fixed bad margin in quickswitcher -> Fixed video call icon background -> Fixed some text being unreadable

I'm pretty sure this is most, if not all, bugs for CV 6.1.0 on stable as of right now. Unsure if this is a thing we want to do but there's an unthemed element (the login screen) which is otherwise themed (the buttons and outlines are all proper but the bg is missing)

.authBox-hW6HRx{
  background: rgba(0,0,0,.8);
}

and also potentially missing a background here:

.fullScreen-1tMzXY .flexCenter-2Y_vCg{ /*Fullscreen share background - just join a private call, share something/use a video, then fullscreen it - there's no background there.*/
  background: rgba(0, 0, 0, calc(var(--background-shading) * .6));
}

The quickswitcher scroll background (and a few other scroll backgrounds) aren't the right color as well

In addition, it would be good to rewrite the create new server modal so that they start transparent and turn to hover color. This looks ugly to me: https://prnt.sc/17pdq6b

Something like https://prnt.sc/17pdsfi is much better imo

Other than these four issues, this should be it. I did have to add an !important in order to resolve the video call icon bug, but it is necessary. I avoided a second by using a :not(:hover) but if you dislike this and would rather use two using the original structure that works out just as well.

The rest is pretty self explanatory

If there's more I guess I'll send again but itll probably be until canary changes roll to stable that we have issues of a high enough significance.