Skip to content

Conversation

@ceyhun
Copy link
Contributor

@ceyhun ceyhun commented Sep 29, 2020

Fixes #14674

To test:

  1. Start the app from Xcode
  2. Go through one of the cases to open page editor
  • My Site -> Select site -> Floating + button (bottom right) -> Site page
  • My Site -> Select site -> Pages -> + (top right)
  • My Site -> Select site -> Pages -> Select any page to edit
  • My Site -> Select site -> Pages -> Search -> Select any page to edit
  • Press home button -> Pull down to search in Spotlight -> search the name of a draft page -> select to open the app and see the editor
  1. Background app using the Home button
  2. Stop the debugger in Xcode ⚠️ Do not use the app switcher to force quit it during debugging. The system automatically deletes its preserved state when the user force quits the app. See here to learn more.
  3. Launch the app again using Xcode (Now UIKit initiates the state restoration process)
  4. Expect page editor to be shown from where it left off

Deep link test:

  1. Need to change build configuration to Release-Internal for this to work (Changing only the scheme to WordPress Internal won't work, need to "Edit Scheme..." and modify the "Build Configuration" as well)
  2. Start the app from Xcode
  3. Background app using the Home button
  4. Run in terminal xcrun simctl openurl booted "https://apps.wordpress.com/get#/page"
  5. App should open with the editor to create a new page
  6. Background app using the Home button
  7. Stop the debugger in Xcode
  8. Launch the app again
  9. Expect page editor to be shown from where it left off

PR submission checklist:

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

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Sep 29, 2020

You can test the changes on this Pull Request by downloading it from AppCenter here with build number: 35604. IPA is available here. If you need access to this, you can ask a maintainer to add you.

@peril-wordpress-mobile
Copy link

You can trigger optional UI/connected tests for these changes by visiting CircleCI here.

Copy link
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.

Nice job tracking down this fix @ceyhun!

The flows look good to me. I also tested a few cases related to Modal Layout Picker:

To disable or enable the development version of Modal Layout Picker
  • Open the app from the build that allows FeatureFlags such as a PR build or a local development build
    • By default, the modal layout picker will be disabled.
    • From the site page:
      • Click on your Gravatar.
      • Click on App Settings.
      • Click on Debug.
      • Toggle "Gutenberg Modal Layout Picker" to enable or disable the picker
  1. Turn on the Modal Layout Picker feature
  2. Open the editor by:
    • Select FAB icon -> Site Page -> Create Blank Page
    • Select FAB icon -> Site Page -> Select a layout -> Create Page
    • Select FAB icon -> Site Page -> Create Blank Page
    • Select FAB icon -> Site Page -> choose a layout -> Create Page
    • Select FAB icon -> Site Page -> choose a layout -> Preview -> Create Page
    • Select Pages -> Select the ➕ -> Create Blank Page
    • Select Pages -> Select the ➕-> Select a layout -> Create Page
    • Select Pages -> Select the ➕ -> Create Blank Page
    • Select Pages -> Select the ➕ -> choose a layout -> Create Page
    • Select Pages -> Select the ➕ -> choose a layout -> Preview -> Create Page
  3. Follow the reset of the steps from the above starting at step 3

@ceyhun
Copy link
Contributor Author

ceyhun commented Sep 30, 2020

Thanks for testing the extra cases @chipsnyder 🙇‍♂️

@ceyhun ceyhun merged commit 6a6c886 into develop Sep 30, 2020
@ceyhun ceyhun deleted the issue/14674-invisible-editor-overlay branch September 30, 2020 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Editor: Leaving editor open in background for a long time turns it into an invisible overlay

3 participants