search

teia-community / teia-ui

Front-end code for teia
https://teia.art
MIT License
26 stars 36 forks source link

Update index.module.scss #395

Closed Theohorsmeier closed 8 months ago

Theohorsmeier commented 8 months ago

added min-height to audio to fix the audio player having zero height

melMass commented 8 months ago

Thanks!

I wonder if it could be a bit smaller though, here is 32px:

image

vs 54px:

image

It might be nice to apply theme too, but it can be a separate PR

Theohorsmeier commented 8 months ago

Oh yeah 32 does look better tbh .. I mostly picked 54 because that seemed to have been default somewhere 🫣

On Thu, Mar 14, 2024, 21:01 Mel Massadian @.***> wrote:

Thanks!

I wonder if it could be a bit smaller though, here is 32px:

image.png (view on web) https://github.com/teia-community/teia-ui/assets/7041726/5aef40aa-45c5-4cb8-958e-38871753f64a

vs 54px:

image.png (view on web) https://github.com/teia-community/teia-ui/assets/7041726/2a90e6a5-e4f3-43a4-a8ab-17c2cae0d602

It might be nice to apply theme too, but it can be a separate PR

— Reply to this email directly, view it on GitHub https://github.com/teia-community/teia-ui/pull/395#issuecomment-1998325713, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEF552OHEDGHFYWEGARTU5LYYH63BAVCNFSM6AAAAABEWXH6HSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOJYGMZDKNZRGM . You are receiving this because you authored the thread.Message ID: @.***>