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.

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

feat(Database Browser): Copy cell value using CTRL+C #1272

Merged

Conversation

douglasmuraoka
Copy link
Contributor

Currently, in order to copy cell content, users need to edit the cell, select all content manually and copy. This PR allows users to copy the entire cell content just by selecting the cell and pressing CTRL+C (or CMD+C).
When a value is copied to the clipboard, a message will appear on the bottom right of the screen.
Also, it is not possible to copy sensitive data such as _User password and sessionToken.

Copy link
Member

@davimacedo davimacedo left a comment

Choose a reason for hiding this comment

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

It looks good to me! LGTM!

@davimacedo davimacedo merged commit 87dd271 into parse-community:master Sep 9, 2019
beiguancyc pushed a commit to beiguancyc/parse-dashboard that referenced this pull request Sep 12, 2019
* commit '9f2d7e07fdb155b3c56d718976ba2077cc4e96a0':
  2.0.2 (parse-community#1278)
  Fix: Filter tab not working for classes starting with _ (parse-community#1275)
  Fix: DataBrowser is not updating accordingly (parse-community#1276)
  feat(Database Browser): Copy cell value using CTRL+C (parse-community#1272)
  Update parse to the latest version 🚀 (parse-community#1274)
  Update babel7 to the latest version 🚀 (parse-community#1273)
  Update webpack-cli to the latest version 🚀 (parse-community#1268)
  Update eslint-plugin-jest to the latest version 🚀 (parse-community#1267)
  Update NodeJS required version (parse-community#1265)
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.

2 participants