From edc7db0acd4f9b289dc174c2c70565e1bb7fdc07 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=98rjan=20Aare=20J=C3=B8rgensen?= <oajo@equinor.com>
Date: Thu, 19 Sep 2024 15:44:17 +0200
Subject: [PATCH] Fix heading levels and nesting

---
 .../GrossDepositionEnviromentGroup.tsx                    | 2 +-
 .../OutcropAnalogueGroup/OutcropAnalogueGroup.tsx         | 4 +++-
 .../StratigrapicGroups/StratigrapicGroups.tsx             | 4 +++-
 src/features/ModelView/DeleteButton/DeleteModel.tsx       | 4 +++-
 .../ModelAreaCoordinates/ModelAreaCoordinates.tsx         | 4 +++-
 src/features/ModelView/ModelFilesView/ModelFilesView.tsx  | 4 +++-
 .../ModelView/ModelMetadataView/ModelMetadataView.tsx     | 5 ++++-
 src/features/ModelView/ModelNameFrame/ModelNameFrame.tsx  | 8 ++++++--
 8 files changed, 26 insertions(+), 9 deletions(-)

diff --git a/src/components/GrossDepositionEnviroment/GrossDepositionEnviromentGroup/GrossDepositionEnviromentGroup.tsx b/src/components/GrossDepositionEnviroment/GrossDepositionEnviromentGroup/GrossDepositionEnviromentGroup.tsx
index 26e446c..e2f665a 100644
--- a/src/components/GrossDepositionEnviroment/GrossDepositionEnviromentGroup/GrossDepositionEnviromentGroup.tsx
+++ b/src/components/GrossDepositionEnviroment/GrossDepositionEnviromentGroup/GrossDepositionEnviromentGroup.tsx
@@ -106,7 +106,7 @@ export const GrossDepositionEnviromentGroup = ({
   return (
     <>
       <Styled.Wrapper>
-        <Typography variant="h4">
+        <Typography variant="h4" as="h3">
           Gross Depositional Environment (GDE)
         </Typography>
         <Table>
diff --git a/src/components/OutcropAnalogue/OutcropAnalogueGroup/OutcropAnalogueGroup.tsx b/src/components/OutcropAnalogue/OutcropAnalogueGroup/OutcropAnalogueGroup.tsx
index 43a99fd..5b5c141 100644
--- a/src/components/OutcropAnalogue/OutcropAnalogueGroup/OutcropAnalogueGroup.tsx
+++ b/src/components/OutcropAnalogue/OutcropAnalogueGroup/OutcropAnalogueGroup.tsx
@@ -79,7 +79,9 @@ export const OutcropAnalogueGroup = ({
 
   return (
     <Styled.Wrapper>
-      <Typography variant="h4">Outcrop Analogue</Typography>
+      <Typography variant="h4" as="h3">
+        Outcrop Analogue
+      </Typography>
       <Table>
         <Table.Head>
           <Table.Row>
diff --git a/src/components/StrategraphicColumn/StratigrapicGroups/StratigrapicGroups.tsx b/src/components/StrategraphicColumn/StratigrapicGroups/StratigrapicGroups.tsx
index eab54f3..905c993 100644
--- a/src/components/StrategraphicColumn/StratigrapicGroups/StratigrapicGroups.tsx
+++ b/src/components/StrategraphicColumn/StratigrapicGroups/StratigrapicGroups.tsx
@@ -28,7 +28,9 @@ export const StratigrapicGroups = ({
 
   return (
     <Styled.Wrapper>
-      <Typography variant="h4">Stratigraphic column</Typography>
+      <Typography variant="h4" as="h3">
+        Stratigraphic column
+      </Typography>
 
       <Table>
         <Table.Head>
diff --git a/src/features/ModelView/DeleteButton/DeleteModel.tsx b/src/features/ModelView/DeleteButton/DeleteModel.tsx
index cf04141..ea942e3 100644
--- a/src/features/ModelView/DeleteButton/DeleteModel.tsx
+++ b/src/features/ModelView/DeleteButton/DeleteModel.tsx
@@ -32,7 +32,9 @@ export const DeleteModel = () => {
 
   return (
     <Styled.Wrapper>
-      <Typography variant="h3">Delete model</Typography>
+      <Typography variant="h3" as="h2">
+        Delete model
+      </Typography>
       <Typography variant="body_long">
         This will delete the model along with all related cases and results.
       </Typography>
diff --git a/src/features/ModelView/ModelAreaCoordinates/ModelAreaCoordinates.tsx b/src/features/ModelView/ModelAreaCoordinates/ModelAreaCoordinates.tsx
index bc5c320..cf67f7c 100644
--- a/src/features/ModelView/ModelAreaCoordinates/ModelAreaCoordinates.tsx
+++ b/src/features/ModelView/ModelAreaCoordinates/ModelAreaCoordinates.tsx
@@ -8,7 +8,9 @@ export const ModelAreaCoordinates = ({
 }) => {
   return (
     <Styled.Wrapper>
-      <Typography variant="h3">Model areas</Typography>
+      <Typography variant="h3" as="h2">
+        Model areas
+      </Typography>
       <Typography variant="body_long">
         You can define multiple areas for calculation in your model by entering
         coordinates.
diff --git a/src/features/ModelView/ModelFilesView/ModelFilesView.tsx b/src/features/ModelView/ModelFilesView/ModelFilesView.tsx
index 3012797..acdf06b 100644
--- a/src/features/ModelView/ModelFilesView/ModelFilesView.tsx
+++ b/src/features/ModelView/ModelFilesView/ModelFilesView.tsx
@@ -10,7 +10,9 @@ export const ModelFilesView = () => {
 
   return (
     <Styled.TableWrapper>
-      <Typography variant="h3">Files</Typography>
+      <Typography variant="h3" as="h2">
+        Files
+      </Typography>
       <Styled.FileTable>
         <Table.Head>
           <Table.Row className="table-row">
diff --git a/src/features/ModelView/ModelMetadataView/ModelMetadataView.tsx b/src/features/ModelView/ModelMetadataView/ModelMetadataView.tsx
index 34b3f05..1d5599e 100644
--- a/src/features/ModelView/ModelMetadataView/ModelMetadataView.tsx
+++ b/src/features/ModelView/ModelMetadataView/ModelMetadataView.tsx
@@ -272,7 +272,6 @@ export const ModelMetadataView = ({
 
   return (
     <Styled.Wrapper>
-      <Typography variant="h3">Model metadata</Typography>
       {!isAddUploading && (
         <Styled.DescriptionMeta>
           <>
@@ -310,6 +309,10 @@ export const ModelMetadataView = ({
         </>
       )}
 
+      <Typography variant="h3" as="h2">
+        Model metadata
+      </Typography>
+
       <div>
         <OutcropAnalogueGroup
           modelIdParent={modelIdParent}
diff --git a/src/features/ModelView/ModelNameFrame/ModelNameFrame.tsx b/src/features/ModelView/ModelNameFrame/ModelNameFrame.tsx
index aa66620..13af2e8 100644
--- a/src/features/ModelView/ModelNameFrame/ModelNameFrame.tsx
+++ b/src/features/ModelView/ModelNameFrame/ModelNameFrame.tsx
@@ -6,9 +6,13 @@ export const ModelNameFrame = ({ model }: { model?: AnalogueModelDetail }) => {
   return (
     <Styled.NameFrame className="metadata-name-frame">
       {model ? (
-        <Typography variant="h2">{model.name}</Typography>
+        <Typography variant="h2" as="h1">
+          {model.name}
+        </Typography>
       ) : (
-        <Typography variant="h2">Loading ....</Typography>
+        <Typography variant="h2" as="h1">
+          Loading ....
+        </Typography>
       )}
     </Styled.NameFrame>
   );