review + fix

gridsuite · Dec 8, 2023 · 97b32d4 · 97b32d4
1 parent 7894328
commit 97b32d4
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 5 deletions.
diff --git a/demo/src/app.js b/demo/src/app.js
@@ -715,7 +715,7 @@ const AppContent = ({ language, onLanguageClick }) => {
                             appVersion={AppPackage.version}
                             appLicense={AppPackage.license}
                             getGlobalVersion={simulateGetGlobalVersion}
-                            getAdditionalComponents={
+                            getAdditionalModules={
                                 simulateGetAdditionalComponents
                             }
                             onEquipmentLabellingClick={

diff --git a/src/components/TopBar/AboutDialog.js b/src/components/TopBar/AboutDialog.js
@@ -246,6 +246,9 @@ const AboutDialog = ({
                             //dunno why the have the background as black in dark mode
                             bgcolor: 'unset',
                         },
+                        '.MuiAccordionSummary-content > .MuiSvgIcon-root': {
+                            marginRight: '0.5rem',
+                        },
                     }}
                 >
                     {/* TODO found how to scroll only in this section, to keep logo and global version always visible */}
@@ -291,7 +294,12 @@ const AboutDialog = ({
                         <AccordionDetails>
                             <Grid container sx={{ pl: 2 }} spacing={1}>
                                 {loadingAdditionalModules ? (
-                                    <Grid item xs>
+                                    <Grid
+                                        item
+                                        xs
+                                        display="inline-flex"
+                                        justifyContent="center"
+                                    >
                                         <CircularProgress color="inherit" />
                                     </Grid>
                                 ) : (

diff --git a/src/components/TopBar/TopBar.js b/src/components/TopBar/TopBar.js
@@ -169,7 +169,7 @@ const TopBar = ({
     appsAndUrls,
     onAboutClick,
     getGlobalVersion,
-    getAdditionalComponents,
+    getAdditionalModules,
     onThemeClick,
     theme,
     onEquipmentLabellingClick,
@@ -754,7 +754,7 @@ const TopBar = ({
                     appVersion={appVersion}
                     appLicense={appLicense}
                     getGlobalVersion={getGlobalVersion}
-                    getAdditionalComponents={getAdditionalComponents}
+                    getAdditionalModules={getAdditionalModules}
                 />
             </Toolbar>
         </AppBar>
@@ -777,7 +777,7 @@ TopBar.propTypes = {
     theme: PropTypes.string,
     onAboutClick: PropTypes.func,
     getGlobalVersion: PropTypes.func,
-    getAdditionalComponents: PropTypes.func,
+    getAdditionalModules: PropTypes.func,
     onEquipmentLabellingClick: PropTypes.func,
     equipmentLabelling: PropTypes.bool,
     withElementsSearch: PropTypes.bool,