Skip to content

feat: pfe-clipboard#1235

Merged
castastrophe merged 116 commits intomasterfrom
US267056-pfe-clipboard
Jan 25, 2021
Merged

feat: pfe-clipboard#1235
castastrophe merged 116 commits intomasterfrom
US267056-pfe-clipboard

Conversation

@heyMP
Copy link
Copy Markdown
Contributor

@heyMP heyMP commented Dec 17, 2020

pfe-clipboard

pfe-clipboard allows developers to include button to "copy url to clipboard" that can be placed web pages and handle the entire process of displaying a button that, when clicked, will copy the url to the users system clipboard.

Related Issues

Preview

Link(s) to demo page(s) where this element can be viewed:

Testing instructions

  1. Open the pfe-clipboard demo page in your browser.
  2. In your url bar, append a unique query parameter?testing123
  3. Click on one of the 'Copy Url' buttons
  4. Open a separate browser tab and paste your clipboard into the url
  5. You should see the url of the original demo page as well as the unique query parameter
  • Example: https://deploy-preview-1235--happy-galileo-ea79c4.netlify.app/elements/pfe-clipboard/demo/?testing123

Browser requirements

Your component should work in all of the following environments:

  • Latest 2 versions of Edge
  • Internet Explorer 11 (should be useable, not pixel perfect)
  • Latest 2 versions of Firefox (one on Mac OS, one of Windows OS)
  • Firefox 68 (or latest version for Red Hat Enterprise Linux distribution)
  • Latest 2 versions of Chrome (one on Mac OS, one of Windows OS)
  • Latest 2 versions of Safari
  • Android mobile device (such as the Galaxy S9)
  • Apple mobile device (such as the iPhone X)
  • Apple tablet device (such as the iPhone Pro)

Ready-for-merge Checklist

Check off items as they are completed. Feel free to delete items if they are not applicable.

  • Expected files: all files in this pull request are related to one request or issue (no stragglers or scope-creep).
  • Tests have been updated to cover these changes.
  • Browser testing passed.
  • Repository compiles and tests pass.
  • Changelog updated (not needed for documentation updates).
  • Documentation (README.md, WHY.md, etc.) updated or added.
  • Link to the demo recording:
  • Approved by designer.

Merging

Please squash when merging and ensure your commit message uses conventional commit formatting.

Be sure to share your updates with the patternfly-elements-contribute@redhat.com mailing list!

Copy link
Copy Markdown
Contributor

@castastrophe castastrophe left a comment

Choose a reason for hiding this comment

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

Lemons Grated Taste Minty?

@castastrophe castastrophe added ready: design review blocked Waiting on some other team or process and removed needs changelog Be sure to update the Changelog before merging. run e2e Trigger automated visual regression tests labels Jan 25, 2021
@heyMP heyMP removed blocked Waiting on some other team or process ready: design review labels Jan 25, 2021
@castastrophe castastrophe merged commit 84365f4 into master Jan 25, 2021
@castastrophe castastrophe deleted the US267056-pfe-clipboard branch January 25, 2021 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

demo Updating demo pages docs Documentation updates feature New feature or request functionality Functionality, typically pertaining to the JavaScript. priority: low Severity level: 3 styles An issue or PR pertaining only to CSS/Sass tests Related to testing tools Development and build tools

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants