fix(web): center composer skill labels - #6043
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
ApprovabilityVerdict: Approved dd4074b This is a minor CSS styling fix that removes a vertical offset class from skill chip labels, centering them by using the standard chip label class. The change is purely visual with no runtime behavior impact. You can customize Macroscope's approvability policy. Learn more. |
|
Can confirm that this is an issue. My agent fixed it in exactly the same way. |
|
Note 🤖 GPT-5.6 Sol responding on behalf of Theo We're closing this PR as we clean up the T3 Code backlog. Thank you for taking the time to put this together. Closing as already implemented in #7150. Skill labels on current main use the shared centered chip label class, and the skill-only vertical offset is gone. The alignment change in this PR no longer needs a separate merge. If you believe we closed this in error, please reopen the PR and leave a comment explaining what we missed. If GitHub does not let you reopen it, leave a comment here and we'll take another look. |
What Changed
Composer skill chips now use the shared inline chip label alignment. The skill-only
top-[0.15em]offset was removed.Why
The chip flex container already centers the icon and label. A previous prompt baseline fix added an extra vertical offset to only the skill label, pushing it below the icon even though the outer chip alignment was correct. Removing that internal offset keeps the baseline behavior and centers the chip content.
UI Changes
Before
After
Verification
Checklist
Model and harness: GPT-5.6-Sol through Codex in T3 Code.
Note
Low Risk
Cosmetic composer chip styling only; no logic, data, or API changes.
Overview
Composer skill chips now use the shared
COMPOSER_INLINE_CHIP_LABEL_CLASS_NAMEfor their text instead of a skill-only label class that appliedrelative top-[0.15em].That extra vertical offset is removed from
composerInlineChip.ts, so the skill name lines up with the icon the same way other inline chip labels do, relying on the chip’s flex alignment instead of a per-label nudge.Reviewed by Cursor Bugbot for commit dd4074b. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Center composer skill chip labels by removing skill-specific top offset
Removes
COMPOSER_INLINE_SKILL_CHIP_LABEL_CLASS_NAMEfrom composerInlineChip.ts, which applied a relative top offset to skill chip labels. ComposerSkillDecorator now uses the genericCOMPOSER_INLINE_CHIP_LABEL_CLASS_NAME, centering the label visually.Macroscope summarized dd4074b.