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

Feature/#704 add copy to clipboard button in application view #708

Conversation

tomlovesgithub
Copy link
Contributor

  • Add function for copying panel-pack content to clipboard
  • Add button, with text "Copy to clipboard" which triggers function onClick
  • Button text changes to "Copied" for 3 seconds after click, then returns to "Copy to clipboard"

-Did play around with positioning of button, needs either to be outside of tabs area (like pdf and doc buttons), so that the button remains visible when tabs are clicked or inside each tab.

Copy link
Member

@warrensearle warrensearle left a comment

Choose a reason for hiding this comment

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

This works in that it extracts the text and copies to clipboard.
Be great to see if the team find this useful.

Copy link
Contributor

@YaaL YaaL left a comment

Choose a reason for hiding this comment

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

See comments

src/views/Exercises/Applications/Application.vue Outdated Show resolved Hide resolved
@tomlovesgithub
Copy link
Contributor Author

Please excuse the atrocity above, i tried to squash a commit with rebase -i ...wont try that again

@tomlovesgithub tomlovesgithub requested a review from YaaL July 29, 2020 16:14
Copy link
Contributor

@YaaL YaaL left a comment

Choose a reason for hiding this comment

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

See comments

src/views/Exercises/Applications/Application.vue Outdated Show resolved Hide resolved
src/views/Exercises/Applications/Application.vue Outdated Show resolved Hide resolved
@tomlovesgithub tomlovesgithub force-pushed the feature/#704-Add-Copy-to-clipboard-button-in-application-view branch from 0cb4a56 to 6e5b017 Compare July 30, 2020 17:05
Copy link
Member

@warrensearle warrensearle left a comment

Choose a reason for hiding this comment

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

This works really well 👍
(see comment for small tweak)

src/views/Exercises/Applications/Application.vue Outdated Show resolved Hide resolved
Copy link
Member

@warrensearle warrensearle left a comment

Choose a reason for hiding this comment

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

Awesome 👍
Let's see how useful this is. @tomlovesgithub can you get this properly user tested by the ops team?

Copy link
Contributor

@YaaL YaaL left a comment

Choose a reason for hiding this comment

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

Minor fix - see comments

src/views/Exercises/Applications/Application.vue Outdated Show resolved Hide resolved
src/views/Exercises/Applications/Application.vue Outdated Show resolved Hide resolved
@tomlovesgithub tomlovesgithub requested a review from YaaL July 31, 2020 11:33
Copy link
Contributor

@YaaL YaaL left a comment

Choose a reason for hiding this comment

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

One tiny last change and it's good to go

src/views/Exercises/Applications/Application.vue Outdated Show resolved Hide resolved
@tomlovesgithub tomlovesgithub requested a review from YaaL July 31, 2020 12:03
Copy link
Contributor

@YaaL YaaL left a comment

Choose a reason for hiding this comment

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

Looks good now!

@tomlovesgithub tomlovesgithub merged commit 8e3f1a1 into develop Jul 31, 2020
@tomlovesgithub tomlovesgithub deleted the feature/#704-Add-Copy-to-clipboard-button-in-application-view branch July 31, 2020 12:07
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.

3 participants