-
Notifications
You must be signed in to change notification settings - Fork 286
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Princess configuration #3041
Princess configuration #3041
Conversation
Conflicts: megamek/i18n/megamek/client/messages_en.properties megamek/mmconf/princessBehaviors.xml megamek/src/megamek/client/ui/swing/lobby/ChatLounge.java megamek/src/megamek/common/Board.java megamek/src/megamek/common/IBoard.java
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two minor nitpicks, looks pretty good otherwise.
coordsList.setCellRenderer(new BuildingHexRenderer()); | ||
coordsList.setFont(UIUtil.getScaledFont()); | ||
coordsList.setVisibleRowCount(6); | ||
coordsList.setPrototypeCellValue(new Coords(-21, 22)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems a little specific.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you mean the prototype cell value, that's only passed to the renderer to get the cell size when the list is empty. This prevents it from changing size when it goes from empty to containing items (the changed size doesn't appear directly, only when resizing the dialog, but then it looks a little silly)
private static JMenu prioTargetMenu(ClientGUI cg, boolean enabled, ActionListener listener, | ||
Collection<Entity> entities) { | ||
|
||
JMenu menu = new JMenu("Set Prio Target for"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm sure that we have enough bits that we can spell "Priority" fully.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, done
Just a comment from the forums that might be applicable here. https://bg.battletech.com/forums/megamek-games/princess-settings-popup-tooltip-vs-help/ |
Resolves #2986
Resolves #2958
Resolves #2853