Skip to content

feat(cell-action): add external linking as action #95892

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

lzhao-sentry
Copy link
Contributor

@lzhao-sentry lzhao-sentry commented Jul 18, 2025

Changes

Have external linking as a dropdown menu option for CellAction. Removed cell actions for some ID linked fields (since it conflicts with the dropdown & should just directly open the link)

Video

external.linking.mov

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jul 18, 2025
@lzhao-sentry lzhao-sentry marked this pull request as ready for review July 21, 2025 13:36
@lzhao-sentry lzhao-sentry requested review from a team as code owners July 21, 2025 13:36
cursor[bot]

This comment was marked as outdated.

Copy link
Member

@gggritso gggritso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👀 ! Two questions:

  1. In a few places the isUrl functionality is checking the feature flag, but sometimes not, is that intentional?
  2. I'm worried that people won't remember to import handleCellActionFallback, and this'll result in inconsistent fallback handling. Is it possible to handle the fallback inside CellActions?

cursor[bot]

This comment was marked as outdated.

@lzhao-sentry
Copy link
Contributor Author

  1. I'm worried that people won't remember to import handleCellActionFallback, and this'll result in inconsistent fallback handling. Is it possible to handle the fallback inside CellActions?

@gggritso I think we would need to refactor how handleCellAction works since there's no consistent way to know if an action was successfully handled. We could add a return type, but I feel like that puts more work on the user.

Would it be better to split it into two helper functions (e.g., copyToClipboard and openExternalLink)? I know it turns it into two imports, but I think the naming makes it clear the purpose of the functions?

@gggritso
Copy link
Member

@lzhao-sentry I like that idea! Go for it 👍🏻

cursor[bot]

This comment was marked as resolved.

Copy link
Member

@gggritso gggritso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏻

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