UI bug fixes

ui/winstudentgoaltracker/src/app/desktop/components/benchmark-card-full/benchmark-card-full.ts

@@ -82,6 +82,7 @@ export class BenchmarkCardFull implements OnDestroy {
       if (result.success) {
         this.successMessage.set('Benchmark created.');
         this.savedBenchmarkText = this.benchmarkText;
+        this.studentService.notifyDataChanged();
         if (result.payload?.benchmarkId) {
           this.router.navigate(['/students', this.studentId, 'goals', this.goalId, 'benchmarks', result.payload.benchmarkId]);
         }

ui/winstudentgoaltracker/src/app/desktop/components/goal-card-full/goal-card-full.html

@@ -15,8 +15,8 @@
 @if (loaded()) {
 <div class="detail-card">
     <div class="field">
-        <label class="field-label" for="baseline">Baseline</label>
+        <label class="field-label" for="category">Category</label>
-        <input id="baseline" class="field-input" type="text" [(ngModel)]="baseline" />
+        <input id="category" class="field-input" type="text" [(ngModel)]="category" />
     </div>
     <div class="field">
         <label class="field-label" for="description">Description</label>
@@ -24,8 +24,9 @@
             placeholder="Enter description..."></textarea>
     </div>
     <div class="field">
-        <label class="field-label" for="category">Category</label>
+        <label class="field-label" for="baseline">Baseline</label>
-        <input id="category" class="field-input" type="text" [(ngModel)]="category" />
+        <textarea id="baseline" class="field-input field-textarea" [(ngModel)]="baseline" rows="3"
+            placeholder="Enter baseline..."></textarea>
     </div>
     <div class="actions">
         <button class="toolbar-btn" (click)="onCancel()" [disabled]="!hasChanges()">Cancel</button>

ui/winstudentgoaltracker/src/app/desktop/components/goal-card-full/goal-card-full.ts

@@ -113,7 +113,7 @@ export class GoalCardFull implements OnDestroy {
   }
 
   onBenchmarks() {
-    this.router.navigate(['/students', this.studentId, 'benchmarks']);
+    this.router.navigate(['/students', this.studentId, 'goals', this.goalId, 'benchmarks']);
   }
 
   ngOnDestroy() {

ui/winstudentgoaltracker/src/app/desktop/components/goal-list/goal-list.ts

@@ -50,6 +50,7 @@ export class GoalList implements OnDestroy {
         this.goals.update(list => [...list, goal]);
         this.showAddModal.set(false);
         this.studentService.notifyDataChanged();
+        this.router.navigate(['/students', this.studentId, 'goals', goal.goalId]);
     }
 
     onModalCancelled() {

ui/winstudentgoaltracker/src/app/desktop/components/student-card-list/student-card-list.ts

@@ -1,4 +1,5 @@
 import { Component, inject, signal } from '@angular/core';
+import { Router } from '@angular/router';
 import { StudentCard } from '../student-card/student-card';
 import { AddStudentModal } from '../add-student-modal/add-student-modal';
 import { DummyStudentService } from '../../../shared/services/dummy-student.service';
@@ -24,6 +25,7 @@ export class StudentCardList {
   // ************************** Declarations *************************
 
   private readonly studentService = inject(StudentService);
+  private readonly router = inject(Router);
   protected readonly students = signal<StudentCardDto[]>([]);
   protected readonly displayMode = signal<DisplayMode>('card');
   protected readonly showAddModal = signal(false);
@@ -45,6 +47,7 @@ export class StudentCardList {
     this.students.update(list => this.sortByIdentifier([...list, student]));
     this.showAddModal.set(false);
     this.studentService.notifyDataChanged();
+    this.router.navigate(['/students', student.studentId]);
   }
 
   onModalCancelled() {

ui/winstudentgoaltracker/src/app/desktop/pages/home/home.ts

@@ -153,6 +153,7 @@ export class Home implements OnDestroy {
       .filter(b => b.goalId === goalId)
       .map(b => ({
         label: b.benchmark,
+        routerLink: ['/students', studentId, 'goals', goalId, 'benchmarks', b.benchmarkId],
       }));
   }
 

ui/winstudentgoaltracker/src/app/mobile/pages/students/students.html

@@ -1,5 +1,12 @@
+@if (loaded() && students().length === 0) {
+<div class="empty-state">
+    <p>You don't currently have any students.</p>
+    <p>Please log into the <strong>desktop app</strong> to add your students.</p>
+</div>
+} @else {
 <div class="card-grid">
     @for (student of students(); track student.studentId) {
     <app-student-card [student]="student" />
     }
-</div>
+</div>
+}

ui/winstudentgoaltracker/src/app/mobile/pages/students/students.scss

@@ -6,4 +6,14 @@
     display: grid;
     grid-template-columns: repeat(2, 1fr);
     gap: 0.75rem;
+}
+
+.empty-state {
+    padding: 1.5rem 1rem;
+    text-align: center;
+    color: #888;
+    font-size: 0.875rem;
+    background: #fff;
+    border-radius: 10px;
+    border: 1px solid #e5e5e5;
 }

ui/winstudentgoaltracker/src/app/mobile/pages/students/students.ts

@@ -22,6 +22,7 @@ export class Students {
 
   private readonly studentService = inject(StudentService);
   protected readonly students = signal<StudentCardDto[]>([]);
+  protected readonly loaded = signal(false);
 
   public errorMessage = signal<String | null>(null);
 
@@ -46,6 +47,7 @@ export class Students {
       else
       {
         this.students.set(data.payload || []);
+        this.loaded.set(true);
       }
     });
   }