Skip to content

Improve open-age chart layout, colors and scroll stability - #17

Merged
zhaozhaozz merged 1 commit into
mainfrom
feature/open-age-pie-layout
Oct 15, 2026
Merged

zhaozhaozz merged 1 commit into
mainfrom
feature/open-age-pie-layout

Conversation

@zhaozhaozz

Copy link
Copy Markdown
Contributor

The open-age charts on the Issue and PR pages occupied an entire row, used four similar blue shades, and scrolled the page when filtering restored keyboard focus.

Place each chart and legend beside the open-list heading, view switch and filters on wide screens, stacking them on narrow screens. Keep the table or board full width below. Use the existing categorical colors --s2, --s4, --s7 and --s3, with a dark selection outline and no sector translation.

Reserve space for the age-filter chip and keep the chart subtitle constant. Preserve the viewport across rendering and restore focus with preventScroll; shorter results retain enough page height to avoid clamping the saved scroll position. The board controls share the same header, while other summary tables retain their positions and filtering behavior. Bump all five static resource versions to 20261015.4.

Validation:

  • Updated open-age Playwright journey passed, covering bucket boundaries, selection and cancellation, chip clearing, search intersection, independent Issue/PR filters, table/board views and unaffected side tables.
  • The journey checks 1440px and 390px layouts in light/dark themes, distinct categorical colors, no horizontal page overflow, and scroll/header/list-position changes of at most 2px after filtering.
  • The other browser journeys: 23 passed, 1 existing failure in the Real E2E score-curve test (.score-failed: expected 1, found 0). Reproduced the same missing mark with baseline 87608be4 scripts: 12 run marks, 11 score dots, 0 failed-score marks. This unrelated test is unchanged.
  • node --check static/app.js, node --check static/board.js, and git diff --check passed.

No collection was manually dispatched.

@zhaozhaozz
zhaozhaozz merged commit 690351f into main Oct 15, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant