Skip to content

fix(sidebar): match native icon size - #17

Merged
Ericwong5021 merged 5 commits into
mainfrom
fix/issue-16-native-icon-size
Sep 18, 2026
Merged

Ericwong5021 merged 5 commits into
mainfrom
fix/issue-16-native-icon-size

Conversation

@Ericwong5021

Copy link
Copy Markdown
Owner

Closes #16

Changes

  • Read the native Codex sidebar SVG's computed width and height.
  • Apply those dimensions to the injected Better Codex navigation icon through CSS custom properties.
  • Keep the existing 16px design-token size as a compatibility fallback.
  • Regenerate the injected UI bundle and add regression coverage.

Validation

  • npm test — 288 tests, 275 passed, 13 skipped, 0 failed
  • npm run typecheck — passed

@Ericwong5021
Ericwong5021 merged commit f69a3ed into main Sep 18, 2026
21 of 22 checks passed
@Ericwong5021 Ericwong5021 mentioned this pull request Sep 21, 2026
1 task done
@Ericwong5021
Ericwong5021 deleted the fix/issue-16-native-icon-size branch October 4, 2026 06:36
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.

[Bug]: icon 大小不匹配

1 participant