Merge pull request #338 from citation-file-format/319-allow-thing-under-edit-to-be-moved

Author: abelsiqueira

src/components/AuthorCardEditing.vue

@@ -98,6 +98,20 @@
             />
         </div>
         <q-card-actions align="right">
+            <q-btn
+                color="blue"
+                dense
+                icon="ion-arrow-up"
+                tabindex="-1"
+                v-on:click="$emit('moveUp')"
+            />
+            <q-btn
+                color="blue"
+                dense
+                icon="ion-arrow-down"
+                tabindex="-1"
+                v-on:click="$emit('moveDown')"
+            />
             <q-btn
                 color="negative"
                 dense
@@ -166,7 +180,7 @@ export default defineComponent({
             validateOrcid: makeOptionalFieldValidator('/definitions/person/properties/orcid') // or /definitions/orcid ?
         }
     },
-    emits: ['closePressed', 'removePressed', 'update']
+    emits: ['closePressed', 'removePressed', 'update', 'moveUp', 'moveDown']
 })
 </script>
 <style scoped>

src/components/IdentifierCardEditing.vue

@@ -46,6 +46,20 @@
             </div>
         </q-card-section>
         <q-card-actions align="right">
+            <q-btn
+                dense
+                color="blue"
+                icon="ion-arrow-up"
+                tabindex="-1"
+                v-on:click="$emit('moveUp')"
+            />
+            <q-btn
+                dense
+                color="blue"
+                icon="ion-arrow-down"
+                tabindex="-1"
+                v-on:click="$emit('moveDown')"
+            />
             <q-btn
                 color="negative"
                 dense
@@ -107,6 +121,6 @@ export default defineComponent({
             ]
         }
     },
-    emits: ['closePressed', 'removePressed', 'updateType', 'updateValue', 'updateDescription']
+    emits: ['closePressed', 'removePressed', 'updateType', 'updateValue', 'updateDescription', 'moveUp', 'moveDown']
 })
 </script>

src/components/ScreenAuthors.vue

@@ -33,6 +33,8 @@
                             v-on:update="setAuthorField"
                             v-on:closePressed="() => (editingId = -1)"
                             v-on:removePressed="removeAuthor"
+                            v-on:moveDown="moveAuthorDown(index)"
+                            v-on:moveUp="moveAuthorUp(index)"
                         />
                     </div>
                 </div>
@@ -98,13 +100,17 @@ export default defineComponent({
         }
         const moveAuthorUp = (index: number) => {
             moveUp(index, authors.value, setAuthors)
-            if (editingId.value === index - 1) {
+            if (editingId.value === index && index > 0) {
+                editingId.value = editingId.value - 1
+            } else if (editingId.value === index - 1) {
                 editingId.value = editingId.value + 1
             }
         }
         const moveAuthorDown = (index: number) => {
             moveDown(index, authors.value, setAuthors)
-            if (editingId.value === index + 1) {
+            if (editingId.value === index && index < authors.value.length - 1) {
+                editingId.value = editingId.value + 1
+            } else if (editingId.value === index + 1) {
                 editingId.value = editingId.value - 1
             }
         }

src/components/ScreenIdentifiers.vue

@@ -45,6 +45,8 @@
                             v-on:updateDescription="setIdentifierDescriptionField"
                             v-on:closePressed="() => (editingId = -1)"
                             v-on:removePressed="removeIdentifier"
+                            v-on:moveDown="moveIdentifierDown(index)"
+                            v-on:moveUp="moveIdentifierUp(index)"
                         />
                     </div>
                 </div>
@@ -126,13 +128,17 @@ export default defineComponent({
         }
         const moveIdentifierUp = (index: number) => {
             moveUp(index, identifiers.value, setIdentifiers)
-            if (editingId.value === index - 1) {
+            if (editingId.value === index && index > 0) {
+                editingId.value = editingId.value - 1
+            } else if (editingId.value === index - 1) {
                 editingId.value = editingId.value + 1
             }
         }
         const moveIdentifierDown = (index: number) => {
             moveDown(index, identifiers.value, setIdentifiers)
-            if (editingId.value === index + 1) {
+            if (editingId.value === index && index < identifiers.value.length - 1) {
+                editingId.value = editingId.value + 1
+            } else if (editingId.value === index + 1) {
                 editingId.value = editingId.value - 1
             }
         }