From 8b65789e462e6fc858ae2394be274f354e6978e9 Mon Sep 17 00:00:00 2001
From: John Ciulla <72523035+LittleJC3@users.noreply.github.com>
Date: Sat, 15 Mar 2025 16:20:24 -0400
Subject: [PATCH] Remove manage questions and make edit/manage more obvious

---
 .../edit-quest-dialog.component.html          |  4 ++--
 .../edit-quest-dialog.component.scss          |  3 ++-
 .../preview-quest.component.html              | 22 ++++---------------
 .../preview-quest/preview-quest.component.ts  |  3 ---
 4 files changed, 8 insertions(+), 24 deletions(-)

diff --git a/src/app/dashboard/edit-quest-dialog/edit-quest-dialog.component.html b/src/app/dashboard/edit-quest-dialog/edit-quest-dialog.component.html
index 3b0bd0f..0ef6b30 100644
--- a/src/app/dashboard/edit-quest-dialog/edit-quest-dialog.component.html
+++ b/src/app/dashboard/edit-quest-dialog/edit-quest-dialog.component.html
@@ -18,9 +18,9 @@ <h2 mat-dialog-title>Edit Quest</h2>
         </mat-form-field>
     </form>
 
-    <div class="add-question">
+    <div class="modify-question">
         <button mat-stroked-button type="button" (click)="onAddQuestion()">
-            <mat-icon fontIcon="add"></mat-icon> Add Questions
+            <mat-icon fontIcon="edit"></mat-icon> Manage Questions
         </button>
     </div>
 
diff --git a/src/app/dashboard/edit-quest-dialog/edit-quest-dialog.component.scss b/src/app/dashboard/edit-quest-dialog/edit-quest-dialog.component.scss
index 6d4a9f8..d99912e 100644
--- a/src/app/dashboard/edit-quest-dialog/edit-quest-dialog.component.scss
+++ b/src/app/dashboard/edit-quest-dialog/edit-quest-dialog.component.scss
@@ -10,6 +10,7 @@
     width: 100%;
 }
 
-.add-question {
+.modify-question {
+    padding-top: 1rem;
     width: 100%;
 }
\ No newline at end of file
diff --git a/src/app/preview-quest/preview-quest.component.html b/src/app/preview-quest/preview-quest.component.html
index 9b70133..78b1ab6 100644
--- a/src/app/preview-quest/preview-quest.component.html
+++ b/src/app/preview-quest/preview-quest.component.html
@@ -4,9 +4,10 @@
       <mat-card-title>{{ quest?.questName }}</mat-card-title>
       <mat-card-subtitle>{{ quest?.questDescription }}</mat-card-subtitle>
       <span class="spacer"></span>
-      <button mat-icon-button (click)="openEditQuestDialog(quest)">
+      <button mat-stroked-button (click)="openEditQuestDialog(quest)">
         <mat-icon>edit</mat-icon>
-      </button>
+        Edit Quest / Questions
+      </button>      
     </mat-card-header>
     <mat-card-content>
       <mat-chip color="primary" selected>{{ quest?.questSubject }}</mat-chip>
@@ -15,7 +16,7 @@
   <button mat-raised-button color="primary" (click)="startQuest()" [disabled]="quest?.questions?.length === 0">
     Start the Quest
   </button>
-  <mat-tab-group (selectedTabChange)="onTabChange($event)">
+  <mat-tab-group>
     <mat-tab label="Preview Questions">
       <ng-container *ngIf="quest; else loading">
         <!-- Check if there are questions associated with the quest -->
@@ -31,21 +32,6 @@
         </ng-template>
       </ng-container>
     </mat-tab>
-    <mat-tab label="Manage Questions">
-      <ng-container *ngIf="quest; else loading">
-        <div *ngIf="quest?.questions?.length === 0; else manageQuestions">
-          <p>No questions have been assigned to this quest.</p>
-        </div>
-
-        <ng-template #manageQuestions>
-          <app-questions 
-            [quest]="quest"    
-            [questions]="quest?.questions || []" 
-            [questionsEditable]="false">
-          </app-questions>
-        </ng-template>
-      </ng-container>
-    </mat-tab>
   </mat-tab-group>
 
   <ng-template #loading>
diff --git a/src/app/preview-quest/preview-quest.component.ts b/src/app/preview-quest/preview-quest.component.ts
index 733ec44..ff8ed62 100644
--- a/src/app/preview-quest/preview-quest.component.ts
+++ b/src/app/preview-quest/preview-quest.component.ts
@@ -70,9 +70,6 @@ export class PreviewQuestComponent implements OnInit {
     });
   }
 
-  onTabChange($event: MatTabChangeEvent) {
-    // Handle tab changes if needed
-  }
 
   startQuest() {
     this.router.navigate(['/quest/start/', this.quest?.id], {