From 0ce17dd2dfc9ea48cb8b5cd7fa4cc1738cf75986 Mon Sep 17 00:00:00 2001
From: Olga Bulat <obulat@gmail.com>
Date: Fri, 30 Oct 2020 23:32:56 +0300
Subject: [PATCH] Update AttributionDetails step for clarity

---
 src/components/AttributionDetailsStep.vue            | 12 ++++++------
 src/locales/en.json                                  | 10 +++++-----
 .../AttributionDetailsStep.spec.js.snap              | 12 ++++++------
 .../components/__snapshots__/Stepper.spec.js.snap    |  3 ++-
 4 files changed, 19 insertions(+), 18 deletions(-)

diff --git a/src/components/AttributionDetailsStep.vue b/src/components/AttributionDetailsStep.vue
index 6868ca9e9..1e0e9f4ff 100644
--- a/src/components/AttributionDetailsStep.vue
+++ b/src/components/AttributionDetailsStep.vue
@@ -14,12 +14,6 @@
                         :placeholder="this.$t('stepper.AD.form.creator-name.placeholder')"
                     />
                 </b-field>
-                <b-field :label="this.$t('stepper.AD.form.creator-profile.label')">
-                    <b-input
-                        v-model="creatorProfileUrl"
-                        :placeholder="this.$t('stepper.AD.form.creator-profile.placeholder')"
-                    />
-                </b-field>
                 <b-field :label="this.$t('stepper.AD.form.work-title.label')">
                     <b-input
                         v-model="workTitle"
@@ -32,6 +26,12 @@
                         :placeholder="this.$t('stepper.AD.form.work-url.placeholder')"
                     />
                 </b-field>
+                <b-field :label="this.$t('stepper.AD.form.creator-profile.label')">
+                    <b-input
+                        v-model="creatorProfileUrl"
+                        :placeholder="this.$t('stepper.AD.form.creator-profile.placeholder')"
+                    />
+                </b-field>
             </form>
         </div>
     </div>
diff --git a/src/locales/en.json b/src/locales/en.json
index be482b795..ccdf87d33 100644
--- a/src/locales/en.json
+++ b/src/locales/en.json
@@ -68,20 +68,20 @@
             "instructions": "Filling out this form is optional, but helps others attribute your work to you, and fills in machine-readable code.",
             "form": {
                 "creator-name": {
-                    "label": "Work Author",
+                    "label": "Creator of Work",
                     "placeholder": "Jane Doe"
                 },
                 "creator-profile": {
-                    "label": "URL of creator profile",
-                    "placeholder": "www.author.com"
+                    "label": "Link to Creator Profile",
+                    "placeholder": "https://janedoe.com"
                 },
                 "work-title": {
                     "label": "Title of Work",
                     "placeholder": "This work"
                 },
                 "work-url": {
-                    "label": "Work URL",
-                    "placeholder": "www.author.com/work.jpg"
+                    "label": "Link to Work",
+                    "placeholder": "https://janedoe.com/best-photo-ever.jpg"
                 }
             }
         }
diff --git a/tests/unit/specs/components/__snapshots__/AttributionDetailsStep.spec.js.snap b/tests/unit/specs/components/__snapshots__/AttributionDetailsStep.spec.js.snap
index 93a63c717..13a6d3fa6 100644
--- a/tests/unit/specs/components/__snapshots__/AttributionDetailsStep.spec.js.snap
+++ b/tests/unit/specs/components/__snapshots__/AttributionDetailsStep.spec.js.snap
@@ -53,7 +53,7 @@ exports[`AttributionDetailsStep Component Rendering Component mounted if status
         <label
           class="label"
         >
-          stepper.AD.form.creator-profile.label
+          stepper.AD.form.work-title.label
         </label>
          
         <div
@@ -62,7 +62,7 @@ exports[`AttributionDetailsStep Component Rendering Component mounted if status
           <input
             autocomplete="on"
             class="input"
-            placeholder="stepper.AD.form.creator-profile.placeholder"
+            placeholder="stepper.AD.form.work-title.placeholder"
             type="text"
           />
            
@@ -82,7 +82,7 @@ exports[`AttributionDetailsStep Component Rendering Component mounted if status
         <label
           class="label"
         >
-          stepper.AD.form.work-title.label
+          stepper.AD.form.work-url.label
         </label>
          
         <div
@@ -91,7 +91,7 @@ exports[`AttributionDetailsStep Component Rendering Component mounted if status
           <input
             autocomplete="on"
             class="input"
-            placeholder="stepper.AD.form.work-title.placeholder"
+            placeholder="stepper.AD.form.work-url.placeholder"
             type="text"
           />
            
@@ -111,7 +111,7 @@ exports[`AttributionDetailsStep Component Rendering Component mounted if status
         <label
           class="label"
         >
-          stepper.AD.form.work-url.label
+          stepper.AD.form.creator-profile.label
         </label>
          
         <div
@@ -120,7 +120,7 @@ exports[`AttributionDetailsStep Component Rendering Component mounted if status
           <input
             autocomplete="on"
             class="input"
-            placeholder="stepper.AD.form.work-url.placeholder"
+            placeholder="stepper.AD.form.creator-profile.placeholder"
             type="text"
           />
            
diff --git a/tests/unit/specs/components/__snapshots__/Stepper.spec.js.snap b/tests/unit/specs/components/__snapshots__/Stepper.spec.js.snap
index fd96ad0a5..50422af2d 100644
--- a/tests/unit/specs/components/__snapshots__/Stepper.spec.js.snap
+++ b/tests/unit/specs/components/__snapshots__/Stepper.spec.js.snap
@@ -10,7 +10,8 @@ exports[`Stepper.vue renders correctly has expected UI initially 1`] = `
     </div>
     <firststep-stub stepid="0" status="current"></firststep-stub>
     <nav class="step-navigation">
-      <!----> <a role="button" class="pagination-next disabled">NEXT STEP</a></nav>
+      <!----> <a role="button" class="pagination-next disabled">NEXT STEP</a>
+    </nav>
   </div>
   <div class="step-container BY inactive enabled">
     <div class="step-header">