fix(emcn): prevent leading glyph clipping in chip inputs - #6833
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryLow Risk Overview Adds a Vitest/jsdom regression test that asserts those spacing classes are present on the rendered input. Reviewed by Cursor Bugbot for commit a43693a. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThe PR prevents leading glyph clipping in the shared chip input by adding equal negative-margin and text-indent utilities, preserving the existing visual alignment.
Confidence Score: 5/5The PR appears safe to merge with no actionable correctness, security, or repository-rule issues identified. The balanced negative margin and text indentation provide the intended glyph clearance while preserving alignment, and the new regression test protects that class-level contract.
|
| Filename | Overview |
|---|---|
| packages/emcn/src/components/chip-input/chip-input.tsx | Adds balanced margin and indentation utilities to create leading paint clearance without changing text alignment. |
| packages/emcn/src/components/chip-input/chip-input.test.tsx | Adds jsdom coverage confirming that the shared input retains both required spacing classes. |
Reviews (1): Last reviewed commit: "fix(emcn): prevent leading glyph clippin..." | Re-trigger Greptile
Summary
Type of Change
Testing
Checklist