Repository navigation
style(ui): left-align every table column - #1366
Merged
Merged
Conversation
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
life-ustc | 2171f45 | Oct 24 2026, 08:15 PM |
Contributor
|
🎉 This PR is included in version 1.133.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
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
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.
Goal
Closes #1365. Every application table column aligns left, including numeric, date, status and action columns.
Changed Surfaces
Update table consumers and the shared filter header; keep labels before their filter/sort indicators and preserve tabular numerals. Bus stop columns follow the same rule.
The homepage had only 2 gzip bytes of budget headroom, so changed route hashes exceeded the existing limit. Use equivalent native Svelte props in
AppTopbarto remove 70 gzip bytes from the shared root module; its defaults, callbacks and markup are unchanged.Evidence
Full branch CI: https://github.com/Life-USTC/server/actions/runs/40128835583 (38/38 jobs, commit
2171f4521).bun run check: 572 files / 4,136 tests passed; Svelte, TypeScript, specifications and OpenAPI checks passed. Production and test Worker builds passed. All six public-route client budgets passed; the homepage is 209,932 / 210,000 gzip bytes with 69 initial asset requests.All 63 affected table and shell browser cases passed, plus four matched-evidence and filter/multiselect/comparison/pagination cases. No retries or skips. All six mobile page screenshots remain byte-identical to the baseline.
Production verified at
34e226c6: both viewports passed section/teacher alignment, multi-select and comparison filtering, pagination and reload preservation, with no page errors.Matched zh-cn screenshots use identical fixtures at 1280×900 and 390×844.
Docs / Contracts
Update the four existing UI alignment requirements. Existing browser tests observe left text edges and cell alignment across 28 table roots with unchanged pixel tolerances; the source guard now rejects both right- and center-aligned columns.
Risk Areas
Shared filter-header positioning, responsive tables and topbar reactivity; verified with existing filter, sorting, pagination, layout, menu, theme and locale cases.
Cleanup
No stock UI primitives, generated files or one-time scripts changed.