fix more menu and add comment

ls1intum · Nov 4, 2024 · 747478d · 747478d
1 parent d6fd7c7
commit 747478d
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/main/webapp/app/overview/course-overview.component.html b/src/main/webapp/app/overview/course-overview.component.html
@@ -1,3 +1,4 @@
+<!-- @if can't be used for isShownViaLti as it removes the element from the DOM, disrupting dropdown stability for the More-Menu. -->
 <div class="sidebar-wrapper" [ngClass]="{ 'd-none': isShownViaLti }">
     <div class="vw-100 bg-body">
         <mat-sidenav-container
@@ -165,7 +166,7 @@
                                 <div id="manage-button">
                                     <a
                                         class="btn btn-sm btn-outline-secondary"
-                                        [ngClass]="{ 'd-none d-sm-inline': pageTitle === 'communication' }"
+                                        [ngClass]="{ 'd-none d-sm-block': pageTitle === 'communication' }"
                                         [routerLink]="['/course-management', course.id]"
                                         jhiTranslate="artemisApp.courseOverview.manage"
                                     ></a>
@@ -191,7 +192,7 @@
                             <ng-container class="d-flex ms-auto" #controlsViewContainer />
                         }
                         <!-- Check for !isLTI to render the router outlet only once and avoid side effects on route updates. -->
-                        @if (!isLti) {
+                        @if (!isShownViaLti) {
                             <ng-template *ngTemplateOutlet="routerOutlet" />
                         }
                     </div>
@@ -205,7 +206,7 @@
         <ng-container class="d-flex ms-auto" #controlsViewContainer />
     }
     <!-- Check isLTI to render the router outlet only once and avoid side effects on route updates. -->
-    @if (isLti) {
+    @if (isShownViaLti) {
         <ng-template *ngTemplateOutlet="routerOutlet" />
     }
 </div>