Incorporate feel and look good guide changes for snapshots pages, dat…

…astreams, rollups and notification settings (#1132) * Incorporate feel and look good guide changes for snapshots pages, datastreams, rollups and notification settings Signed-off-by: Sandeep Kumawat <[email protected]> * update snapshots of pages after feel and look good chanegs Signed-off-by: Sandeep Kumawat <[email protected]> --------- Signed-off-by: Sandeep Kumawat <[email protected]> Co-authored-by: Sandeep Kumawat <[email protected]> (cherry picked from commit a53ff41) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-project · Aug 20, 2024 · 5d5ed83 · 5d5ed83
1 parent a88bc23
commit 5d5ed83
Show file tree

Hide file tree

Showing 64 changed files with 226 additions and 234 deletions.
diff --git a/public/components/ContentPanel/ContentPanel.tsx b/public/components/ContentPanel/ContentPanel.tsx
@@ -72,10 +72,10 @@ const ContentPanel: React.SFC<ContentPanelProps> = ({
         <EuiFlexItem>
           {typeof title === "string" ? (
             <EuiTitle size={titleSize}>
-              <h3>
+              <h1>
                 {title}
                 <span className="panel-header-count"> {itemCount > 0 ? `(${itemCount})` : null} </span>
-              </h3>
+              </h1>
             </EuiTitle>
           ) : (
             title

diff --git a/public/components/ContentPanel/__snapshots__/ContentPanel.test.tsx.snap b/public/components/ContentPanel/__snapshots__/ContentPanel.test.tsx.snap
@@ -12,7 +12,7 @@ exports[`<ContentPanel /> spec renders the component 1`] = `
     <div
       class="euiFlexItem"
     >
-      <h3
+      <h1
         class="euiTitle euiTitle--large"
       >
         Testing
@@ -22,7 +22,7 @@ exports[`<ContentPanel /> spec renders the component 1`] = `
            
            
         </span>
-      </h3>
+      </h1>
     </div>
     <div
       class="euiFlexItem euiFlexItem--flexGrowZero"

diff --git a/public/components/DeleteModal/DeleteModal.tsx b/public/components/DeleteModal/DeleteModal.tsx
@@ -54,7 +54,7 @@ export default function DeleteTemplateModal(props: DeleteModalProps) {
             ))}
           </ul>
           <EuiSpacer />
-          <EuiText color="subdued">
+          <EuiText color="subdued" size="s">
             To confirm your action, type <b style={{ color: "#000" }}>delete</b>.
           </EuiText>
           <EuiFieldText

diff --git a/public/components/DeleteModal/__snapshots__/DeleteModal.test.tsx.snap b/public/components/DeleteModal/__snapshots__/DeleteModal.test.tsx.snap
@@ -65,7 +65,7 @@ HTMLCollection [
                   class="euiSpacer euiSpacer--l"
                 />
                 <div
-                  class="euiText euiText--medium"
+                  class="euiText euiText--small"
                 >
                   <div
                     class="euiTextColor euiTextColor--subdued"

diff --git a/public/components/IndexDetail/__snapshots__/IndexDetail.test.tsx.snap b/public/components/IndexDetail/__snapshots__/IndexDetail.test.tsx.snap
@@ -12,7 +12,7 @@ exports[`<IndexDetail /> spec renders the component 1`] = `
     <div
       class="euiFlexItem"
     >
-      <h3
+      <h1
         class="euiTitle euiTitle--small"
       >
         Define index
@@ -22,7 +22,7 @@ exports[`<IndexDetail /> spec renders the component 1`] = `
            
            
         </span>
-      </h3>
+      </h1>
     </div>
   </div>
   <hr

diff --git a/public/containers/IndexDetail/__snapshots__/IndexDetail.test.tsx.snap b/public/containers/IndexDetail/__snapshots__/IndexDetail.test.tsx.snap
@@ -13,7 +13,7 @@ exports[`<IndexDetail /> spec render the component 1`] = `
       <div
         class="euiFlexItem"
       >
-        <h3
+        <h1
           class="euiTitle euiTitle--small"
         >
           Source index details
@@ -23,7 +23,7 @@ exports[`<IndexDetail /> spec render the component 1`] = `
              
              
           </span>
-        </h3>
+        </h1>
       </div>
     </div>
     <hr

diff --git a/public/containers/IndexForm/__snapshots__/IndexForm.test.tsx.snap b/public/containers/IndexForm/__snapshots__/IndexForm.test.tsx.snap
@@ -13,7 +13,7 @@ exports[`<IndexForm /> spec render page 1`] = `
       <div
         class="euiFlexItem"
       >
-        <h3
+        <h1
           class="euiTitle euiTitle--small"
         >
           Define index
@@ -23,7 +23,7 @@ exports[`<IndexForm /> spec render page 1`] = `
              
              
           </span>
-        </h3>
+        </h1>
       </div>
     </div>
     <hr
@@ -195,7 +195,7 @@ exports[`<IndexForm /> spec render page 1`] = `
       <div
         class="euiFlexItem"
       >
-        <h3
+        <h1
           class="euiTitle euiTitle--small"
         >
           Index settings
@@ -205,7 +205,7 @@ exports[`<IndexForm /> spec render page 1`] = `
              
              
           </span>
-        </h3>
+        </h1>
       </div>
     </div>
     <hr

diff --git a/...gePolicy/components/ChangeManagedIndices/__snapshots__/ChangeManagedIndices.test.tsx.snap b/...gePolicy/components/ChangeManagedIndices/__snapshots__/ChangeManagedIndices.test.tsx.snap
@@ -12,7 +12,7 @@ exports[`<ChangeManagedIndices /> spec renders the component 1`] = `
     <div
       class="euiFlexItem"
     >
-      <h3
+      <h1
         class="euiTitle euiTitle--small"
       >
         Choose managed indices
@@ -22,7 +22,7 @@ exports[`<ChangeManagedIndices /> spec renders the component 1`] = `
            
            
         </span>
-      </h3>
+      </h1>
     </div>
   </div>
   <hr

diff --git a/public/pages/ChangePolicy/components/NewPolicy/__snapshots__/NewPolicy.test.tsx.snap b/public/pages/ChangePolicy/components/NewPolicy/__snapshots__/NewPolicy.test.tsx.snap
@@ -12,7 +12,7 @@ exports[`<NewPolicy /> spec renders the component 1`] = `
     <div
       class="euiFlexItem"
     >
-      <h3
+      <h1
         class="euiTitle euiTitle--small"
       >
         Choose new policy
@@ -22,7 +22,7 @@ exports[`<NewPolicy /> spec renders the component 1`] = `
            
            
         </span>
-      </h3>
+      </h1>
     </div>
   </div>
   <hr

diff --git a/public/pages/ChangePolicy/containers/ChangePolicy/__snapshots__/ChangePolicy.test.tsx.snap b/public/pages/ChangePolicy/containers/ChangePolicy/__snapshots__/ChangePolicy.test.tsx.snap
@@ -23,7 +23,7 @@ exports[`<ChangePolicy /> spec renders the component 1`] = `
       <div
         class="euiFlexItem"
       >
-        <h3
+        <h1
           class="euiTitle euiTitle--small"
         >
           Choose managed indices
@@ -33,7 +33,7 @@ exports[`<ChangePolicy /> spec renders the component 1`] = `
              
              
           </span>
-        </h3>
+        </h1>
       </div>
     </div>
     <hr
@@ -223,7 +223,7 @@ exports[`<ChangePolicy /> spec renders the component 1`] = `
       <div
         class="euiFlexItem"
       >
-        <h3
+        <h1
           class="euiTitle euiTitle--small"
         >
           Choose new policy
@@ -233,7 +233,7 @@ exports[`<ChangePolicy /> spec renders the component 1`] = `
              
              
           </span>
-        </h3>
+        </h1>
       </div>
     </div>
     <hr

diff --git a/public/pages/Commons/BaseAggregationAndMetricSettings.tsx b/public/pages/Commons/BaseAggregationAndMetricSettings.tsx
@@ -92,7 +92,7 @@ export const BaseMetricsColumns: Readonly<EuiTableFieldDataColumnType<MetricItem
 export function sequenceTableComponents(selectedDimensionField, items, columns, pagination, sorting, onChange) {
   if (selectedDimensionField.length == 0) {
     return (
-      <EuiText>
+      <EuiText size="xs">
         <dd>No fields added for aggregation</dd>
       </EuiText>
     );
@@ -120,12 +120,12 @@ export function additionalMetricsComponent(selectedMetrics) {
   return (
     <EuiFlexGroup gutterSize="xs">
       <EuiFlexItem grow={false}>
-        <EuiText>
+        <EuiText size="s">
           <h3>Additional metrics</h3>
         </EuiText>
       </EuiFlexItem>
       <EuiFlexItem grow={false}>
-        <EuiText color="subdued" textAlign="left">
+        <EuiText color="subdued" textAlign="left" size="s">
           <h3>{`(${selectedMetrics.length})`}</h3>
         </EuiText>
       </EuiFlexItem>
@@ -136,7 +136,7 @@ export function additionalMetricsComponent(selectedMetrics) {
 export function sourceFieldComponents(selectedMetrics, items, columns, pagination, sorting, onChange) {
   if (selectedMetrics.length == 0) {
     return (
-      <EuiText>
+      <EuiText size="xs">
         <dd>No fields added for metrics</dd>
       </EuiText>
     );

diff --git a/.../containers/CreateComposableTemplate/__snapshots__/CreateComposableTemplate.test.tsx.snap b/.../containers/CreateComposableTemplate/__snapshots__/CreateComposableTemplate.test.tsx.snap
@@ -68,7 +68,7 @@ exports[`<CreateComposableTemplate /> spec it goes to templates page when click
         <div
           class="euiFlexItem"
         >
-          <h3
+          <h1
             class="euiTitle euiTitle--small"
           >
             Define component template
@@ -78,7 +78,7 @@ exports[`<CreateComposableTemplate /> spec it goes to templates page when click
                
                
             </span>
-          </h3>
+          </h1>
         </div>
       </div>
       <hr
@@ -272,7 +272,7 @@ exports[`<CreateComposableTemplate /> spec it goes to templates page when click
         <div
           class="euiFlexItem"
         >
-          <h3
+          <h1
             class="euiTitle euiTitle--small"
           >
             Index settings
@@ -282,7 +282,7 @@ exports[`<CreateComposableTemplate /> spec it goes to templates page when click
                
                
             </span>
-          </h3>
+          </h1>
         </div>
         <div
           class="euiFlexItem euiFlexItem--flexGrowZero"

diff --git a/...teComposableTemplate/containers/TemplateDetail/__snapshots__/TemplateDetail.test.tsx.snap b/...teComposableTemplate/containers/TemplateDetail/__snapshots__/TemplateDetail.test.tsx.snap
@@ -53,7 +53,7 @@ exports[`<TemplateDetail /> spec render component 1`] = `
       <div
         class="euiFlexItem"
       >
-        <h3
+        <h1
           class="euiTitle euiTitle--small"
         >
           Define component template
@@ -63,7 +63,7 @@ exports[`<TemplateDetail /> spec render component 1`] = `
              
              
           </span>
-        </h3>
+        </h1>
       </div>
     </div>
     <hr
@@ -307,7 +307,7 @@ exports[`<TemplateDetail /> spec render component 1`] = `
       <div
         class="euiFlexItem"
       >
-        <h3
+        <h1
           class="euiTitle euiTitle--small"
         >
           Index settings
@@ -317,7 +317,7 @@ exports[`<TemplateDetail /> spec render component 1`] = `
              
              
           </span>
-        </h3>
+        </h1>
       </div>
       <div
         class="euiFlexItem euiFlexItem--flexGrowZero"

diff --git a/...CreateDataStream/containers/CreateDataStream/__snapshots__/CreateDataStream.test.tsx.snap b/...CreateDataStream/containers/CreateDataStream/__snapshots__/CreateDataStream.test.tsx.snap
@@ -100,7 +100,7 @@ exports[`<CreateDataStream /> spec it goes to data streams page when click cance
         <div
           class="euiFlexItem"
         >
-          <h3
+          <h1
             class="euiTitle euiTitle--small"
           >
             Define data stream
@@ -110,7 +110,7 @@ exports[`<CreateDataStream /> spec it goes to data streams page when click cance
                
                
             </span>
-          </h3>
+          </h1>
         </div>
       </div>
       <hr

diff --git a/public/pages/CreateDataStream/containers/DataStreamDetail/DataStreamDetail.tsx b/public/pages/CreateDataStream/containers/DataStreamDetail/DataStreamDetail.tsx
@@ -117,8 +117,7 @@ const DataStreamDetail = (props: DataStreamDetailProps, ref: Ref<FieldInstance>)
         <EuiText size="s" color="subdued">
           A data stream is composed of multiple backing indexes. Search requests are routed to all the backing indexes, while indexing
           requests <br></br>
-          are routed to the latest write index.
-          <ExternalLink href={coreServices.docLinks.links.opensearch.dataStreams} />
+          are routed to the latest write index. <ExternalLink href={coreServices.docLinks.links.opensearch.dataStreams} />
         </EuiText>
       ),
     },

diff --git a/...CreateDataStream/containers/DataStreamDetail/__snapshots__/DataStreamDetail.test.tsx.snap b/...CreateDataStream/containers/DataStreamDetail/__snapshots__/DataStreamDetail.test.tsx.snap
@@ -97,7 +97,7 @@ exports[`<DataStreamDetail /> spec render component 1`] = `
       <div
         class="euiFlexItem"
       >
-        <h3
+        <h1
           class="euiTitle euiTitle--small"
         >
           Define data stream
@@ -107,7 +107,7 @@ exports[`<DataStreamDetail /> spec render component 1`] = `
              
              
           </span>
-        </h3>
+        </h1>
       </div>
     </div>
     <hr

diff --git a/.../CreateIndexTemplate/components/DefineTemplate/__snapshots__/DefineTemplate.test.tsx.snap b/.../CreateIndexTemplate/components/DefineTemplate/__snapshots__/DefineTemplate.test.tsx.snap
@@ -13,7 +13,7 @@ exports[`<ComposableTemplatesActions /> spec renders the component in edit mode
       <div
         class="euiFlexItem"
       >
-        <h3
+        <h1
           class="euiTitle euiTitle--small"
         >
           Template settings
@@ -23,7 +23,7 @@ exports[`<ComposableTemplatesActions /> spec renders the component in edit mode
              
              
           </span>
-        </h3>
+        </h1>
       </div>
     </div>
     <hr
@@ -408,7 +408,7 @@ exports[`<ComposableTemplatesActions /> spec renders the component in non-edit m
       <div
         class="euiFlexItem"
       >
-        <h3
+        <h1
           class="euiTitle euiTitle--small"
         >
           Template settings
@@ -418,7 +418,7 @@ exports[`<ComposableTemplatesActions /> spec renders the component in non-edit m
              
              
           </span>
-        </h3>
+        </h1>
       </div>
     </div>
     <hr
-Original file line number
+Diff line change
@@ Expand Up / @@ -12,7 +12,7 @@ exports[`<ContentPanel /> spec renders the component 1`] = ` @@
         <div
           class="euiFlexItem"
         >
-          <h3
+          <h1
             class="euiTitle euiTitle--large"
           >
             Testing
@@ Expand All / @@ -22,7 +22,7 @@ exports[`<ContentPanel /> spec renders the component 1`] = ` @@
             </span>
-          </h3>
+          </h1>
         </div>
         <div
           class="euiFlexItem euiFlexItem--flexGrowZero"
@@ Expand Down @@