[3902] Move the Help menu entry at the last position

Bug: eclipse-sirius#3902 Signed-off-by: Pierre-Charles David <pierre-charles.david@obeo.fr>
jerome-obeo · Aug 29, 2024 · 99dcd90 · 99dcd90
1 parent 81780f1
commit 99dcd90
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/CHANGELOG.adoc b/CHANGELOG.adoc
@@ -120,6 +120,7 @@ A migration participant has been added to automatically keep compatible all diag
 - https://github.com/eclipse-sirius/sirius-web/issues/3880[#3880] [sirius-web] Make some tests methods more generic
 - https://github.com/eclipse-sirius/sirius-web/issues/3834[#3834] [deck] Add more variables for DeckDescription
 - https://github.com/eclipse-sirius/sirius-web/issues/3897[#3897] [sirius-web] Improve the search for projects
+- https://github.com/eclipse-sirius/sirius-web/issues/3902[#3902] [core] Move the Help menu entry at the last position (after any contributed entries)
 
 == v2024.7.0
 

diff --git a/packages/sirius-web/frontend/sirius-web-application/src/navigationBar/NavigationBarMenu.tsx b/packages/sirius-web/frontend/sirius-web-application/src/navigationBar/NavigationBarMenu.tsx
@@ -49,15 +49,15 @@ export const NavigationBarMenu = ({}: NavigationBarMenuProps) => {
         open={Boolean(state.menuAnchor)}
         anchorEl={state.menuAnchor}
         onClose={handleClose}>
+        {menuItemComponentExtensions.map(({ Component: NavigationBarMenuItem }, index) => (
+          <NavigationBarMenuItem key={index} />
+        ))}
         <MenuItem component="a" href={url} target="_blank" rel="noopener noreferrer">
           <ListItemIcon>
             <HelpIcon />
           </ListItemIcon>
           <ListItemText primary="Help" />
         </MenuItem>
-        {menuItemComponentExtensions.map(({ Component: NavigationBarMenuItem }, index) => (
-          <NavigationBarMenuItem key={index} />
-        ))}
       </Menu>
     </>
   );