jellyfin / jellyfin-web

Web Client for Jellyfin

https://jellyfin.org

GNU General Public License v2.0

2.42k stars 1.28k forks source link

Fix form dialog modal unnecessary footer button text wrapping #6250

Open tylers-username opened 1 month ago

tylers-username commented 1 month ago

Changes Change form formDialogContent footer buttons (formDialogFooterItem) to only wrap text when the width of the footer area has been completely consumed.

Issues

6211

jellyfin-bot commented 1 month ago

Cloudflare Pages deployment

Latest commit	`4bd44e4`
Status	✅ Deployed!
Preview URL	https://10e5028d.jellyfin-web.pages.dev
Type	🔀 Preview

View build logs

dmitrylyzo commented 1 month ago

Need to test this on webOS 1.2 - flex-basis: fit-content may not be supported.

UPD: webOS 1.2 and webOS 5 don't support `flex-basis: fit-content`. Buttons have been rearranged in rows.	Before	After

Just a note, buttons now have different sizes. When changing the UI, it is always better to provide “before” and “after” screens.	Before	After

Technically #6211 has been fixed by #6212.

sonarcloud[bot] commented 2 weeks ago

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud