jdan / 98.css

A design system for building faithful recreations of old UIs
https://jdan.github.io/98.css
MIT License
9k stars 296 forks source link

Add `disabled` styling for Maximize title bar buttons #190

Closed vtlanglois closed 7 months ago

vtlanglois commented 7 months ago

In this PR

πŸ“ˆ Impact

With this additions, users will be able to style windows as if they cannot be maximize. This can be useful when users wish to:

Screenshot 2024-01-21 at 12 36 33β€―PM

Reasons for Drafting this PR

I have another PR related to title bar buttons up (#189). As that PR could cause merge conflicts with this PR and has changes that, if approved, need to be applied to for disabled maximize buttons to work as intended without English-text aria-labels, this will be a draft until action is done on that PR.

This prevents a possible bug from arising if work is not done to apply the changes in #189 to this PR.

πŸ“” Dev Note

I tried looking for examples of the remaining 4 title bar buttons having a disabled state, but couldn't find concrete evidence of that in Windows 98.

[^1]: styling based on multimedia application UIs screenshots from this website: https://guidebookgallery.org/screenshots/win98

vercel[bot] commented 7 months ago

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Comments Updated (UTC)
98css βœ… Ready (Inspect) Visit Preview πŸ’¬ Add feedback Jan 28, 2024 2:09am
vtlanglois commented 7 months ago

Looks good! Is it possible to apply a filter to the button to get the same effect? (Is it possible to apply this filter to other buttons/icons too?)

No need for my approval to merge these btw. Feel free to use your own discretion ✨

Thanks!

I don't have experience in using filters on SVGs, but I could see it being possible. I could look into this on my free time, ofc no guarantees with work and all. Should this be made into an Issue in case another dev is able to implement this?