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

Floating and standalone plugins respect minimum dimensions #135

Merged
merged 1 commit into from
Oct 1, 2020

Conversation

Sarath18
Copy link
Contributor

@Sarath18 Sarath18 commented Oct 1, 2020

  • Set parentItem after resizing the dialog

  • Create minimum height and width card properties

  • Set height and width of card on adding a child plugin

  • Enforce minimum dimensions on floating plugin resize

Signed-off-by: Sarathkrishnan Ramesh [email protected]

* Set parentItem after resizing the dialog

* Create minimum height and width card properties

* Set height and width of card on adding a child plugin

* Enforce minimum dimensions on floating plugin resize

Signed-off-by: Sarathkrishnan Ramesh <[email protected]>
@Sarath18
Copy link
Contributor Author

Sarath18 commented Oct 1, 2020

Instead of setting height and width on Component.onCompleted I set the height and width of the card pane when the child plugin is added. This way we don't require to change any of the plugins.

@chapulina chapulina self-assigned this Oct 1, 2020
@chapulina chapulina added the 🏰 citadel Ignition Citadel label Oct 1, 2020
Copy link
Contributor

@chapulina chapulina left a comment

Choose a reason for hiding this comment

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

Great improvements, thanks for the fix!

@chapulina chapulina merged commit 7913b7d into gazebosim:ign-gui3 Oct 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏰 citadel Ignition Citadel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants