Fixed issue: Unknown shape type found (cvat-ai#998)

cleemesser · Mar 5, 2020 · 0e2b262 · 0e2b262
1 parent 928b729
commit 0e2b262
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/cvat/apps/engine/static/engine/js/shapeCreator.js b/cvat/apps/engine/static/engine/js/shapeCreator.js
@@ -211,12 +211,17 @@ class ShapeCreatorView {
         this._labelSelector.attr('title', `
             ${shortkeys['change_default_label'].view_value} - ${shortkeys['change_default_label'].description}`);
 
-        let labels = window.cvat.labelsInfo.labels();
-        for (let labelId in labels) {
-            let option = $(`<option value=${labelId}> ${labels[labelId].normalize()} </option>`);
-            option.appendTo(this._labelSelector);
+        const labels = window.cvat.labelsInfo.labels();
+        const labelsKeys = Object.keys(labels);
+        for (let i = 0; i < labelsKeys.length; i += 1) {
+            this._labelSelector.append(
+                // eslint-disable-next-line
+                $(`<option value=${labelsKeys[i]}> ${labels[labelsKeys[i]].normalize()} </option>`)
+            );
         }
+        this._labelSelector.val(labelsKeys[0]);
 
+        this._typeSelector.val('box');
         this._typeSelector.on('change', (e) => {
             // FIXME: In the future we have to make some generic solution
             const mode = this._modeSelector.prop('value');

diff --git a/cvat/apps/engine/templates/engine/annotation.html b/cvat/apps/engine/templates/engine/annotation.html
@@ -429,7 +429,7 @@
             <option value="interpolation" class="regular"> Interpolation </option>
         </select>
         <select id="shapeTypeSelector" class="regular h2">
-            <option value="box" class="regular"> Box </option>
+            <option value="box" class="regular" selected> Box </option>
             <option value="polygon" class="regular"> Polygon </option>
             <option value="polyline" class="regular"> Polyline </option>
             <option value="points" class="regular"> Points </option>