Skip to content
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

Update UI to use BorderLayout, fixes resizeable issues #16

Closed
wants to merge 2 commits into from
Closed

Update UI to use BorderLayout, fixes resizeable issues #16

wants to merge 2 commits into from

Conversation

KronosDesign
Copy link
Contributor

No description provided.

private ClientPanel panel;
private NavigationPanel navigationPanel;
private PluginPanel pluginPanel;
private boolean expanded;
private int collapsedWidth = ClientPanel.PANEL_WIDTH + NavigationPanel.PANEL_WIDTH;
private int expandedWidth = collapsedWidth + PluginPanel.PANEL_WIDTH;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These two fields seem constant, should be static final

@Adam-
Copy link
Member

Adam- commented Apr 12, 2017

This seems to move the nav bar to be on the left side of the plugin area instead of the right side, which I don't want.

@RobinWeymans
Copy link
Contributor

And do you want the ClientPanel to change size when opening and closing the PluginPanel?

@KronosDesign
Copy link
Contributor Author

KronosDesign commented Apr 12, 2017

@RobinWeymans I will eventually revisit the entire UI but for now it works well enough.
Ideally, i'd like to create a slide-out style menu that would sit on top of the ClientPanel.
Should also stop the current screen jittering that occurs on resize.

@Adam-
Copy link
Member

Adam- commented Apr 12, 2017

Had to modify this to slightly to make it not resize the height down smaller than the panel height (I don't know why) when closing the side panel... Otherwise seems to work, merged.

@Adam- Adam- closed this Apr 12, 2017
@Adam- Adam- mentioned this pull request Apr 12, 2017
FrostyFridge added a commit to FrostyFridge/runelite that referenced this pull request Mar 16, 2019
Update to new RL api for new game version
Lucwousin referenced this pull request in Lucwousin/runelite Apr 23, 2019
Jacoblairm pushed a commit to Jacoblairm/runelite that referenced this pull request Jun 2, 2019
UKMonster pushed a commit to UKMonster/microbot that referenced this pull request Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants