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

[Gutenberg] Hide help button from UBE #15674

Merged
merged 7 commits into from
Jan 14, 2022

Conversation

SiobhyB
Copy link
Contributor

@SiobhyB SiobhyB commented Dec 8, 2021

Addresses wordpress-mobile/gutenberg-mobile#4339

Related PRs and Installable Builds

Description

As described in wordpress-mobile/gutenberg-mobile#4339, a help button is currently displaying in the bottom right-hand corner of the unsupported block editor (UBE) in both the Android and iOS apps. The button can be seen while logged into WordPress.com. When tapped on, the button prompts users to navigate away from the editor on Android, creating the potential for edits to be lost.

With this PR, the button is hidden from view in order to avoid confusion and potential loss of edits. A high-level overview of the code changes can be found in the Types of changes section of the Gutenberg PR.

Testing

Please refer to the Gutenberg PR as the "central PR" with the most up-to-date testing instructions.


PR submission checklist:

  • I have completed the Regression Notes.
  • 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 Dec 8, 2021

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

@SiobhyB SiobhyB added [Status] Not Ready for Merge gutenberg-mobile Gutenberg Editing and display of Gutenberg blocks. labels Dec 8, 2021
@SiobhyB SiobhyB changed the title Hide help button from UBE [DO NOT MERGE] Hide help button from UBE Dec 8, 2021
@SiobhyB SiobhyB self-assigned this Dec 8, 2021
@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Dec 8, 2021

You can test the changes on this Pull Request by downloading the APKs:

@SiobhyB SiobhyB changed the title [DO NOT MERGE] Hide help button from UBE [Gutenberg] Hide help button from UBE Dec 9, 2021
Siobhan added 2 commits December 9, 2021 15:57
'getOnGutenbergReadyExternalStyles' will fetch and return external styles defined in Gutenberg Mobile's 'external-style-overrides.css' file.
@SiobhyB SiobhyB marked this pull request as ready for review December 14, 2021 18:36
@SiobhyB SiobhyB requested a review from twstokes December 14, 2021 21:11
Added following feedback here: #15674 (comment)
Copy link
Contributor

@twstokes twstokes left a comment

Choose a reason for hiding this comment

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

LGTM @SiobhyB to merge after pointing to the appropriate tag. 🎉

@peril-wordpress-mobile
Copy link

Warnings
⚠️ PR is not assigned to a milestone.

Generated by 🚫 dangerJS

@SiobhyB SiobhyB added this to the 19.1 milestone Jan 14, 2022
@SiobhyB SiobhyB merged commit 9811e03 into develop Jan 14, 2022
@SiobhyB SiobhyB deleted the gutenberg/update/hide-help-button-from-ube branch January 14, 2022 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Gutenberg Editing and display of Gutenberg blocks.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants