Skip to content

fix(layout): let info tables wrap and give PHP extensions the full width - #1069

Merged
kesselb merged 1 commit into
masterfrom
fix/table-overflow-and-extensions
Aug 2, 2026
Merged

kesselb merged 1 commit into
masterfrom
fix/table-overflow-and-extensions

Conversation

@karlitschek

Copy link
Copy Markdown
Member

Core's CSS reset applies white-space: nowrap to every table, which the cells inherit. A long value — the CPU model line is the obvious one — then cannot wrap, so the table grows past its column and prints on top of the section beside it. Reset it on the info tables and let only the value column wrap; keeping nowrap on the label column also stops the auto table layout from squeezing the labels to make room for a long value.

The PHP extension list was a cell inside that same half-width table, which left a long list of pills crammed into three or four to a row. It moves to its own component on a full-width column after PHP and Database, so it still reads as part of that section but has the whole row to flow across.

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Undo core's `white-space: nowrap` on the info tables, so long values and
translated labels wrap instead of growing the table past its column and
printing over the section beside it.

Move the PHP extension list out of that half-width table into its own
full-width section, so the pills flow across the whole row.

Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Frank Karlitschek <karlitschek@users.noreply.github.com>
@kesselb
kesselb force-pushed the fix/table-overflow-and-extensions branch from 33ad668 to 35c3d63 Compare August 2, 2026 22:07
@kesselb
kesselb enabled auto-merge August 2, 2026 22:07
@kesselb kesselb added this to the Nextcloud 35 milestone Aug 2, 2026
@kesselb kesselb self-assigned this Aug 2, 2026
@kesselb
kesselb merged commit c1633c7 into master Aug 2, 2026
38 checks passed
@kesselb
kesselb deleted the fix/table-overflow-and-extensions branch August 2, 2026 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants