[Bugfix] Update UI and handle new navigation (#416)

* update reporting UI and handle new navigation Signed-off-by: Adam Tackett <[email protected]> * fix linting errors Signed-off-by: Adam Tackett <[email protected]> * fix linting errors Signed-off-by: Adam Tackett <[email protected]> * fix spacing from comment Signed-off-by: Adam Tackett <[email protected]> * fix comments Signed-off-by: Adam Tackett <[email protected]> * fix lint spacing in app Signed-off-by: Adam Tackett <[email protected]> * more lint Signed-off-by: Adam Tackett <[email protected]> * and more lint Signed-off-by: Adam Tackett <[email protected]> --------- Signed-off-by: Adam Tackett <[email protected]> Co-authored-by: Adam Tackett <[email protected]>
opensearch-project · Sep 5, 2024 · 20dea69 · 20dea69
1 parent 0af5edd
commit 20dea69
Show file tree

Hide file tree

Showing 17 changed files with 685 additions and 2,633 deletions.
diff --git a/public/components/app.tsx b/public/components/app.tsx
@@ -4,31 +4,21 @@
  */
 
 import React from 'react';
-import { FormattedMessage, I18nProvider } from '@osd/i18n/react';
+import { I18nProvider } from '@osd/i18n/react';
 import { HashRouter as Router, Route, Switch } from 'react-router-dom';
-
-import {
-  EuiPage,
-  EuiPageBody,
-  EuiPageContentBody,
-  EuiPageContentHeader,
-  EuiPageContentHeaderSection,
-} from '@elastic/eui';
-import CSS from 'csstype';
+import { i18n } from '@osd/i18n';
+import { EuiPage, EuiPageBody, EuiPageContentBody } from '@elastic/eui';
 import {
   CoreStart,
-  CoreSystem,
   ChromeBreadcrumb,
   IUiSettingsClient,
 } from '../../../../src/core/public';
 import { NavigationPublicPluginStart } from '../../../../src/plugins/navigation/public';
-
 import { CreateReport } from './report_definitions/create/create_report_definition';
 import { Main } from './main/main';
 import { ReportDetails } from './main/report_details/report_details';
 import { ReportDefinitionDetails } from './main/report_definition_details/report_definition_details';
 import { EditReportDefinition } from './report_definitions/edit/edit_report_definition';
-import { i18n } from '@osd/i18n';
 
 export interface CoreInterface {
   http: CoreStart['http'];
@@ -44,29 +34,18 @@ interface ReportsDashboardsAppDeps {
   chrome: CoreStart['chrome'];
 }
 
-const styles: CSS.Properties = {
-  float: 'left',
-  width: '100%',
-  maxWidth: '1600px',
-};
-
 export const ReportsDashboardsApp = ({
   basename,
-  notifications,
   http,
-  navigation,
   chrome,
 }: ReportsDashboardsAppDeps) => {
   // Render the application DOM.
   return (
     <Router basename={'/' + basename}>
       <I18nProvider>
-        <div style={styles}>
+        <div>
           <EuiPage>
             <EuiPageBody>
-              <EuiPageContentHeader>
-                <EuiPageContentHeaderSection></EuiPageContentHeaderSection>
-              </EuiPageContentHeader>
               <EuiPageContentBody>
                 <Switch>
                   <Route
@@ -78,6 +57,7 @@ export const ReportsDashboardsApp = ({
                           { defaultMessage: 'Report Details' }
                         )}
                         httpClient={http}
+                        chrome={chrome}
                         {...props}
                         setBreadcrumbs={chrome.setBreadcrumbs}
                       />
@@ -92,6 +72,7 @@ export const ReportsDashboardsApp = ({
                           { defaultMessage: 'Report Definition Details' }
                         )}
                         httpClient={http}
+                        chrome={chrome}
                         {...props}
                         setBreadcrumbs={chrome.setBreadcrumbs}
                       />
@@ -106,6 +87,7 @@ export const ReportsDashboardsApp = ({
                           { defaultMessage: 'Create Report' }
                         )}
                         httpClient={http}
+                        chrome={chrome}
                         {...props}
                         setBreadcrumbs={chrome.setBreadcrumbs}
                       />
@@ -136,6 +118,7 @@ export const ReportsDashboardsApp = ({
                         httpClient={http}
                         {...props}
                         setBreadcrumbs={chrome.setBreadcrumbs}
+                        chrome={chrome}
                       />
                     )}
                   />