From a49ee089fb87279dbbde80ce2688c7605b9f9ddf Mon Sep 17 00:00:00 2001
From: James Gowdy <jgowdy@elastic.co>
Date: Tue, 20 Dec 2022 12:56:45 +0000
Subject: [PATCH] [ML] Fix modal titles

---
 .../delete_space_aware_item_check_modal.tsx    | 10 ++++++----
 .../validate_job/validate_job_view.js          |  4 +++-
 .../add_to_dashboard_controls.tsx              |  4 +++-
 .../close_jobs_confirm_modal.tsx               |  4 +++-
 .../stop_datafeeds_confirm_modal.tsx           |  4 +++-
 .../delete_job_modal/delete_job_modal.tsx      | 18 ++++++++++--------
 .../edit_job_flyout/tabs/custom_urls.tsx       | 10 ++++++----
 .../reset_job_modal/reset_job_modal.tsx        | 18 ++++++++++--------
 .../start_datafeed_modal.js                    | 18 ++++++++++--------
 .../components/data_view/change_data_view.tsx  | 10 ++++++----
 .../advanced_detector_modal/modal_wrapper.tsx  | 10 ++++++----
 .../edit/import_modal/import_modal.js          | 10 ++++++----
 .../edit/new_event_modal/new_event_modal.js    | 10 ++++++----
 .../components/forecasting_modal/modal.js      | 10 ++++++----
 .../models_management/delete_models_modal.tsx  | 18 ++++++++++--------
 .../models_management/deployment_setup.tsx     |  4 ++--
 .../anomaly_charts_initializer.tsx             |  4 ++--
 .../anomaly_swimlane_initializer.tsx           | 10 ++++++----
 18 files changed, 104 insertions(+), 72 deletions(-)

diff --git a/x-pack/plugins/ml/public/application/components/delete_space_aware_item_check_modal/delete_space_aware_item_check_modal.tsx b/x-pack/plugins/ml/public/application/components/delete_space_aware_item_check_modal/delete_space_aware_item_check_modal.tsx
index 4b576695c981a4..8ac2a22be5a930 100644
--- a/x-pack/plugins/ml/public/application/components/delete_space_aware_item_check_modal/delete_space_aware_item_check_modal.tsx
+++ b/x-pack/plugins/ml/public/application/components/delete_space_aware_item_check_modal/delete_space_aware_item_check_modal.tsx
@@ -339,10 +339,12 @@ export const DeleteSpaceAwareItemCheckModal: FC<Props> = ({
         <>
           <EuiModalHeader>
             <EuiModalHeaderTitle>
-              <FormattedMessage
-                id="xpack.ml.deleteSpaceAwareItemCheckModal.modalTitle"
-                defaultMessage="Checking space permissions"
-              />
+              <h1>
+                <FormattedMessage
+                  id="xpack.ml.deleteSpaceAwareItemCheckModal.modalTitle"
+                  defaultMessage="Checking space permissions"
+                />
+              </h1>
             </EuiModalHeaderTitle>
           </EuiModalHeader>
 
diff --git a/x-pack/plugins/ml/public/application/components/validate_job/validate_job_view.js b/x-pack/plugins/ml/public/application/components/validate_job/validate_job_view.js
index 64910f5abfd214..cc76a708fc77b8 100644
--- a/x-pack/plugins/ml/public/application/components/validate_job/validate_job_view.js
+++ b/x-pack/plugins/ml/public/application/components/validate_job/validate_job_view.js
@@ -84,7 +84,9 @@ const LoadingSpinner = () => (
 const Modal = ({ close, title, children }) => (
   <EuiModal onClose={close} style={{ width: '800px' }}>
     <EuiModalHeader>
-      <EuiModalHeaderTitle>{title}</EuiModalHeaderTitle>
+      <EuiModalHeaderTitle>
+        <h1>{title}</h1>
+      </EuiModalHeaderTitle>
     </EuiModalHeader>
 
     <EuiModalBody>{children}</EuiModalBody>
diff --git a/x-pack/plugins/ml/public/application/explorer/dashboard_controls/add_to_dashboard_controls.tsx b/x-pack/plugins/ml/public/application/explorer/dashboard_controls/add_to_dashboard_controls.tsx
index d030cfc9b92b62..a5b24a2290d673 100644
--- a/x-pack/plugins/ml/public/application/explorer/dashboard_controls/add_to_dashboard_controls.tsx
+++ b/x-pack/plugins/ml/public/application/explorer/dashboard_controls/add_to_dashboard_controls.tsx
@@ -81,7 +81,9 @@ export const AddToDashboardControl: FC<AddToDashboardControlProps> = ({
   return (
     <EuiModal onClose={onClose.bind(null, undefined)} data-test-subj="mlAddToDashboardModal">
       <EuiModalHeader>
-        <EuiModalHeaderTitle>{title}</EuiModalHeaderTitle>
+        <EuiModalHeaderTitle>
+          <h1>{title}</h1>
+        </EuiModalHeaderTitle>
       </EuiModalHeader>
       <EuiModalBody>
         {children}
diff --git a/x-pack/plugins/ml/public/application/jobs/jobs_list/components/confirm_modals/close_jobs_confirm_modal.tsx b/x-pack/plugins/ml/public/application/jobs/jobs_list/components/confirm_modals/close_jobs_confirm_modal.tsx
index f0c78c3828aebf..e1ec74412990bc 100644
--- a/x-pack/plugins/ml/public/application/jobs/jobs_list/components/confirm_modals/close_jobs_confirm_modal.tsx
+++ b/x-pack/plugins/ml/public/application/jobs/jobs_list/components/confirm_modals/close_jobs_confirm_modal.tsx
@@ -87,7 +87,9 @@ export const CloseJobsConfirmModal: FC<Props> = ({
     return (
       <EuiModal data-test-subj="mlCloseJobsConfirmModal" onClose={closeModal}>
         <EuiModalHeader>
-          <EuiModalHeaderTitle>{title}</EuiModalHeaderTitle>
+          <EuiModalHeaderTitle>
+            <h1>{title}</h1>
+          </EuiModalHeaderTitle>
         </EuiModalHeader>
         <EuiModalBody>
           <ManagedJobsWarningCallout
diff --git a/x-pack/plugins/ml/public/application/jobs/jobs_list/components/confirm_modals/stop_datafeeds_confirm_modal.tsx b/x-pack/plugins/ml/public/application/jobs/jobs_list/components/confirm_modals/stop_datafeeds_confirm_modal.tsx
index 2edd875279ad01..e7bd57229d3ffe 100644
--- a/x-pack/plugins/ml/public/application/jobs/jobs_list/components/confirm_modals/stop_datafeeds_confirm_modal.tsx
+++ b/x-pack/plugins/ml/public/application/jobs/jobs_list/components/confirm_modals/stop_datafeeds_confirm_modal.tsx
@@ -88,7 +88,9 @@ export const StopDatafeedsConfirmModal: FC<Props> = ({
     return (
       <EuiModal data-test-subj="mlStopDatafeedsConfirmModal" onClose={closeModal}>
         <EuiModalHeader>
-          <EuiModalHeaderTitle>{title}</EuiModalHeaderTitle>
+          <EuiModalHeaderTitle>
+            <h1>{title}</h1>
+          </EuiModalHeaderTitle>
         </EuiModalHeader>
         <EuiModalBody>
           <ManagedJobsWarningCallout
diff --git a/x-pack/plugins/ml/public/application/jobs/jobs_list/components/delete_job_modal/delete_job_modal.tsx b/x-pack/plugins/ml/public/application/jobs/jobs_list/components/delete_job_modal/delete_job_modal.tsx
index b7cadf0def05a3..3bb86efd4263a7 100644
--- a/x-pack/plugins/ml/public/application/jobs/jobs_list/components/delete_job_modal/delete_job_modal.tsx
+++ b/x-pack/plugins/ml/public/application/jobs/jobs_list/components/delete_job_modal/delete_job_modal.tsx
@@ -87,14 +87,16 @@ export const DeleteJobModal: FC<Props> = ({ setShowFunction, unsetShowFunction,
       <EuiModal data-test-subj="mlDeleteJobConfirmModal" onClose={closeModal}>
         <EuiModalHeader>
           <EuiModalHeaderTitle>
-            <FormattedMessage
-              id="xpack.ml.jobsList.deleteJobModal.deleteJobsTitle"
-              defaultMessage="Delete {jobsCount, plural, one {{jobId}} other {# jobs}}?"
-              values={{
-                jobsCount: jobIds.length,
-                jobId: jobIds[0],
-              }}
-            />
+            <h1>
+              <FormattedMessage
+                id="xpack.ml.jobsList.deleteJobModal.deleteJobsTitle"
+                defaultMessage="Delete {jobsCount, plural, one {{jobId}} other {# jobs}}?"
+                values={{
+                  jobsCount: jobIds.length,
+                  jobId: jobIds[0],
+                }}
+              />
+            </h1>
           </EuiModalHeaderTitle>
         </EuiModalHeader>
         <EuiModalBody>
diff --git a/x-pack/plugins/ml/public/application/jobs/jobs_list/components/edit_job_flyout/tabs/custom_urls.tsx b/x-pack/plugins/ml/public/application/jobs/jobs_list/components/edit_job_flyout/tabs/custom_urls.tsx
index 9a7508c634c6b9..d77ed5c3bb2db8 100644
--- a/x-pack/plugins/ml/public/application/jobs/jobs_list/components/edit_job_flyout/tabs/custom_urls.tsx
+++ b/x-pack/plugins/ml/public/application/jobs/jobs_list/components/edit_job_flyout/tabs/custom_urls.tsx
@@ -288,10 +288,12 @@ class CustomUrlsUI extends Component<CustomUrlsProps, CustomUrlsState> {
       >
         <EuiModalHeader>
           <EuiModalHeaderTitle>
-            <FormattedMessage
-              id="xpack.ml.jobsList.editJobFlyout.customUrls.addCustomUrlButtonLabel"
-              defaultMessage="Add custom URL"
-            />
+            <h1>
+              <FormattedMessage
+                id="xpack.ml.jobsList.editJobFlyout.customUrls.addCustomUrlButtonLabel"
+                defaultMessage="Add custom URL"
+              />
+            </h1>
           </EuiModalHeaderTitle>
         </EuiModalHeader>
 
diff --git a/x-pack/plugins/ml/public/application/jobs/jobs_list/components/reset_job_modal/reset_job_modal.tsx b/x-pack/plugins/ml/public/application/jobs/jobs_list/components/reset_job_modal/reset_job_modal.tsx
index 97f6255d213faf..4bdf404cb6a60c 100644
--- a/x-pack/plugins/ml/public/application/jobs/jobs_list/components/reset_job_modal/reset_job_modal.tsx
+++ b/x-pack/plugins/ml/public/application/jobs/jobs_list/components/reset_job_modal/reset_job_modal.tsx
@@ -85,14 +85,16 @@ export const ResetJobModal: FC<Props> = ({ setShowFunction, unsetShowFunction, r
     <EuiModal data-test-subj="mlResetJobConfirmModal" onClose={closeModal}>
       <EuiModalHeader>
         <EuiModalHeaderTitle>
-          <FormattedMessage
-            id="xpack.ml.jobsList.resetJobModal.resetJobsTitle"
-            defaultMessage="Reset {jobsCount, plural, one {{jobId}} other {# jobs}}?"
-            values={{
-              jobsCount: jobIds.length,
-              jobId: jobIds[0],
-            }}
-          />
+          <h1>
+            <FormattedMessage
+              id="xpack.ml.jobsList.resetJobModal.resetJobsTitle"
+              defaultMessage="Reset {jobsCount, plural, one {{jobId}} other {# jobs}}?"
+              values={{
+                jobsCount: jobIds.length,
+                jobId: jobIds[0],
+              }}
+            />
+          </h1>
         </EuiModalHeaderTitle>
       </EuiModalHeader>
       <EuiModalBody>
diff --git a/x-pack/plugins/ml/public/application/jobs/jobs_list/components/start_datafeed_modal/start_datafeed_modal.js b/x-pack/plugins/ml/public/application/jobs/jobs_list/components/start_datafeed_modal/start_datafeed_modal.js
index c4c53b00591f4d..3a2728a8104271 100644
--- a/x-pack/plugins/ml/public/application/jobs/jobs_list/components/start_datafeed_modal/start_datafeed_modal.js
+++ b/x-pack/plugins/ml/public/application/jobs/jobs_list/components/start_datafeed_modal/start_datafeed_modal.js
@@ -149,14 +149,16 @@ export class StartDatafeedModal extends Component {
         >
           <EuiModalHeader>
             <EuiModalHeaderTitle>
-              <FormattedMessage
-                id="xpack.ml.jobsList.startDatafeedModal.startJobsTitle"
-                defaultMessage="Start {jobsCount, plural, one {{jobId}} other {# jobs}}"
-                values={{
-                  jobsCount: startableJobs.length,
-                  jobId: startableJobs[0].id,
-                }}
-              />
+              <h1>
+                <FormattedMessage
+                  id="xpack.ml.jobsList.startDatafeedModal.startJobsTitle"
+                  defaultMessage="Start {jobsCount, plural, one {{jobId}} other {# jobs}}"
+                  values={{
+                    jobsCount: startableJobs.length,
+                    jobId: startableJobs[0].id,
+                  }}
+                />
+              </h1>
             </EuiModalHeaderTitle>
           </EuiModalHeader>
 
diff --git a/x-pack/plugins/ml/public/application/jobs/new_job/pages/components/datafeed_step/components/data_view/change_data_view.tsx b/x-pack/plugins/ml/public/application/jobs/new_job/pages/components/datafeed_step/components/data_view/change_data_view.tsx
index cc3b20966935b8..9617458c6e8112 100644
--- a/x-pack/plugins/ml/public/application/jobs/new_job/pages/components/datafeed_step/components/data_view/change_data_view.tsx
+++ b/x-pack/plugins/ml/public/application/jobs/new_job/pages/components/datafeed_step/components/data_view/change_data_view.tsx
@@ -129,10 +129,12 @@ export const ChangeDataViewModal: FC<Props> = ({ onClose }) => {
       <EuiModal onClose={onClose} data-test-subj="mlJobMgmtImportJobsFlyout">
         <EuiModalHeader>
           <EuiModalHeaderTitle>
-            <FormattedMessage
-              id="xpack.ml.newJob.wizard.datafeedStep.dataView.step0.title"
-              defaultMessage="Change data view"
-            />
+            <h1>
+              <FormattedMessage
+                id="xpack.ml.newJob.wizard.datafeedStep.dataView.step0.title"
+                defaultMessage="Change data view"
+              />
+            </h1>
           </EuiModalHeaderTitle>
         </EuiModalHeader>
 
diff --git a/x-pack/plugins/ml/public/application/jobs/new_job/pages/components/pick_fields_step/components/advanced_detector_modal/modal_wrapper.tsx b/x-pack/plugins/ml/public/application/jobs/new_job/pages/components/pick_fields_step/components/advanced_detector_modal/modal_wrapper.tsx
index 12344c651bd113..3ab60cd8225ccf 100644
--- a/x-pack/plugins/ml/public/application/jobs/new_job/pages/components/pick_fields_step/components/advanced_detector_modal/modal_wrapper.tsx
+++ b/x-pack/plugins/ml/public/application/jobs/new_job/pages/components/pick_fields_step/components/advanced_detector_modal/modal_wrapper.tsx
@@ -34,10 +34,12 @@ export const ModalWrapper: FC<Props> = ({ onCreateClick, closeModal, saveEnabled
     >
       <EuiModalHeader>
         <EuiModalHeaderTitle>
-          <FormattedMessage
-            id="xpack.ml.newJob.wizard.pickFieldsStep.advancedDetectorModal.title"
-            defaultMessage="Create detector"
-          />
+          <h1>
+            <FormattedMessage
+              id="xpack.ml.newJob.wizard.pickFieldsStep.advancedDetectorModal.title"
+              defaultMessage="Create detector"
+            />
+          </h1>
         </EuiModalHeaderTitle>
       </EuiModalHeader>
 
diff --git a/x-pack/plugins/ml/public/application/settings/calendars/edit/import_modal/import_modal.js b/x-pack/plugins/ml/public/application/settings/calendars/edit/import_modal/import_modal.js
index 2abec566b43d13..86b54d359ef901 100644
--- a/x-pack/plugins/ml/public/application/settings/calendars/edit/import_modal/import_modal.js
+++ b/x-pack/plugins/ml/public/application/settings/calendars/edit/import_modal/import_modal.js
@@ -147,10 +147,12 @@ export class ImportModal extends Component {
             <EuiFlexGroup direction="column" gutterSize="none">
               <EuiFlexItem grow={false}>
                 <EuiModalHeaderTitle>
-                  <FormattedMessage
-                    id="xpack.ml.calendarsEdit.eventsTable.importEventsTitle"
-                    defaultMessage="Import events"
-                  />
+                  <h1>
+                    <FormattedMessage
+                      id="xpack.ml.calendarsEdit.eventsTable.importEventsTitle"
+                      defaultMessage="Import events"
+                    />
+                  </h1>
                 </EuiModalHeaderTitle>
               </EuiFlexItem>
               <EuiFlexItem grow={false}>
diff --git a/x-pack/plugins/ml/public/application/settings/calendars/edit/new_event_modal/new_event_modal.js b/x-pack/plugins/ml/public/application/settings/calendars/edit/new_event_modal/new_event_modal.js
index ae2815ebb5cd66..255405c8cd2654 100644
--- a/x-pack/plugins/ml/public/application/settings/calendars/edit/new_event_modal/new_event_modal.js
+++ b/x-pack/plugins/ml/public/application/settings/calendars/edit/new_event_modal/new_event_modal.js
@@ -266,10 +266,12 @@ export class NewEventModal extends Component {
         >
           <EuiModalHeader>
             <EuiModalHeaderTitle>
-              <FormattedMessage
-                id="xpack.ml.calendarsEdit.newEventModal.createNewEventTitle"
-                defaultMessage="Create new event"
-              />
+              <h1>
+                <FormattedMessage
+                  id="xpack.ml.calendarsEdit.newEventModal.createNewEventTitle"
+                  defaultMessage="Create new event"
+                />
+              </h1>
             </EuiModalHeaderTitle>
           </EuiModalHeader>
 
diff --git a/x-pack/plugins/ml/public/application/timeseriesexplorer/components/forecasting_modal/modal.js b/x-pack/plugins/ml/public/application/timeseriesexplorer/components/forecasting_modal/modal.js
index 9b7adcf2bce329..1ae5ee2b056042 100644
--- a/x-pack/plugins/ml/public/application/timeseriesexplorer/components/forecasting_modal/modal.js
+++ b/x-pack/plugins/ml/public/application/timeseriesexplorer/components/forecasting_modal/modal.js
@@ -33,10 +33,12 @@ export function Modal(props) {
     <EuiModal onClose={props.close} maxWidth={860} data-test-subj="mlModalForecast">
       <EuiModalHeader>
         <EuiModalHeaderTitle>
-          <FormattedMessage
-            id="xpack.ml.timeSeriesExplorer.forecastingModal.forecastingTitle"
-            defaultMessage="Forecasting"
-          />
+          <h1>
+            <FormattedMessage
+              id="xpack.ml.timeSeriesExplorer.forecastingModal.forecastingTitle"
+              defaultMessage="Forecasting"
+            />
+          </h1>
         </EuiModalHeaderTitle>
       </EuiModalHeader>
 
diff --git a/x-pack/plugins/ml/public/application/trained_models/models_management/delete_models_modal.tsx b/x-pack/plugins/ml/public/application/trained_models/models_management/delete_models_modal.tsx
index b4f0bc0ec505e2..f6799469fd1e69 100644
--- a/x-pack/plugins/ml/public/application/trained_models/models_management/delete_models_modal.tsx
+++ b/x-pack/plugins/ml/public/application/trained_models/models_management/delete_models_modal.tsx
@@ -69,14 +69,16 @@ export const DeleteModelsModal: FC<DeleteModelsModalProps> = ({ modelIds, onClos
     >
       <EuiModalHeader>
         <EuiModalHeaderTitle>
-          <FormattedMessage
-            id="xpack.ml.trainedModels.modelsList.deleteModal.header"
-            defaultMessage="Delete {modelsCount, plural, one {{modelId}} other {# models}}?"
-            values={{
-              modelId: modelIds[0],
-              modelsCount: modelIds.length,
-            }}
-          />
+          <h1>
+            <FormattedMessage
+              id="xpack.ml.trainedModels.modelsList.deleteModal.header"
+              defaultMessage="Delete {modelsCount, plural, one {{modelId}} other {# models}}?"
+              values={{
+                modelId: modelIds[0],
+                modelsCount: modelIds.length,
+              }}
+            />
+          </h1>
         </EuiModalHeaderTitle>
       </EuiModalHeader>
 
diff --git a/x-pack/plugins/ml/public/application/trained_models/models_management/deployment_setup.tsx b/x-pack/plugins/ml/public/application/trained_models/models_management/deployment_setup.tsx
index f8082997a1c6ea..c999376eebeaf0 100644
--- a/x-pack/plugins/ml/public/application/trained_models/models_management/deployment_setup.tsx
+++ b/x-pack/plugins/ml/public/application/trained_models/models_management/deployment_setup.tsx
@@ -289,7 +289,7 @@ export const StartUpdateDeploymentModal: FC<StartDeploymentModalProps> = ({
           <EuiFlexGroup justifyContent={'spaceBetween'}>
             <EuiFlexItem grow={false}>
               <EuiTitle size={'s'}>
-                <h2>
+                <h1>
                   {isUpdate ? (
                     <FormattedMessage
                       id="xpack.ml.trainedModels.modelsList.updateDeployment.modalTitle"
@@ -303,7 +303,7 @@ export const StartUpdateDeploymentModal: FC<StartDeploymentModalProps> = ({
                       values={{ modelId }}
                     />
                   )}
-                </h2>
+                </h1>
               </EuiTitle>
             </EuiFlexItem>
 
diff --git a/x-pack/plugins/ml/public/embeddables/anomaly_charts/anomaly_charts_initializer.tsx b/x-pack/plugins/ml/public/embeddables/anomaly_charts/anomaly_charts_initializer.tsx
index 6e724cf672bb0f..9c2c373a3c1e27 100644
--- a/x-pack/plugins/ml/public/embeddables/anomaly_charts/anomaly_charts_initializer.tsx
+++ b/x-pack/plugins/ml/public/embeddables/anomaly_charts/anomaly_charts_initializer.tsx
@@ -53,12 +53,12 @@ export const AnomalyChartsInitializer: FC<AnomalyChartsInitializerProps> = ({
     >
       <EuiModalHeader>
         <EuiModalHeaderTitle>
-          <h2>
+          <h1>
             <FormattedMessage
               id="xpack.ml.anomalyChartsEmbeddable.setupModal.title"
               defaultMessage="Anomaly explorer charts configuration"
             />
-          </h2>
+          </h1>
         </EuiModalHeaderTitle>
       </EuiModalHeader>
 
diff --git a/x-pack/plugins/ml/public/embeddables/anomaly_swimlane/anomaly_swimlane_initializer.tsx b/x-pack/plugins/ml/public/embeddables/anomaly_swimlane/anomaly_swimlane_initializer.tsx
index e5e2c2fd3b6bed..c4c2fc7262eac5 100644
--- a/x-pack/plugins/ml/public/embeddables/anomaly_swimlane/anomaly_swimlane_initializer.tsx
+++ b/x-pack/plugins/ml/public/embeddables/anomaly_swimlane/anomaly_swimlane_initializer.tsx
@@ -93,10 +93,12 @@ export const AnomalySwimlaneInitializer: FC<AnomalySwimlaneInitializerProps> = (
     <EuiModal initialFocus="[name=panelTitle]" onClose={onCancel}>
       <EuiModalHeader>
         <EuiModalHeaderTitle>
-          <FormattedMessage
-            id="xpack.ml.swimlaneEmbeddable.setupModal.title"
-            defaultMessage="Anomaly swim lane configuration"
-          />
+          <h1>
+            <FormattedMessage
+              id="xpack.ml.swimlaneEmbeddable.setupModal.title"
+              defaultMessage="Anomaly swim lane configuration"
+            />
+          </h1>
         </EuiModalHeaderTitle>
       </EuiModalHeader>