Skip to content
This repository was archived by the owner on Apr 29, 2026. It is now read-only.

Improve packages sort dropdown readability#4

Open
ElliotBadinger wants to merge 1 commit into
earendil-works:mainfrom
ElliotBadinger:fix-packages-select-contrast
Open

Improve packages sort dropdown readability#4
ElliotBadinger wants to merge 1 commit into
earendil-works:mainfrom
ElliotBadinger:fix-packages-select-contrast

Conversation

@ElliotBadinger
Copy link
Copy Markdown

What

Fixes the packages page sort dropdown so Chrome/Linux renders the native select/options with the site dark palette instead of a light popup with low-contrast option text.

Related report: earendil-works/pi#3979

Why

The closed sort control is dark, but opening it in stock Chrome/Linux could show a white native menu with very light unselected option text. That makes the package catalog's primary sorting control hard to read.

Verification

  • blargh --in src --out /tmp/pi-website-build-check
  • Chrome 147 on Linux, clean profile: computed .pkg-sort/option styles are dark background with light text:
    • selectColorScheme = dark
    • selectBg = rgb(24, 24, 27)
    • selectColor = rgb(250, 250, 250)
    • optionBg = rgb(24, 24, 27)
    • optionColor = rgb(250, 250, 250)
  • Runtime sort smoke check: setting the select to name still updates the value and leaves the package list populated.
  • Local review pass: one reviewer approved; one verifier passed. Both noted native option rendering can vary across browser/OS, so Firefox/Safari remain smoke-test targets.

Chrome on Linux renders the packages sort select popup from native form-control styling, so the closed control looked dark while opened options could inherit a light menu surface with low-contrast option text. Scope the fix to the package sort select by declaring a dark color scheme and explicit option colors that match the existing terminal palette.

Constraint: pi-website has no local test script; verification is browser/runtime and static build based
Rejected: Replace the native select with custom markup | too broad for a contrast-only issue
Confidence: medium
Scope-risk: narrow
Directive: Keep native select verification browser-specific because option popup rendering varies by OS/browser
Tested: blargh --in src --out /tmp/pi-website-build-check
Tested: Chrome 147 Linux clean profile computed .pkg-sort and option colors as dark background with light text
Tested: Sort select change handler still updates value to name with 1864 package cards
Tested: Subagent review APPROVE and verifier PASS
Not-tested: Firefox/Safari native select popup rendering
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant