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

Drastically simplifies ModalViewContainer. #913

Merged
merged 1 commit into from Jan 25, 2020
Merged

Conversation

rjrjr
Copy link
Contributor

@rjrjr rjrjr commented Jan 24, 2020

Trying to protect container authors from calling the Dialog
constructor was foolish. Didn't actually simplify anything,
and made it very difficult to manage window size.

Panel sample behavior around soft keyboard is a lot better now.

@rjrjr rjrjr changed the title Drasticaly simplifies ModalViewContainer. Drastically simplifies ModalViewContainer. Jan 24, 2020
@rjrjr rjrjr force-pushed the ray/better-modals branch 2 times, most recently from 38e5e5a to 5e2de83 Compare January 25, 2020 00:09
@rjrjr
Copy link
Contributor Author

rjrjr commented Jan 25, 2020

Update fixes stale kdoc on PanelContainer.

Trying to protect container authors from calling the `Dialog`
constructor was foolish. Didn't actually simplify anything,
and made it very difficult to manage window size.

Panel sample behavior around soft keyboard is a lot better now.
@rjrjr rjrjr merged commit 3cec6d2 into master Jan 25, 2020
@rjrjr rjrjr deleted the ray/better-modals branch January 25, 2020 02:46
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

2 participants