Merge branch '208-page-refreshes-if-enter-key-is-pressed-inside-text-field-input' into 'develop'

Resolve "Page refreshes if enter key is pressed inside text field input"

Closes #208

See merge request bramw/baserow!130

changelog.md

@@ -1,5 +1,7 @@
 # Changelog
 
+* Fixed bug where the page refreshes if you press enter in an input in the row modal.
+
 ## Released (2020-12-01)
 
 * Added select_for_update where it was still missing.

web-frontend/modules/database/components/row/RowEditModal.vue

@@ -3,32 +3,30 @@
     <h2 v-if="primary !== undefined" class="box__title">
       {{ getHeading(primary, row) }}
     </h2>
-    <form>
+    <RowEditModalField
-      <RowEditModalField
+      v-for="field in getFields(fields, primary)"
-        v-for="field in getFields(fields, primary)"
+      :ref="'field-' + field.id"
-        :ref="'field-' + field.id"
+      :key="'row-edit-field-' + field.id"
-        :key="'row-edit-field-' + field.id"
+      :table="table"
+      :field="field"
+      :row="row"
+      @update="update"
+      @field-updated="$emit('field-updated')"
+      @field-deleted="$emit('field-deleted')"
+    ></RowEditModalField>
+    <div class="actions">
+      <a
+        ref="createFieldContextLink"
+        @click="$refs.createFieldContext.toggle($refs.createFieldContextLink)"
+      >
+        <i class="fas fa-plus"></i>
+        add field
+      </a>
+      <CreateFieldContext
+        ref="createFieldContext"
         :table="table"
-        :field="field"
+      ></CreateFieldContext>
-        :row="row"
+    </div>
-        @update="update"
-        @field-updated="$emit('field-updated')"
-        @field-deleted="$emit('field-deleted')"
-      ></RowEditModalField>
-      <div class="actions">
-        <a
-          ref="createFieldContextLink"
-          @click="$refs.createFieldContext.toggle($refs.createFieldContextLink)"
-        >
-          <i class="fas fa-plus"></i>
-          add field
-        </a>
-        <CreateFieldContext
-          ref="createFieldContext"
-          :table="table"
-        ></CreateFieldContext>
-      </div>
-    </form>
   </Modal>
 </template>