Skip to content

chore(fe): action popover item UX#8831

Draft
jmelahman wants to merge 1 commit intomainfrom
jamison/actions-disable-follow-up
Draft

chore(fe): action popover item UX#8831
jmelahman wants to merge 1 commit intomainfrom
jamison/actions-disable-follow-up

Conversation

@jmelahman
Copy link
Contributor

@jmelahman jmelahman commented Feb 26, 2026

Description

Closes https://linear.app/onyx-app/issue/ENG-3756/actions-not-set-up-state-needs-to-be-consistent

How Has This Been Tested?


Summary by cubic

Standardizes how Actions popover items display “not set up” and “unavailable” states by introducing a true disabled style, separate from strikethrough. Addresses ENG-3756 by making the state consistent and clearer.

  • Bug Fixes
    • Added a disabled variant to LineItem (prop + styles for button/text/icon) via new classes: line-item-text-disabled and line-item-icon-disabled.
    • Updated ActionLineItem to use disabled for “no connectors”/“unavailable,” and reserve strikethrough for explicitly disabled actions; click remains blocked when disabled.

Written for commit b05c7b7. Summary will update on new commits.

@github-actions
Copy link
Contributor

🖼️ Visual Regression Report

Project Changed Added Removed Unchanged Report
admin 1 0 0 108 View Report
exclusive 0 0 0 8 ✅ No changes

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