Skip to content
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

Jump to bottom

Shortcut to copy value from an individual cell with alt/opt + click #1585

Closed
dsmmcken opened this issue Oct 19, 2023 · 1 comment · Fixed by #1694
Closed

Shortcut to copy value from an individual cell with alt/opt + click #1585

dsmmcken opened this issue Oct 19, 2023 · 1 comment · Fixed by #1694
Assignees
@georgecwan
Labels
enhancement New feature or request
Milestone
November 2023

Comments

@dsmmcken
Copy link
Contributor

As a user, I would like a quicker way to copy a cells value.

Copying and individual values from a cell is such a common task that it would be good to have a dedicated shortcut for doing so. Since tables in Deephaven typically use row selection mode, copying an individual cell requires using the context menu.

My proposed shortcut is an alt (opt on mac) + click of a cell to quickly copy it's value. To make it more discoverable, I suggest we switch to a custom copy cell cursor when hovering a cell with a copyable value in it and the user presses or holds alt. Further we can display the shortcut text in the context menu next to the copy cell value.

Copy cell cursor:
image

SVG file:
Copy-Cell-Cursor

Context menu:
image
@dsmmcken dsmmcken added enhancement New feature or request triage Issue requires triage labels Oct 19, 2023
@vbabich vbabich added this to the November 2023 milestone Oct 24, 2023
@vbabich vbabich removed the triage Issue requires triage label Oct 24, 2023
@dsmmcken
Copy link
Contributor Author

Make sure it also works on the header cells too.

@georgecwan georgecwan mentioned this issue Dec 14, 2023
Merged
georgecwan added a commit that referenced this issue Dec 22, 2023
@georgecwan
feat: Add alt+click shortcut to copy cell and column headers (#1694)
4a8a81a 
Adds a alt/opt+click shortcut to copy the value of a cell or column
header.

Resolves #1585.
mofojed added a commit to deephaven/deephaven-core that referenced this issue Dec 22, 2023
@github-actions @deephaven-internal @mofojed
Update web version 0.58.0 (#4982)
c245cc7 
Release notes https://github.com/deephaven/web-client-ui/releases/tag/v0.58.0

### Bug Fixes

* `figure_title` and `chart_title` were not mapped up correctly ([#1676](deephaven/web-client-ui#1676)) ([73e0b65](deephaven/web-client-ui@73e0b65)), closes [#1674](deephaven/web-client-ui#1674) [#1675](deephaven/web-client-ui#1675)


### Features

* "Group" column for rollup/tree tables ([#1636](deephaven/web-client-ui#1636)) ([ba1d51b](deephaven/web-client-ui@ba1d51b)), closes [#1555](deephaven/web-client-ui#1555)
* Add alt+click shortcut to copy cell and column headers ([#1694](deephaven/web-client-ui#1694)) ([4a8a81a](deephaven/web-client-ui@4a8a81a)), closes [deephaven/web-client-ui#1585](deephaven/web-client-ui#1585)
* Theming - Spectrum variable mapping and light theme ([#1680](deephaven/web-client-ui#1680)) ([2278697](deephaven/web-client-ui@2278697)), closes [#1669](deephaven/web-client-ui#1669) [#1539](deephaven/web-client-ui#1539)




---------

Co-authored-by: deephaven-internal <deephaven-internal@users.noreply.github.com>
Co-authored-by: mikebender <mikebender@deephaven.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@georgecwan georgecwan

Labels
enhancement New feature or request
Projects
None yet
Milestone
November 2023
Development

Successfully merging a pull request may close this issue.

feat: Add alt+click shortcut to copy cell and column headers georgecwan/web-client-ui
3 participants
@dsmmcken @vbabich @georgecwan