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

Add copy button to field slugs to make it easier to copy/paste #7291

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

heybran
Copy link
Contributor

@heybran heybran commented Apr 13, 2024

Description

Add a new CopyButton react component to be used along with field name. It allows user to copy the field name, and keyboard support styles are also considered.

Related GitHub issue(s)

#7237

Testing instructions

  1. Go to Edit Pods screen
  2. Hover over field name or using keyboard to navigate to field wrapper, Copy Button (currently displayed as copy icon) should be visible and clickable.
  3. Click copy button and the field name should be copied to your clipboard.

Screenshots / screencast

Screen.Recording.2024-04-13.at.21.03.34.mov

Using keyboard to copy field name.

Screen.Recording.2024-04-13.at.22.34.02.mov

Changelog text for these changes

Enhancement: Pods field name will now have a copy button near it to copy field name(slug). #7237 (@heybran)

PR checklist

@heybran
Copy link
Contributor Author

heybran commented Apr 13, 2024

@sc0ttkclark

You'll see lots of changes on this file: https://github.com/pods-framework/pods/pull/7291/files#diff-95ad6580fe94f4558f5fbc01ba5a4cf11a162c41f486d86cb203f0d024d11e5e

That is due to I run pnpm run format-js, will that be ok? If not, I can manually change them back to its original format.

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.

None yet

1 participant