search

CapnKitten / Material-Discord

Material design theme for Discord
356 stars 154 forks source link

Minor theme inconsistencies #220

Open s1nnohlakes opened 1 month ago

s1nnohlakes commented 1 month ago
Found some minor theme inconsistencies throughout. Bug Image
Notes box is not styled properly [✅ FIXED] image
App authorization page left unstyled / broken image
Discord Shop partially broken / unstyled image
Search carousel partially unstyled [✅ FIXED] image
Pinned messages carousel has this weird line at the bottom of the title (not sure if it was intentional but it doesn't fit) [✅ FIXED] image
Emoji title headers cuts off Nitro gradient in the emoji picker [✅ FIXED] image
Server images in emoji picker are slightly cut off [✅ FIXED] image
Nitro gift page has been broken for a while now [✅ FIXED] image
CapnKitten commented 1 month ago

All but the Discord shop are fixed; that's the new layout that isn't out for everyone yet. The authorization modal won't be styled right away.

s1nnohlakes commented 1 month ago

Alright, thanks for your work! I forgot to include one weird bug that has been happening since about a few weeks ago or so, and one other inconsistency.

Weirdly, when you're typing a message in a DM, and it happens to be long enough, the chat box starts to weirdly affect and resize other elements around. I don't think this is intentional behavior, as this doesn't happen normally. It only happens on DMs, not on servers.

Video reproducing the bug:

https://github.com/user-attachments/assets/c22bde5a-4dcd-469e-99bd-7747fd53eaea

Some text boxes throughout the UI are not styled properly: image image image image image

CapnKitten commented 1 month ago

The weird DM profile issue will be fixed. As for the input fields, those are properly styled using the filled style of MD3. I chose the filled style because not all of the input fields have labels for the animation. I try to follow MD3 as closely as possible and you can see the components here directly from Google.

s1nnohlakes commented 1 month ago

Ah I see, I see. I suppose I didn't take that in mind, my apologies.

s1nnohlakes commented 1 month ago

One more thing, the apps picker dialog isn't really properly styled either (or at least a part of it) image

CapnKitten commented 1 month ago

Fix for that is now live.

s1nnohlakes commented 1 month ago

Found another inconsistency; these two elements don't have the same size. image

CapnKitten commented 1 month ago

Fix is live.

s1nnohlakes commented 1 month ago

Another one that came up recently; weird behaviors with the new message context modal (not sure if this is out for everyone)

With theme installed:

https://github.com/user-attachments/assets/e156d06a-45b1-473c-bd86-ce16ffb16289

Intended behavior (Without theme):

https://github.com/user-attachments/assets/684678f3-62d5-4726-98e3-e907b370b25e

CapnKitten commented 1 month ago

Ah yea, that was recently updated by Discord. Fix live.

s1nnohlakes commented 1 month ago

Looks like Discord made some changes, here are the latest bugs I've found so far. image image image image image image image

Also, not sure if this is intentional, but the Nitro page, when opened on the DMs page, has a gray background (which doesn't really fit as opening the Nitro page in settings reveals a dark background instead. image

s1nnohlakes commented 1 month ago

Update: Now the "Add Status" button is broken as well. image

CapnKitten commented 1 month ago

That was fixed 3 days ago.

s1nnohlakes commented 1 month ago

The Add Status button still seems to be unfixed on my side