Skip to content

Conversation

@chrisbreiding
Copy link
Contributor

@chrisbreiding chrisbreiding commented Aug 9, 2019

No longer covers nav and footer when the error details are expanded:

Screen Shot 2019-08-09 at 10 16 52 AM

Bonus: caps the height of the error details and scrolls them instead of the entire error message so 'Try Again' button stays visible (as long as desktop gui height is at a reasonable size):

desktop-gui-error-scrolling

Pre-merge Tasks

  • Have tests been added/updated for the changes in this PR?
  • Has the original issue been tagged with a release in ZenHub?

@chrisbreiding chrisbreiding changed the title fix desktop-gui layout issues when plugins file error Fix desktop-gui layout issues when there's a plugins file error Aug 9, 2019
@cypress
Copy link

cypress bot commented Aug 9, 2019



Test summary

3330 0 47 0


Run details

Project cypress
Status Passed
Commit 826081b
Started Aug 9, 2019 2:26 PM
Ended Aug 9, 2019 2:31 PM
Duration 05:15 💡
OS Linux Debian - 8.10
Browser Multiple

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard


.full-alert {
// `margin: auto` fixes issue with flexbox vertically-centered element + oveflow
// https://stackoverflow.com/questions/33454533/cant-scroll-to-top-of-flex-item-that-is-overflowing-container
Copy link
Member

Choose a reason for hiding this comment

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

lol ❤️ css

Copy link
Member

@jennifer-shehane jennifer-shehane left a comment

Choose a reason for hiding this comment

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

👍

@jennifer-shehane jennifer-shehane merged commit da497e4 into develop Aug 9, 2019
@chrisbreiding chrisbreiding deleted the issue-4959-desktop-layout-issues branch April 5, 2022 18:28
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.

css problems on plugins error

3 participants