add copy button to kv secrets masked input (#5627)

ui/app/styles/components/masked-input.scss

@@ -1,9 +1,11 @@
 .masked-input {
-  position: relative;
+  display: flex;
+  align-items: center;
 }
 
-.masked-input .masked-value {
-  padding-left: 2.5rem;
+.masked-input.masked.display-only,
+.masked-input:not(.masked) {
+  align-items: start;
 }
 
 // we want to style the boxes the same everywhere so they
@@ -11,14 +13,17 @@
 .masked-input.masked .masked-value {
   font-size: 9px;
   font-family: $family-primary;
-}
-
-.masked-input.masked .masked-value {
   line-height: 2.5;
 }
+
+.masked-input.display-only .masked-value {
+  order: 1;
+}
+
 // aligns the boxes on the input page
 .masked-input.masked:not(.display-only) .masked-value {
   line-height: 3;
+  border-radius: $radius 0 0 $radius;
 }
 
 //override bulma's pre styling
@@ -27,48 +32,52 @@
   font-size: 1rem;
   padding-top: 0;
   padding-bottom: 0;
+  padding-left: $spacing-s;
 }
 
-.masked-input-toggle {
-  background: transparent;
-  position: absolute;
-  height: auto;
-  top: $size-6/4;
-  bottom: $size-6/4;
-  left: 1px;
-  line-height: 1rem;
-  min-width: 0;
-  max-height: 2rem;
-  padding: 0 $size-8;
-  z-index: 100;
-  border: 0;
-  box-shadow: none;
-  color: $blue;
+.button.masked-input-toggle,
+.button.copy-button {
+  min-width: $spacing-xl;
+  border-left: 0;
+  border-radius: 0 $radius $radius 0;
+  color: $grey;
+}
 
-  &:active,
-  &.is-active,
-  &:focus,
-  &.is-focused,
-  &:hover,
-  &:focus:not(:active) {
-    color: $blue;
+.display-only {
+  .button.masked-input-toggle,
+  .button.copy-button {
+    background: transparent;
+    height: auto;
+    line-height: 1rem;
+    min-width: $spacing-l;
+    z-index: 100;
     border: 0;
     box-shadow: none;
+    color: $grey-light;
+    padding-left: 0;
+    padding-right: 0;
+
+    &:active,
+    &.is-active,
+    &:focus,
+    &.is-focused,
+    &:hover,
+    &:focus:not(:active) {
+      color: $blue;
+      border: 0;
+      box-shadow: none;
+    }
   }
 }
 
-.masked-input.display-only .masked-input-toggle {
-  top: 0;
-  font-size: 0.5rem;
-  height: 1rem;
-  padding-left: 0;
+.masked-input.masked .masked-value {
+  color: $grey-light;
+}
+
+.masked-input:not(.masked) .masked-input-toggle {
+  color: $blue;
 }
 
 .masked-input .input:focus + .masked-input-toggle {
   background: rgba($white, 0.95);
 }
-
-.masked-input.masked .masked-value,
-.masked-input.masked .masked-input-toggle {
-  color: $grey-light;
-}

ui/app/templates/components/masked-input.hbs

@@ -1,4 +1,4 @@
-<div class="masked-input {{if shouldObscure "masked"}} {{if displayOnly "display-only"}}" data-test-masked-input>
+<div class="masked-input {{if shouldObscure "masked"}} {{if displayOnly "display-only"}} {{if allowCopy "allow-copy"}}" data-test-masked-input>
 	{{#if displayOnly}}
 		<pre class="masked-value display-only is-word-break">{{displayValue}}</pre>
 	{{else}}
@@ -15,7 +15,16 @@
     data-test-textarea
     />
 	{{/if}}
-  <button {{action "toggleMask"}}  class="{{if (eq value "") "has-text-grey"}} masked-input-toggle button is-compact" data-test-button>
+  {{#if allowCopy}}
+    <CopyButton
+      @clipboardText={{value}}
+      @class="copy-button button is-compact"
+      data-test-copy-button
+    >
+      <ICon @glyph="copy" aria-hidden="true" @size=16 />
+    </CopyButton>
+  {{/if}}
+  <button {{action "toggleMask"}}  class="{{if (eq value "") "has-text-grey"}} masked-input-toggle button {{if displayOnly "is-compact"}}" data-test-button>
     {{i-con glyph=(if shouldObscure "hidden" "visible") aria-hidden="true" size=16}}
 	</button>
 </div>

ui/app/templates/components/secret-edit-display.hbs

@@ -52,7 +52,7 @@
             @onChange={{@editActions.handleChange}}
             @value={{secret.value}}
             data-test-secret-value="true"
-         /> 
+         />
         </div>
         <div class="column is-narrow info-table-row-edit">
           {{#if (eq @secretData.length (inc index))}}
@@ -66,7 +66,7 @@
               {{action @editActions.deleteRow secret.name}}
               aria-label="Delete row"
             >
-              <ICon 
+              <ICon
                 @size="22"
                 @glyph="trash-a"
                 @excludeIconClass={{true}}
@@ -78,4 +78,4 @@
       </div>
     {{/each}}
     </div>
-  {{/if}}
+  {{/if}}

ui/app/templates/partials/secret-form-show.hbs

@@ -27,7 +27,7 @@
     </div>
     {{#each-in modelForData.secretData as |key value|}}
       {{#info-table-row label=key value=value alwaysRender=true}}
-        {{masked-input value=value displayOnly=true}}
+        {{masked-input value=value displayOnly=true allowCopy=true}}
       {{/info-table-row}}
     {{/each-in}}
   {{/if}}

ui/tests/integration/components/masked-input-test.js

@@ -40,6 +40,18 @@ module('Integration | Component | masked input', function(hooks) {
     assert.notOk(component.textareaIsPresent);
   });
 
+  test('it renders a copy button when allowCopy is true', async function(assert) {
+    await render(hbs`{{masked-input allowCopy=true}}`);
+
+    assert.ok(component.copyButtonIsPresent);
+  });
+
+  test('it does not render a copy button when allowCopy is false', async function(assert) {
+    await render(hbs`{{masked-input allowCopy=false}}`);
+
+    assert.notOk(component.copyButtonIsPresent);
+  });
+
   test('it unmasks text on focus', async function(assert) {
     this.set('value', 'value');
     await render(hbs`{{masked-input value=value}}`);

ui/tests/pages/components/masked-input.js

@@ -5,6 +5,7 @@ export default {
   wrapperClass: attribute('class', '[data-test-masked-input]'),
   enterText: fillable('[data-test-textarea]'),
   textareaIsPresent: isPresent('[data-test-textarea]'),
+  copyButtonIsPresent: isPresent('[data-test-copy-button]'),
   toggleMasked: clickable('[data-test-button]'),
   async focusField() {
     return focus('[data-test-textarea]');