Fix code smell: added keyup event for color-picker span (#161)

* fix: added keyup event for color-picker span * fix: translation * fix: use anchor tag as action element instead of span
onecx · Jul 1, 2024 · 74c58b2 · 74c58b2
1 parent 08b3529
commit 74c58b2
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/src/app/theme/theme-designer/theme-designer.component.html b/src/app/theme/theme-designer/theme-designer.component.html
@@ -217,13 +217,15 @@
                     </div>
                   </ng-template>
                 </p-overlayPanel>
-                <span
+                <a
+                  tabindex="0"
                   id="theme_detail_item_color_{{ item }}_toggle"
                   class="p-inputgroup-addon cursor-pointer"
                   (click)="op.toggle($event)"
+                  (keyup)="op.toggle($event)"
                   [ngStyle]="{ 'background-color': group.formGroup.value[item] }"
                 >
-                </span>
+                </a>
               </div>
             </div>
           </div>

diff --git a/src/assets/i18n/en.json b/src/assets/i18n/en.json
@@ -77,7 +77,7 @@
     "MODIFICATION_DATE": "Changed on",
     "MODIFICATION_USER": "Changed by",
     "OPERATOR": "Created by Operator",
-    "OPERATOR_MESSAGE": "This theme was created by the operator. All changes will be lost on the next deployment.",
+    "OPERATOR_MESSAGE": "This theme was created by the operator. All changes will be lost during the next deployment.",
     "TOOLTIPS": {
       "CREATION_DATE": "Timestamp of the creation",
       "CREATION_USER": "Name of the user of the creation",