Clarify GBAPI #282

cmu-sei · Feb 14, 2025 · 288be7e · 288be7e
1 parent d125f2a
commit 288be7e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/projects/gameboard-ui/src/app/reports/components/sort-header/sort-header.component.ts b/projects/gameboard-ui/src/app/reports/components/sort-header/sort-header.component.ts
@@ -1,17 +1,17 @@
+import { Component, Input, OnChanges, OnInit, SimpleChanges } from '@angular/core';
 import { ActivatedRoute, Params } from '@angular/router';
 import { SortDirection } from '@/core/models/sort-direction';
 import { fa } from "@/services/font-awesome.service";
 import { RouterService } from '@/services/router.service';
 import { UnsubscriberService } from '@/services/unsubscriber.service';
-import { Component, Input, OnChanges, OnInit, SimpleChanges } from '@angular/core';
 
 @Component({
   selector: 'app-sort-header',
   template: `<div class="component-container cursor-pointer d-flex w-100 justify-content-center align-items-center" [class.active]="isActive" (click)="handleClick()">
     <div class="content-container">
       <ng-content></ng-content>
     </div>
-    <fa-icon *ngIf="isActive" class="d-block ml-2" size="sm" [icon]="currentSortDirection == 'asc' ? fa.caretUp : fa.caretDown"></fa-icon>
+    <fa-icon class="d-block ml-2" [class.text-muted]="!isActive" size="xs" [icon]="!isActive || currentSortDirection == 'asc' ? fa.caretUp : fa.caretDown"></fa-icon>
   </div>`,
   styleUrls: ['./sort-header.component.scss']
 })

diff --git a/projects/gameboard-ui/src/styles.scss b/projects/gameboard-ui/src/styles.scss
@@ -107,7 +107,6 @@ app-root {
 }
 
 /* APP UTILITY CLASSES */
-
 .font-fixed-width {
   font-family: monospace;
 }
@@ -318,6 +317,11 @@ table.gameboard-table {
   background-color: theme-color("success");
 }
 
+.btn.focus,
+.btn:focus {
+  box-shadow: none !important;
+}
+
 .form-group {
   padding: 1.5rem;
   margin-bottom: 0.5rem;