search

tsoding / musializer

Music Visualizer
MIT License
877 stars 92 forks source link

Make timeline height adaptive #77

Closed AMurkin closed 6 months ago

AMurkin commented 6 months ago

1/6 of total height, but 24px min and 150px max

rexim commented 6 months ago

Why?

AMurkin commented 6 months ago

Because now it gets too big for small window size and disproportionately small for big window size.

rexim commented 6 months ago

That's the case for the majority of apps with side panels.

AMurkin commented 6 months ago

Unless its UI is adaptive maybe? Dunno. A matter of taste, perhaps.

rexim commented 6 months ago

Yeah, I also think it's a matter of taste. Thank you for the PR!