Merge pull request #1068 from Owain94/fix-918

equipmentinspectorplugin: Remove the navigation button when disabling the plugin
This commit is contained in:
Ganom
2019-07-20 18:14:37 -04:00
committed by GitHub

View File

@@ -137,7 +137,6 @@ public class EquipmentInspectorPlugin extends Plugin
.panel(equipmentInspectorPanel) .panel(equipmentInspectorPanel)
.build(); .build();
pluginToolbar.addNavigation(navButton); pluginToolbar.addNavigation(navButton);
} }
@@ -148,6 +147,7 @@ public class EquipmentInspectorPlugin extends Plugin
eventBus.unregister(this); eventBus.unregister(this);
menuManager.removePlayerMenuItem(INSPECT_EQUIPMENT); menuManager.removePlayerMenuItem(INSPECT_EQUIPMENT);
pluginToolbar.removeNavigation(navButton);
} }
private void addSubscriptions() private void addSubscriptions()
@@ -160,8 +160,6 @@ public class EquipmentInspectorPlugin extends Plugin
{ {
if (event.getMenuOption().equals(INSPECT_EQUIPMENT)) if (event.getMenuOption().equals(INSPECT_EQUIPMENT))
{ {
executor.execute(() -> executor.execute(() ->
{ {
try try