Merge pull request #4397 from gemini-hlsw/sc-4316-show-in-explore-ste…

…ps-with-a-file-as-grayed Style rows for completed steps as greyed out
gemini-hlsw · Dec 18, 2024 · 32f837d · 32f837d
2 parents 5abb1b1 + 95f29ad
commit 32f837d
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/common/src/main/scala/explore/components/ui/ExploreStyles.scala b/common/src/main/scala/explore/components/ui/ExploreStyles.scala
@@ -374,10 +374,10 @@ object ExploreStyles:
   val SequencesPanel        = Css("explore-sequences-panel")
   val SequencesPanelError   = Css("explore-sequences-panel-error")
   val CellHideBorder        = Css("explore-table-cell-hide-border")
+  val SequenceRowDone       = Css("explore-sequence-row-done")
   val SequenceBracketCell   = Css("explore-sequence-bracket-cell")
   val SequenceTileTitle     = Css("explore-sequence-tile-title")
   val SequenceTileTitleItem = Css("explore-sequence-tile-title-item")
-  val VisitSection          = Css("explore-visit-section")
 
   val ButtonCopy: Css            = Css("explore-button-copy")
   val ButtonsUndo: Css           = Css("explore-buttons-undo")

diff --git a/common/src/main/webapp/sass/explore.scss b/common/src/main/webapp/sass/explore.scss
@@ -1337,6 +1337,11 @@ $search-preview-margin: 5px;
       }
     }
   }
+
+  .p-datatable-tbody > tr.explore-sequence-row-done {
+    color: var(--disabled-row-color);
+    background: var(--disabled-row-background);
+  }
 }
 
 // -------------

diff --git a/explore/src/main/scala/explore/config/sequence/GmosSequenceTable.scala b/explore/src/main/scala/explore/config/sequence/GmosSequenceTable.scala
@@ -250,7 +250,8 @@ private sealed trait GmosSequenceTableBuilder[S, D: Eq] extends SequenceRowBuild
             .map: s =>
               TagMod(
                 s match
-                  case SequenceRow.Executed.ExecutedStep(_, _) => SequenceStyles.RowHasExtra
+                  case SequenceRow.Executed.ExecutedStep(_, _) =>
+                    SequenceStyles.RowHasExtra |+| ExploreStyles.SequenceRowDone
                   case _                                       => TagMod.empty,
                 if (LinkingInfo.developmentMode)
                   s.id.toOption.map(^.title := _.toString).whenDefined