Fix new question page keyboard problems

CSCfi · Dec 10, 2024 · 0cf1200 · 0cf1200
1 parent fbb7902
commit 0cf1200
Show file tree

Hide file tree

Showing 5 changed files with 21 additions and 13 deletions.
diff --git a/ui/src/app/question/basequestion/multiple-choice-option.component.ts b/ui/src/app/question/basequestion/multiple-choice-option.component.ts
@@ -45,9 +45,9 @@ import { QuestionService } from 'src/app/question/question.service';
                 </div>
 
                 @if (allowRemoval) {
-                    <div (click)="removeOption()" class="col-md-1 question-option-trash">
+                    <button (click)="removeOption()" class="col-md-1 question-option-trash btn btn-link">
                         <i class="bi-trash" title="{{ 'i18n_remove' | translate }}"></i>
-                    </div>
+                    </button>
                 }
             </div>
         </div>

diff --git a/ui/src/app/question/basequestion/multiple-choice.component.ts b/ui/src/app/question/basequestion/multiple-choice.component.ts
@@ -89,10 +89,10 @@ import { WeightedMultipleChoiceOptionEditorComponent } from './weighted-multiple
         }
         <div class="row mt-3">
             <div class="col-md-12">
-                <a (click)="addNewOption()" class="attachment-link pointer">
+                <button (click)="addNewOption()" class="attachment-link pointer btn btn-sm btn-link">
                     <i class="bi-plus"></i>
                     {{ 'i18n_question_add_new_option' | translate }}
-                </a>
+                </button>
             </div>
         </div>
     `,

diff --git a/ui/src/app/question/basequestion/weighted-multiple-choice-option.component.ts b/ui/src/app/question/basequestion/weighted-multiple-choice-option.component.ts
@@ -46,9 +46,13 @@ import { FixedPrecisionValidatorDirective } from 'src/app/shared/validation/fixe
                         [disabled]="lotteryOn"
                     />
                 </div>
-                <div class="col-md-1 question-option-trash pointer" [hidden]="lotteryOn" (click)="removeOption()">
+                <button
+                    class="col-md-1 question-option-trash pointer btn btn-link "
+                    [hidden]="lotteryOn"
+                    (click)="removeOption()"
+                >
                     <i class="bi-trash" title="{{ 'i18n_remove' | translate }}"></i>
-                </div>
+                </button>
             </div>
         </div>
     `,

diff --git a/ui/src/app/question/examquestion/multichoice.component.ts b/ui/src/app/question/examquestion/multichoice.component.ts
@@ -53,9 +53,13 @@ import { ExamSectionQuestionOption } from 'src/app/question/question.model';
                             />
                         }
                     </div>
-                    <div [hidden]="lotteryOn()" (click)="removeOption(option)" class="col-md-1 question-option-trash">
+                    <button
+                        [hidden]="lotteryOn()"
+                        (click)="removeOption(option)"
+                        class="col-md-1 question-option-trash btn btn-link"
+                    >
                         <i class="bi-trash" title="{{ 'i18n_remove' | translate }}"></i>
-                    </div>
+                    </button>
                 </div>
             }
             <div class="row mt-2">

diff --git a/ui/src/app/question/examquestion/weighted-multichoice.component.ts b/ui/src/app/question/examquestion/weighted-multichoice.component.ts
@@ -77,13 +77,13 @@ import { ExamSectionQuestionOption } from 'src/app/question/question.model';
                                     [disabled]="lotteryOn()"
                                 />
                             </div>
-                            <div
-                                class="col-md-1 question-option-trash pointer"
+                            <button
+                                class="col-md-1 question-option-trash pointer btn btn-link"
                                 [hidden]="lotteryOn()"
                                 (click)="removeOption(option)"
                             >
                                 <i class="bi-trash" title="{{ 'i18n_remove' | translate }}"></i>
-                            </div>
+                            </button>
                         </div>
                     }
                     <div class="row">
@@ -98,10 +98,10 @@ import { ExamSectionQuestionOption } from 'src/app/question/question.model';
         </div>
         <div class="row mt-2">
             <div class="col-md-12">
-                <a (click)="addNewOption()" class="attachment-link pointer">
+                <button (click)="addNewOption()" class="attachment-link pointer btn btn-sm btn-link">
                     <i class="bi-plus"></i>
                     {{ 'i18n_question_add_new_option' | translate }}
-                </a>
+                </button>
             </div>
         </div>
     `,