Skip to content

feat(components): add dedicated "copy value" context menu option to document card COMPASS-10041 #7919

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.

Sign up for GitHub

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

Closed
gribnoysup wants to merge 2 commits into from
Closed

feat(components): add dedicated "copy value" context menu option to document card COMPASS-10041 #7919

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
aabad33
feat(components): add dedicated copy value context menu option
gribnoysup Mar 27, 2026
0b919fd
Merge branch 'main' into COMPASS-10041-copy-value
gribnoysup Mar 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions packages/compass-components/src/components/document-list/element.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -573,6 +573,12 @@ export const HadronElement: React.FunctionComponent<{
);
},
},
{
label: 'Copy value',
onAction: () => {
void navigator.clipboard.writeText(element.toEJSON());
Copy link
Copy Markdown
Member

@Anemy Anemy Mar 27, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

Should we make the value shell syntax? We'd want to update the copy field and value to be shell syntax as well then. These changes look good if we want to go with EJSON here.
*Looking at it more, we'd also want to change the Copy Document to be shell syntax instead of EJSON for consistency then.

Copy link
Copy Markdown
Collaborator Author

@gribnoysup gribnoysup Apr 8, 2026

Choose a reason for hiding this comment

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

Element doesn't have methods for this, so we will need to extend it first, add tests, etc. This was supposed to be 2 story points that we just do during triage and I don't really have time to do this change too, I'll move this back to backlog

},
},
type.value === 'String' && isValidUrl(value.value)
? {
label: 'Open URL in browser',
Expand Down
Loading