Skip to content

fix(ui): align Skills filters#100526

Merged
steipete merged 1 commit into
mainfrom
codex/pr-99996-skills-filter-clean
Jul 6, 2026
Merged

fix(ui): align Skills filters#100526
steipete merged 1 commit into
mainfrom
codex/pr-99996-skills-filter-clean

Conversation

@steipete

@steipete steipete commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

What Problem This Solves

The Control UI Skills page rendered a raw missing translation key for the agent filter and vertically misaligned the adjacent agent and search controls. The initial CSS fix also applied a fixed height to every shared .field input, which would stretch unrelated checkbox and radio controls.

Fixes #99990. Supersedes #99996 after its contributor fork could not retain clean rebased ancestry; contributor credit is preserved through co-authorship and the maintainer release-note batch.

Why This Change Was Made

  • use the existing translated usage.filters.agent label;
  • give the search field the existing translated common.search label so both filter fields have the same two-row structure;
  • align only text-like inputs and selects at 38px;
  • explicitly exclude checkbox/radio inputs from the shared fixed-height rule;
  • cover translated labels and real Chromium control dimensions.

This keeps the fix at the shared form-control boundary without changing Skills filtering behavior or broadening checkbox/radio styling.

User Impact

Skills filters now show Agent and Search with aligned controls. Checkbox and radio controls elsewhere in the Control UI retain their native sizing.

Evidence

One focused maintainer commit; original contributor preserved with Co-authored-by: 杨爱文 <[email protected]>.

AI-assisted: Yes (Codex).

@openclaw-barnacle openclaw-barnacle Bot added app: web-ui App: web-ui size: XS maintainer Maintainer-authored PR labels Jul 6, 2026
@steipete

steipete commented Jul 6, 2026

Copy link
Copy Markdown
Contributor Author

Land-ready maintainer pass complete on aeec3b6311caa9cca9d3dffc9adc96f35bf5ebe2.

Verification:

No known proof gaps for this Control UI styling/i18n change.

@steipete
steipete merged commit f12312d into main Jul 6, 2026
92 of 96 checks passed
@steipete

steipete commented Jul 6, 2026

Copy link
Copy Markdown
Contributor Author

Merged via squash.

@steipete
steipete deleted the codex/pr-99996-skills-filter-clean branch July 6, 2026 01:24
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request Jul 6, 2026
giodl73-repo pushed a commit to giodl73-repo/openclaw that referenced this pull request Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

app: web-ui App: web-ui maintainer Maintainer-authored PR size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Style / UI bug (visual misalignment without crash)

1 participant