fix(ui): align PlayOutline icon with filled Play shape#4169
fix(ui): align PlayOutline icon with filled Play shape#4169emir-karabeg merged 1 commit intostagingfrom
Conversation
The PlayOutline icon had a non-standard viewBox and mismatched path, causing it to render at an inconsistent size and shape compared to the filled Play icon and other action bar icons.
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview Reviewed by Cursor Bugbot for commit 152f763. Configure here. |
Greptile SummaryCorrects the Confidence Score: 5/5Safe to merge — single-file visual correction with no functional changes. One-file icon fix with no logic, no state, no API changes. The corrected viewBox and updated path are straightforward and consistent with the filled Play shape. No P1 or P0 findings. No files require special attention. Important Files Changed
Class Diagram%%{init: {'theme': 'neutral'}}%%
classDiagram
class Play {
+width: 10
+height: 10
+viewBox: 0 0 10 10
+fill: none (SVG)
+path fill: currentColor
+render() JSX
}
class PlayOutline {
+width: 24
+height: 24
+viewBox: 0 0 24 24 (fixed)
+fill: none
+stroke: currentColor
+strokeWidth: 1.75
+path: scaled rounded shape (fixed)
+render() JSX
}
note for PlayOutline "Before: viewBox='-1 -2 24 24', misaligned triangular path\nAfter: viewBox='0 0 24 24', rounded shape matching Play"
Play ..> PlayOutline : same rounded shape
Reviews (1): Last reviewed commit: "fix(ui): align PlayOutline icon with fil..." | Re-trigger Greptile |
Summary
PlayOutlineicon viewBox from-1 -2 24 24to standard0 0 24 24Playpath so both icons share the same rounded shapeType of Change
Testing
Checklist