Skip to content

fix(logs): port log detail buttons to Button priority=transparent#111824

Merged
JoshuaKGoldberg merged 1 commit intomasterfrom
JoshuaKGoldberg/ref/log-detail-buttons-transparent
Mar 31, 2026
Merged

fix(logs): port log detail buttons to Button priority=transparent#111824
JoshuaKGoldberg merged 1 commit intomasterfrom
JoshuaKGoldberg/ref/log-detail-buttons-transparent

Conversation

@JoshuaKGoldberg
Copy link
Copy Markdown
Member

@JoshuaKGoldberg JoshuaKGoldberg commented Mar 30, 2026

Removes some of the old custom styles in favor of our standard Scraps transparent Button.

Before After

Fixes EXP-860

Made with Cursor

Use the standard Scraps Button with priority="transparent" and icon
prop for the expanded log detail action buttons instead of
priority="link" with inline icon children and manual padding.

Refs LINEAR-EXP-860
Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>

Made-with: Cursor
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 30, 2026
@JoshuaKGoldberg JoshuaKGoldberg changed the title ref(logs): Port log detail buttons to Button priority=transparent fix(logs): port log detail buttons to Button priority=transparent Mar 30, 2026
@JoshuaKGoldberg JoshuaKGoldberg marked this pull request as ready for review March 30, 2026 20:56
@JoshuaKGoldberg JoshuaKGoldberg requested a review from a team as a code owner March 30, 2026 20:56
@JoshuaKGoldberg JoshuaKGoldberg requested a review from a team March 30, 2026 20:58
@linear-code
Copy link
Copy Markdown

linear-code bot commented Mar 30, 2026

@JoshuaKGoldberg JoshuaKGoldberg merged commit ce9f5e3 into master Mar 31, 2026
93 of 95 checks passed
@JoshuaKGoldberg JoshuaKGoldberg deleted the JoshuaKGoldberg/ref/log-detail-buttons-transparent branch March 31, 2026 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants