Settings Dialogue Graphical Bug
This commit is contained in:
parent
c70c052f53
commit
b8b4312e37
1 changed files with 16 additions and 5 deletions
|
@ -54,7 +54,13 @@ public class SettingsDialogue extends JScrollPane
|
||||||
if(!(child instanceof JMenuItem))
|
if(!(child instanceof JMenuItem))
|
||||||
continue;
|
continue;
|
||||||
|
|
||||||
options.add((JMenuItem) child);
|
JMenuItem menuItem = (JMenuItem) child;
|
||||||
|
|
||||||
|
options.add(menuItem);
|
||||||
|
|
||||||
|
//force unselect after a selection has been made
|
||||||
|
//this fixes a graphical bug from forcing menu items on non-menus
|
||||||
|
menuItem.addActionListener(e -> unselectAll());
|
||||||
}
|
}
|
||||||
|
|
||||||
this.options.addAll(options);
|
this.options.addAll(options);
|
||||||
|
@ -64,11 +70,9 @@ public class SettingsDialogue extends JScrollPane
|
||||||
components.add(this);
|
components.add(this);
|
||||||
}
|
}
|
||||||
|
|
||||||
private void buildPanel()
|
public void unselectAll()
|
||||||
{
|
{
|
||||||
display.setLayout(new BoxLayout(display, BoxLayout.Y_AXIS));
|
options.forEach(jMenuItem -> jMenuItem.setArmed(false));
|
||||||
for(JMenuItem menuItem : options)
|
|
||||||
display.add(menuItem);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
public void showDialogue()
|
public void showDialogue()
|
||||||
|
@ -76,6 +80,13 @@ public class SettingsDialogue extends JScrollPane
|
||||||
BetterJOptionPane.showJPanelDialogue(null, this, 460, dialogues::add);
|
BetterJOptionPane.showJPanelDialogue(null, this, 460, dialogues::add);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
private void buildPanel()
|
||||||
|
{
|
||||||
|
display.setLayout(new BoxLayout(display, BoxLayout.Y_AXIS));
|
||||||
|
for(JMenuItem menuItem : options)
|
||||||
|
display.add(menuItem);
|
||||||
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public String getName()
|
public String getName()
|
||||||
{
|
{
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue