fix(wall): uniform cell widths + halve column count to kill h-scroll - #3
Merged
Merged
Conversation
GitHub's table renderer was auto-sizing columns based on content despite <td width="55">, producing visibly uneven cell widths on the org page. Adding an explicit <colgroup> with one <col width="55"/> per column gives the renderer authoritative column widths that take precedence over content-based sizing. Kept <td width="55"> as a fallback in case colgroup is stripped by a future sanitizer change.
sakanni
changed the title
3 of 4 tasks
sakanni
added a commit
that referenced
this pull request
Jun 8, 2026
fix(wall): GRID_COLS 12 -> 6 (the change that missed PR #3)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Two layout issues observed after PR #2 merged to main, fixed in this single PR:
1. Cells were still visibly uneven
GitHub's table renderer was auto-sizing columns based on content despite the per-cell `<td width="55">` attribute. Adding an explicit `` with one `<col width="55"/>` per column gives the renderer authoritative column widths that take precedence over content-based sizing.
2. Horizontal scrollbar was still present
Root cause: column count is the lever, not avatar size. A table can't shrink narrower than `GRID_COLS × CELL_WIDTH`, and 660px (12 cols × 55px) exceeded many viewports - especially mobile and narrow desktops with sidebars open.
Drop `GRID_COLS` 12 → 6, so total table width becomes 6 × 55 = 330px. Fits viewports down to ~360px (most narrow mobile widths).
Trade-off
Wall is now 26 rows tall instead of 13 for 151 contributors. Per design discussion, row count is not constrained - height is fine.
Tests