Skip to content

Layout Template Modal & Button block - Orientation fixes#2086

Merged
geriux merged 6 commits intodevelopfrom
fix/layout-preview-modal
Apr 7, 2020
Merged

Layout Template Modal & Button block - Orientation fixes#2086
geriux merged 6 commits intodevelopfrom
fix/layout-preview-modal

Conversation

@geriux
Copy link
Copy Markdown
Contributor

@geriux geriux commented Mar 31, 2020

Fixes #2064

Gutenberg PR -> WordPress/gutenberg#21290

To test check Gutenberg PR description

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@geriux
Copy link
Copy Markdown
Contributor Author

geriux commented Mar 31, 2020

Hey there @lukewalczak @chipsnyder 👋

So we have a crash regarding to modals on Android. I explain more in the Gutenberg PR description.

@lukewalczak I made a change in the Button block so I added you as a reviewer just to double check all is good.

@chipsnyder would you be able to test as well? not sure how is your PR review queue if not I'll assign someone else 😄

@chipsnyder
Copy link
Copy Markdown
Contributor

@chipsnyder would you be able to test as well? not sure how is your PR review queue if not I'll assign someone else 😄

No problem I can take a look at this tomorrow

Copy link
Copy Markdown
Contributor

@chipsnyder chipsnyder left a comment

Choose a reason for hiding this comment

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

@geriux geriux merged commit 230319c into develop Apr 7, 2020
@geriux geriux deleted the fix/layout-preview-modal branch April 7, 2020 07:41
@pinarol pinarol added this to the 1.26 milestone Apr 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[GlobalStep] Android - Rotating the device while Previewing a Site Page’s Template causes a crash.

3 participants