fix(webgui): scale task sheet width with viewport - #2716
Conversation
WalkthroughThe Nchan sheet fallback width now uses ChangesNchan sheet sizing
Estimated code review effort: 1 (Trivial) | ~5 minutes Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
🔧 PR Test Plugin AvailableA test plugin has been generated for this PR that includes the modified files. Version: 📥 Installation Instructions:Install via Unraid Web UI:
Alternative: Direct Download
|
Summary
Tracked-task sheets open too narrow on desktop. This change scales their default width with the browser viewport and keeps the responsive safeguards.
Why This Exists
The foreground sheet shows task logs and progress output. The
60remdefault leaves unused desktop space and causes unnecessary text wrapping.Resolution
The shared Nchan sheet now uses
clamp(60rem, 70vw, 100rem)as its default. The existing90vwcap keeps it inside narrow desktop viewports.The full-screen mobile rule, stored widths, and
600pxresize minimum remain unchanged.Reviewer Considerations
600px.320pxwide.Behavior Changes
60remto100remwith the browser viewport.Implementation Summary
--nchan-sheet-widthtoclamp(60rem, 70vw, 100rem).90vwas the outer viewport cap.Verification
php -l emhttp/plugins/dynamix/include/DefaultPageLayout/BodyInlineJS.phpcompleted with no syntax errors.git diff --checkcompleted with no errors.600pxat 768px,840pxat 1200px, and1000pxat 1440px and 1920px.100vwmobile override.Risk
Low. This change updates one CSS fallback expression and comments. It does not change task state, stored width handling, or mobile behavior.
Related to OS-645.
Follow-up to OS-377.