Skip to content

Commit 0947e3d

Browse files
committed
Unused custom menus are not showed
1 parent 6bfe877 commit 0947e3d

File tree

1 file changed

+7
-9
lines changed

1 file changed

+7
-9
lines changed

app/src/processing/app/Base.java

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1229,15 +1229,13 @@ public void rebuildBoardsMenu(JMenu toolsMenu, Editor editor) throws Exception {
12291229
Map<String, ButtonGroup> buttonGroupsMap = new HashMap<String, ButtonGroup>();
12301230

12311231
// Generate custom menus for all platforms
1232+
Set<String> titles = new HashSet<String>();
12321233
for (TargetPackage targetPackage : packages.values()) {
1233-
for (TargetPlatform targetPlatform : targetPackage.platforms()) {
1234-
PreferencesMap customMenus = targetPlatform.getCustomMenus();
1235-
for (String menuId : customMenus.keySet()) {
1236-
String title = customMenus.get(menuId);
1237-
makeBoardCustomMenu(toolsMenu, _(title));
1238-
}
1239-
}
1234+
for (TargetPlatform targetPlatform : targetPackage.platforms())
1235+
titles.addAll(targetPlatform.getCustomMenus().values());
12401236
}
1237+
for (String title : titles)
1238+
makeBoardCustomMenu(toolsMenu, _(title));
12411239

12421240
// Cycle through all packages
12431241
for (TargetPackage targetPackage : packages.values()) {
@@ -1399,9 +1397,9 @@ private static void filterVisibilityOfSubsequentBoardMenus(String boardID, int f
13991397
JMenuItem menuItem = menu.getItem(m);
14001398
menuItem.setVisible(menuItem.getAction().getValue("board").equals(boardID));
14011399
}
1402-
menu.setEnabled(ifThereAreVisibleItemsOn(menu));
1400+
menu.setVisible(ifThereAreVisibleItemsOn(menu));
14031401

1404-
if (menu.isEnabled()) {
1402+
if (menu.isVisible()) {
14051403
JMenuItem visibleSelectedOrFirstMenuItem = selectVisibleSelectedOrFirstMenuItem(menu);
14061404
if (!visibleSelectedOrFirstMenuItem.isSelected()) {
14071405
visibleSelectedOrFirstMenuItem.setSelected(true);

0 commit comments

Comments
 (0)