Skip to content
This repository was archived by the owner on Jan 5, 2026. It is now read-only.

#1811 Fix overflowing dialogs so they are scrollable#1859

Merged
tonyanziano merged 7 commits into
masterfrom
corinagum/1811
Sep 11, 2019
Merged

#1811 Fix overflowing dialogs so they are scrollable#1859
tonyanziano merged 7 commits into
masterfrom
corinagum/1811

Conversation

@corinagum
Copy link
Copy Markdown
Contributor

#1811

This fix adds a maximum height to dialogs so that zooming in or overflow will make the dialog scrollable, instead of creating unreachable parts of the modal.

Before scrolling down:
image
After scrolling down:

image

@DesignPolice - just checking in that not having a visible scrollbar on the outside of the dialog is still the desirable appearance.

@coveralls
Copy link
Copy Markdown

coveralls commented Sep 10, 2019

Coverage Status

Coverage remained the same at 65.787% when pulling 5e02a7e on corinagum/1811 into f034cbc on master.

@tonyanziano
Copy link
Copy Markdown
Contributor

Just waiting on a response from @DesignPolice regarding the invisible scrollbar, and then we can merge this in.

@DesignPolice
Copy link
Copy Markdown

DesignPolice commented Sep 11, 2019

Thanks for fixing this @tonyanziano @corinagum - I think we will be updating all of the visuals with Emulator soon, so if this treatment will pass for a few months lets see how it tests - and not spend time doing something we are going to redo. IF it still is an issue I will build a new box with a scroll and smaller X

thanks

===

Edited because @corinagum actually fixed this one 😅 -Tony

Copy link
Copy Markdown
Contributor

@tonyanziano tonyanziano left a comment

Choose a reason for hiding this comment

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

👍

@tonyanziano tonyanziano merged commit c7a8a91 into master Sep 11, 2019
@tonyanziano tonyanziano deleted the corinagum/1811 branch September 11, 2019 19:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants