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

Implements form field copy button #26

Merged
merged 2 commits into from
Mar 3, 2019
Merged

Implements form field copy button #26

merged 2 commits into from
Mar 3, 2019

Conversation

sixlive
Copy link
Owner

@sixlive sixlive commented Jan 27, 2019

Description

Adds the copy button to the form view.

kapture

Tasks

  • Tests
  • Documentation
  • Changlog Entry

Related Issues/PRs

Steps to Verify Functionality

n/a

@@ -1,5 +1,5 @@
<template>
<div>
<div class="flex content-center">
<div v-show="isNotDirty">

Choose a reason for hiding this comment

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

This is great feature! Tested locally and it works great. Thank you.

For some reason on my screen clipboard is all the way at the top of the line.

image

Adding class="pt-2" to <div v-show="isNotDirty"> fixes that

image

Copy link
Owner Author

Choose a reason for hiding this comment

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

What browser are you using?

Copy link

Choose a reason for hiding this comment

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

Chrome Version 72.0.3626.119 (Official Build) (64-bit)

Also tested on Safari Version 12.0.3 (14606.4.5)

Copy link
Owner Author

Choose a reason for hiding this comment

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

I just pushed a fix to this branch, can you let me know if it fixed it for you? I'll merge after that.

@sixlive sixlive merged commit 6e69045 into master Mar 3, 2019
@sixlive sixlive deleted the implement-form-field branch March 3, 2019 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants