search

discord-extensions / modern-indicators

Updates Discord's indicators to feel more modern.
MIT License
134 stars 7 forks source link

Theme is broken #7

Closed literon36 closed 1 year ago

literon36 commented 1 year ago

It no longer shows unread/connected/current channels correctly. (on BetterDiscord) image

cobrabr commented 1 year ago

Same is happening while using it with Vencord.

LuckFire commented 1 year ago

This is most likely due to Discord updating the class names. I will work on fixing this when I have the chance to.

cobrabr commented 1 year ago

You're right. I was taking a quick look, it seems like they've removed the content-1gYQeQ class, and the divs that used to have them now have no class assigned to them: image

I fiddled a bit with it, tried adding the class back in (it still exists, but breaks the design; they seem to have moved the padding and/or margin styles to other places in the CSS, so adding the class back in causes the channel names to be extra-indented) and also tried replacing the class with > div and :first-child in the CSS (it kinda works, but looks wonky).

LuckFire commented 1 year ago

I wonder if that's intentional or not.. is this on all clients (stable, ptb, canary) or is it just specific ones?

Edit: I confirmed this to be on stable and canary, likely the case on ptb too.

LuckFire commented 1 year ago

As of this commit and this pull request, the issue should be fixed.

literon36 commented 1 year ago

Thanks for fixing! Really like the look of this neat little theme!