From 048c44cde2f82634e492d1b07c4662da82b77d64 Mon Sep 17 00:00:00 2001 From: jmillman Date: Thu, 11 Jun 2015 14:09:05 -0400 Subject: [PATCH] rebuild the list of tagnames each time the list is shown --- .../autopsy/imagegallery/gui/GroupPane.java | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/ImageGallery/src/org/sleuthkit/autopsy/imagegallery/gui/GroupPane.java b/ImageGallery/src/org/sleuthkit/autopsy/imagegallery/gui/GroupPane.java index 5e7acd875a..c9220b174f 100644 --- a/ImageGallery/src/org/sleuthkit/autopsy/imagegallery/gui/GroupPane.java +++ b/ImageGallery/src/org/sleuthkit/autopsy/imagegallery/gui/GroupPane.java @@ -347,11 +347,7 @@ public class GroupPane extends BorderPane implements GroupView { HBox.setHgrow(spacer, Priority.ALWAYS); spacer.setMinWidth(Region.USE_PREF_SIZE); - ArrayList grpTagMenues = new ArrayList<>(); - for (final TagName tn : TagUtils.getNonCategoryTagNames()) { - MenuItem menuItem = createGrpTagMenuItem(tn); - grpTagMenues.add(menuItem); - } + try { grpTagSplitMenu.setText(TagUtils.getFollowUpTagName().getDisplayName()); grpTagSplitMenu.setOnAction(createGrpTagMenuItem(TagUtils.getFollowUpTagName()).getOnAction()); @@ -359,7 +355,16 @@ public class GroupPane extends BorderPane implements GroupView { LOGGER.log(Level.WARNING, "failed to load FollowUpTagName", tskCoreException); } grpTagSplitMenu.setGraphic(new ImageView(DrawableAttribute.TAGS.getIcon())); - grpTagSplitMenu.getItems().setAll(grpTagMenues); + grpTagSplitMenu.showingProperty().addListener((ObservableValue ov, Boolean t, Boolean t1) -> { + if (t1) { + ArrayList selTagMenues = new ArrayList<>(); + for (final TagName tn : TagUtils.getNonCategoryTagNames()) { + MenuItem menuItem = TagUtils.createSelTagMenuItem(tn, grpTagSplitMenu); + selTagMenues.add(menuItem); + } + grpTagSplitMenu.getItems().setAll(selTagMenues); + } + }); ArrayList grpCategoryMenues = new ArrayList<>(); for (final Category cat : Category.values()) {