Fix Recent Activity table collapsing FILE column on narrow screens (#217) - #219
Conversation
…tudioNirin#217) The fixed-width columns add up to 500px, so on a phone in portrait (or at high browser zoom) the auto-width FILE column was left with ~0px. word-break: break-word (alias of overflow-wrap: anywhere) plus min-width: 0 then let it collapse to one character per line. - give the table a min-width so FILE keeps ~220px - enable horizontal scrolling on .recent-activity-scroll - use overflow-wrap: break-word for the filename cell
…n#217) Below a card width of 720px use narrower fixed columns so the FILE column is reachable with a short swipe. Uses a container query so it follows the card width rather than the viewport (the sidebar also takes space on desktop).
…eways (StudioNirin#217) The run header spans all five columns, so its right-aligned stat pills end up off-screen when the table is scrolled horizontally on a narrow card. Make the inner flex box sticky and as wide as the visible card.
|
Here is how I got to this fix, what I found, and why there are two optional commits. Which commit matters
ReproductionI rebuilt the Dashboard table in headless Chromium using the real What was actually wrongIn my first comment on #217 I suspected that horizontal scrolling was missing. That was wrong. The table does scroll sideways, but only over the 500px taken by the four fixed columns (130 + 130 + 130 + 110px), where FILE has no width. Commit 1: the actual fixImportant Commit 1 is the fix. Commits 2 and 3: optionalNote Both are optional and can be dropped without affecting the fix. Commit 3 depends on commit 2. After applying commit 1 on my phone, two things were still awkward (the numbers below are from the headless test at 360px, where the card is ~311px wide). TIME and ACTION still use ~260px, so FILE only starts near the right edge and you have to swipe before you see a filename. And the run header (title on the left, "2 restored · 1.31 GB" pills on the right) spans all five columns, so the pills sit off-screen and, after swiping, the title scrolls away and the pills are left without context.
Side effects I checked
TestedFirefox And Chrom both On my phone in portrait and on desktop at 160% zoom with the sidebar expanded and collapsed, plus the headless Chromium comparisons above. I couldn't test Safari. The analysis, fix, tests and commit split were done with Claude Sonnet 5.5; I reviewed the result, tested it on my own devices and opened the PR. |
|
I pulled this and compared I'm happy with all three as they are. @StudioNirin to answer your question on #217, this only touches the Recent Activity table's own CSS, so nothing else on the Dashboard or the other tables is affected. From my side it's good to merge. |
|
Thats fine then, all good here. @DeLo1585 if this is finalised I will merge it |
|
Yes, fir FIRST COMMIT is final from my side, no further changes planned. Thanks both for pulling it and testing it on your setups! Please go ahead and merge. |
You only want one commit merged and not the whole PR? Might be a mobile limitation but im not seeing a way to do that, seems to be all or nothing (I dont use git a whole lot). |
|
Sorry for the confusion, that was a slip in my wording. Please merge the whole PR (all three commits). Brandon tested and approved all three, and I have no further changes planned. Thanks! |
Fixes #217.
On narrow screens (phone in portrait, high browser zoom) the Recent Activity table's FILE column collapsed to ~0px: the four fixed-width columns add up to 500px, and
word-break: break-word(alias ofoverflow-wrap: anywhere) plusmin-width: 0let the filename wrap one character per line and made the FILE and USERS headers overlap.Three commits, only
web/static/css/custom.csschanges:min-widthon the table, horizontal scroll on.recent-activity-scroll,overflow-wrap: break-wordfor the filename cell.Independent of #218 (different sections of the file, no conflicts). Happy to drop commits 2 and 3 if they don't hold up on other setups.
Testing
AI assistance
The root-cause analysis, the fix, the tests in headless Chromium, the commit split and this description were done with Claude Sonnet 5.5 (Anthropic). I reviewed the result, applied the patches to a branch based on
main, tested it on my own devices, pushed and opened this PR.