From f976dbb6a78c3dd03a9292276efc3bdc1411e411 Mon Sep 17 00:00:00 2001
From: Charles Hacskaylo <charles.f.hacskaylo@nasa.gov>
Date: Wed, 18 Sep 2024 15:34:51 -0700
Subject: [PATCH] Enhancements for better wrapping of Status Area icons - Add
 code and CSS for toggling single/multiline display of indicators. - Add code
 to detect overflow state of indicators.

---
 src/ui/layout/AppLayout.vue                   | 22 +++++++++----------
 src/ui/layout/layout.scss                     | 18 +++++++++------
 src/ui/layout/status-bar/StatusIndicators.vue | 19 ++++++++++++++--
 3 files changed, 39 insertions(+), 20 deletions(-)

diff --git a/src/ui/layout/AppLayout.vue b/src/ui/layout/AppLayout.vue
index d662f0aed9..608c52d526 100644
--- a/src/ui/layout/AppLayout.vue
+++ b/src/ui/layout/AppLayout.vue
@@ -33,7 +33,7 @@
       :class="{
         'l-shell__head--expanded': headExpanded,
         'l-shell__head--minify-indicators': !headExpanded,
-        'l-shell__head--can-wrap': headCanWrap
+        'l-shell__head--indicators-single-line': !indicatorsMultiline
       }"
     >
       <CreateButton class="l-shell__create-button" />
@@ -41,10 +41,10 @@
       <StatusIndicators />
       <button
         class="l-shell__head__button c-icon-button"
-        :class="headCanWrap ? 'icon-multiline' : 'icon-singleline'"
-        :aria-label="`Display as ${headCanWrap ? 'multiple lines' : 'single line'}`"
-        :title="`Display as ${headCanWrap ? 'multiple lines' : 'single line'}`"
-        @click="toggleHeadWrapping"
+        :class="indicatorsMultiline ? 'icon-singleline' : 'icon-multiline'"
+        :aria-label="`Display as ${indicatorsMultiline ? 'single line' : 'multiple lines'}`"
+        :title="`Display as ${indicatorsMultiline ? 'single line' : 'multiple lines'}`"
+        @click="toggleIndicatorsMultiline"
       ></button>
       <button
         class="l-shell__head__collapse-button c-icon-button"
@@ -205,10 +205,10 @@ export default {
   data: function () {
     let storedHeadProps = window.localStorage.getItem('openmct-shell-head');
     let headExpanded = true;
-    let headCanWrap = false;
+    let indicatorsMultiline = false;
     if (storedHeadProps) {
       headExpanded = JSON.parse(storedHeadProps).expanded;
-      headCanWrap = JSON.parse(storedHeadProps).wrapping;
+      indicatorsMultiline = JSON.parse(storedHeadProps).multiline;
     }
 
     return {
@@ -220,7 +220,7 @@ export default {
       triggerSync: false,
       triggerReset: false,
       headExpanded,
-      headCanWrap,
+      indicatorsMultiline,
       isResizing: false,
       disableClearButton: false
     };
@@ -278,13 +278,13 @@ export default {
         })
       );
     },
-    toggleHeadWrapping() {
-      this.headCanWrap = !this.headCanWrap;
+    toggleIndicatorsMultiline() {
+      this.indicatorsMultiline = !this.indicatorsMultiline;
 
       window.localStorage.setItem(
         'openmct-shell-head',
         JSON.stringify({
-          wrapping: this.headCanWrap
+          multiline: this.indicatorsMultiline
         })
       );
     },
diff --git a/src/ui/layout/layout.scss b/src/ui/layout/layout.scss
index 76e550f71e..2f2c76f276 100644
--- a/src/ui/layout/layout.scss
+++ b/src/ui/layout/layout.scss
@@ -259,18 +259,22 @@
   }
 
   &__indicators {
+    // Style as multiline by default
+    display: flex;
     flex-wrap: wrap;
     font-size: 11px;
+    min-height: 24px;
     justify-content: flex-end;
 
-    .l-shell__head--expanded & {
-    }
-
-    .l-shell__head--can-wrap & {
-      // Force elements to wrap down when width constrained
-      background: rgba(deeppink, 0.4);
-      height: 24px;
+    .l-shell__head--indicators-single-line & {
+      flex-wrap: nowrap;
+      justify-content: flex-start; // Overflow detection doesn't work with flex-end.
+      max-height: 24px;
       overflow: hidden;
+
+      > *:first-child {
+        margin-left: auto; // Mimics justify-content: flex-end when in single line mode.
+      }
     }
   }
 
diff --git a/src/ui/layout/status-bar/StatusIndicators.vue b/src/ui/layout/status-bar/StatusIndicators.vue
index 55f86dcbf6..3a91621197 100644
--- a/src/ui/layout/status-bar/StatusIndicators.vue
+++ b/src/ui/layout/status-bar/StatusIndicators.vue
@@ -17,7 +17,7 @@
  at runtime from the About dialog for additional information.
 -->
 <template>
-  <div class="l-shell__head-section l-shell__indicators">
+  <div ref="indicators" class="l-shell__head-section l-shell__indicators">
     <component
       :is="indicator.value.vueComponent"
       v-for="indicator in sortedIndicators"
@@ -33,7 +33,8 @@ export default {
   inject: ['openmct'],
   data() {
     return {
-      indicators: this.openmct.indicators.getIndicatorObjectsByPriority().map(shallowRef)
+      indicators: this.openmct.indicators.getIndicatorObjectsByPriority().map(shallowRef),
+      indicatorsOverflowing: false
     };
   },
   computed: {
@@ -45,8 +46,13 @@ export default {
       return [...this.indicators].sort((a, b) => b.value.priority - a.value.priority);
     }
   },
+  mounted() {
+    this.checkOverflow();
+    window.addEventListener('resize', this.checkOverflow);
+  },
   beforeUnmount() {
     this.openmct.indicators.off('addIndicator', this.addIndicator);
+    window.removeEventListener('resize', this.checkOverflow);
   },
   created() {
     this.openmct.indicators.on('addIndicator', this.addIndicator);
@@ -54,6 +60,15 @@ export default {
   methods: {
     addIndicator(indicator) {
       this.indicators.push(shallowRef(indicator));
+    },
+    checkOverflow() {
+      const element = this.$refs.indicators;
+      const sizes = {
+        spaceNeeded: element.scrollWidth,
+        spaceAvail: element.clientWidth
+      };
+      this.indicatorsOverflowing = sizes.spaceNeeded > sizes.spaceAvail;
+      console.log('checkOverflow', this.indicatorsOverflowing, sizes.spaceNeeded, sizes.spaceAvail);
     }
   }
 };