Skip to content

fix(ui): align PlayOutline icon with filled Play shape#4169

Merged
emir-karabeg merged 1 commit intostagingfrom
fix/play-outline-icon
Apr 15, 2026
Merged

fix(ui): align PlayOutline icon with filled Play shape#4169
emir-karabeg merged 1 commit intostagingfrom
fix/play-outline-icon

Conversation

@emir-karabeg
Copy link
Copy Markdown
Collaborator

@emir-karabeg emir-karabeg commented Apr 15, 2026

Summary

  • Fixed PlayOutline icon viewBox from -1 -2 24 24 to standard 0 0 24 24
  • Replaced outline path with scaled version of the filled Play path so both icons share the same rounded shape
  • Ensures consistent sizing with other action bar icons

Type of Change

  • Bug fix

Testing

  • Verified the PlayOutline icon in the block action bar renders at the same size as sibling icons
  • Verified the outline shape matches the filled Play icon in the panel run button

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

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.
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 15, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Apr 15, 2026 2:36am

Request Review

@cursor
Copy link
Copy Markdown

cursor bot commented Apr 15, 2026

PR Summary

Low Risk
Low risk UI-only SVG changes; main risk is minor visual misalignment/regression where PlayOutline is used.

Overview
Updates the PlayOutline SVG to use a standard viewBox (0 0 24 24) and replaces its path with a scaled version matching the filled Play icon shape, so outline/filled play icons render consistently sized and aligned across the UI.

Reviewed by Cursor Bugbot for commit 152f763. Configure here.

@emir-karabeg emir-karabeg merged commit e281ca0 into staging Apr 15, 2026
10 checks passed
@emir-karabeg emir-karabeg deleted the fix/play-outline-icon branch April 15, 2026 02:38
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 15, 2026

Greptile Summary

Corrects the PlayOutline icon's misaligned viewBox (from -1 -2 24 24 to 0 0 24 24) and replaces the old triangular outline path with a rounded path matching the filled Play icon's shape at 24×24 scale. The fix is minimal and self-contained with no logic or API surface changes.

Confidence Score: 5/5

Safe 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

Filename Overview
apps/sim/components/emcn/icons/play.tsx Fixes PlayOutline icon by correcting viewBox from -1 -2 24 24 to standard 0 0 24 24 and replacing the misaligned outline path with a 24×24-scaled version of the filled Play shape

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
Loading

Reviews (1): Last reviewed commit: "fix(ui): align PlayOutline icon with fil..." | Re-trigger Greptile

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.

1 participant