Skip to content

Release 1.46.0#3078

Merged
fluiddot merged 42 commits intotrunkfrom
release/1.46.0
Feb 5, 2021
Merged

Release 1.46.0#3078
fluiddot merged 42 commits intotrunkfrom
release/1.46.0

Conversation

@fluiddot
Copy link
Copy Markdown
Contributor

@fluiddot fluiddot commented Feb 1, 2021

Release for Gutenberg Mobile 1.46.0

Related PRs

Extra PRs that Landed After the Release Was Cut

Changes

Test plan

  • Use the main WP apps to test the changes above.
  • Smoke test the main WP apps for general writing flow.

Release Submission Checklist

  • Approve and run optional Android and iOS UI tests
  • Check if RELEASE-NOTES.txt and gutenberg/packages/react-native-editor/CHANGELOG.md are updated with all the changes that made it to the release.
  • Bundle package of the release is updated.

fluiddot and others added 30 commits January 19, 2021 13:36
…blocks

Reusable blocks - Present block content (preview mode)
Bumps [jetpack](https://github.com/Automattic/jetpack) from `6f07c39` to `60f41b6`.
- [Release notes](https://github.com/Automattic/jetpack/releases)
- [Commits](Automattic/jetpack@6f07c39...60f41b6)

Signed-off-by: dependabot[bot] <support@github.com>
…o_develop

Merge release 1.45.0 to develop
…jetpack-60f41b6

Bump jetpack from `6f07c39` to `60f41b6`
Add support for `"multiple": false` block types
…eusable-blocks

Disable Unsupported block editor for Reusable blocks
@fluiddot
Copy link
Copy Markdown
Contributor Author

fluiddot commented Feb 2, 2021

Android: Writing Flow checklist 🟡

Tested on Samsung Galaxy S20 FE 5G, Android 10

General

  • TC001 - Block insertion flow
  • TC001 - Paste formatted text copied from website
  • TC001 - Multiline components 🟡
    • Text flickering: When a line is deleted in any component the text jumps for a frame.
    • Quote block splits it on new line: Tapping enter on an empty line splits the block, I'm not sure if this is expected but the rest of multiline blocks don't behave this way.

Rich Text Format

  • TC001 - Bold, Italic, strikethrough buttons
  • TC002 - Alignment buttons
  • TC003 - Alignment Split
  • TC004 - Link button works without selection
  • TC005 - Link button works with a selected word
  • TC006 - Adding a link from a copied URL
  • TC007 - Test format detection under the cursor
  • TC008 - Test formatting doesn't remove leading or trailing whitespace
Splitting and merging
  • TC001 - Merge after writing
  • TC002 - Merge after selection
  • TC003 - Merge after deleting text
  • TC004 - Merge after deleting all
  • TC005 - Merge multiple blocks
  • TC006 - Splitting/merge list block

Undo / Redo - Test Cases

  • TC001 - Undo/redo block actions
  • TC002 - Undo/redo text
  • TC003 - Undo/redo text format

@ceyhun
Copy link
Copy Markdown
Contributor

ceyhun commented Feb 2, 2021

iOS Writing Flow checklist 🟡

Tested on iPhone 11 Pro, iOS 14.3

General

  • TC001 - Paste formatted text copied from website.
    Note: The image behind "Join us!" was not copied over but this might be the current case because the How We Work page was recently updated. I tried copying https://wordpress.com/blog/2021/01/25/building-single-page-websites-on-wordpress-com/ instead and images were also pasted.
  • TC001 - Multiline components 🟡
    Quote Block: Split by pressing enter to create a new line at the end of citation and merge back pressing backspace -> Cursor jumps back to quote instead of citation.
    Pullquote Block: Split by pressing enter to create a new line at the end of citation and merge back pressing backspace -> Pullquote block is selected instead of cursor moving back to citation.

Rich Text Format

  • TC001 - Bold, Italic, strikethrough buttons
  • TC002 - Alignment buttons
  • TC003 - Alignment Split
  • TC004 - Link button works without selection
  • TC005 - Link button works with a selected word
  • TC006 - Adding a link from a copied URL
  • TC007 - Test format detection under the cursor
  • TC008 - Test formatting doesn't remove leading or trailing whitespace
Splitting and merging
  • TC001 - Merge after writing
  • TC002 - Merge after selection
  • TC003 - Merge after deleting text
  • TC004 - Merge after deleting all
  • TC005 - Merge multiple blocks
  • TC006 - Splitting/merge list block

Undo / Redo - Test Cases

  • TC001 - Undo/redo block actions
  • TC002 - Undo/redo text
  • TC003 - Undo/redo text format

@fluiddot
Copy link
Copy Markdown
Contributor Author

fluiddot commented Feb 2, 2021

Note: The image behind "Join us!" was not copied over but this might be the current case because the How We Work page was recently updated. I tried copying https://wordpress.com/blog/2021/01/25/building-single-page-websites-on-wordpress-com/ instead and images were also pasted.

I also noticed this issue but I think it's not copying it because the image is a background image.

@fluiddot
Copy link
Copy Markdown
Contributor Author

fluiddot commented Feb 2, 2021

Quote Block: Split by pressing enter to create a new line at the end of citation and merge back pressing backspace -> Cursor jumps back to quote instead of citation.
Pullquote Block: Split by pressing enter to create a new line at the end of citation and merge back pressing backspace -> Pullquote block is selected instead of cursor moving back to citation.

I also reproduced this issue on Android but I didn't report it because there's already an issue that is linked in the test cases. If it's happening on iOS too we could update it including the new case.

@ceyhun
Copy link
Copy Markdown
Contributor

ceyhun commented Feb 2, 2021

I also reproduced this issue on Android but I didn't report it because there's already an issue that is linked in the test cases. If it's happening on iOS too we could update it including the new case.

I think it's not the same issue. This happens when merging a paragraph block into quote block. I just tested on WPiOS 16.6.20210129 and 16.5.20210129 and it's also present there, so it's not a regression in this version. I'll create separate issues for these.

@fluiddot
Copy link
Copy Markdown
Contributor Author

fluiddot commented Feb 2, 2021

I also reproduced this issue on Android but I didn't report it because there's already an issue that is linked in the test cases. If it's happening on iOS too we could update it including the new case.

I think it's not the same issue. This happens when merging a paragraph block into quote block. I just tested on WPiOS 16.6.20210129 and 16.5.20210129 and it's also present there, so it's not a regression in this version. I'll create separate issues for these.

Oh sorry, I misunderstood the steps, right, it's not the same issue. By the way, I've just checked this on Android and it's happening there too.

@ceyhun
Copy link
Copy Markdown
Contributor

ceyhun commented Feb 2, 2021

Quote issue is same on the web, it might be an expected behavior. But not the Pullquote one, I've created an issue for this: #3089

@chipsnyder
Copy link
Copy Markdown
Contributor

chipsnyder commented Feb 2, 2021

Hey 👋 I'll be starting the Sanity-Tests on iOS. For each of these I'll be testing on an iPhone 11 running 13.5.1 and testing against build 41359 from: wordpress-mobile/WordPress-iOS#15752 (comment)

Gallery - 1

  • Gallery block - Close/Re-open post with an ongoing image upload - TC001
  • Gallery block - Close post with an ongoing image upload - TC002

Gallery - 2

  • Gallery block - Insert image from device (failing) - TC006
  • Gallery block - Insert image from device (cancel) - TC009

Gallery - 3

  • Gallery block - Add Gallery Caption - TC003
  • Gallery block - Add Gallery Image Caption - TC04
  • Gallery block - Try adding same images from WP Media library and moving the images around - TC010

Columns - 1

  • Columns - Columns percentage mechanism - column's preview - TC019
  • Columns - Columns percentage mechanism - column's slider input - TC020
  • Columns - Columns percentage mechanism - predefined percentage values are rendered correctly on frontend- TC021
  • Columns percentage mechanism - custom percentage values are rendered correctly on frontend- TC022
  • Columns - Multiple slider in columns parent - TC023
  • Columns - Adding Column with AppenderButton - TC003
  • Columns - Adding and removing Column with BottomSheet Stepper control - TC004

Columns - 2

  • Columns - Max limit of Column number - TC005
  • Columns - Min limit of Column number - TC006
  • Columns - Removing Column with remove button - TC007
  • Columns - Removing the only one left Column with remove button - TC008
  • Columns - Changing alignment on Columns - TC009
  • Columns - Changing alignment on individual Column - TC010
  • Columns - Alignment attribute is properly passed when adding Column - TC011
  • Columns - Deep nesting is possible (iOS only)- TC012

Columns - 3

  • Columns - Check if Column placeholders is visible for the unselected state - TC002
  • Columns - Appearance in vertical device position (vertical layout) - TC013
  • Columns - Appearance in horizontal device position (horizontal layout) - TC014
  • Columns - Mover direction - TC015
  • Columns - Check if Column placeholder is render in nested structure - TC016
  • Columns - Unselected Column placeholder is stretched to content in nested structure- TC017

MediaText - 1

  • Media Text block - Insert image from device (failing) - TC001
  • Media Text block - Close/Re-open post with an ongoing image upload - TC003

MediaText - 2

  • Media Text block - Insert video from device (TC002) - steps
  • Media Text block - Insert image from device (TC002) - steps

MultipleUpload - 1

  • Simultaneous uploads - TC001

Cover - 1

  • Cover - Add image background from WordPress Media library - TC001
  • Cover - Change Minimum height in pixels - TC002
  • Cover - Change Background Opacity - TC009
  • Cover - Image focal point is rendered properly - TC003
  • Cover - Custom color is rendered properly - TC004

Editor Theme - 1

  • Default Colors - Check that default colors still load - TC001
  • Default Gradients - Check that default gradients still load - TC002
  • Custom Colors - Check that custom colors load in the editor - TC003

Social Icons - 1

  • The newly created Social Icons block is added with 4 icons - TC001
  • Active icon gets product color - TC002
  • Only active icons are visible when not selected - TC003

@enejb
Copy link
Copy Markdown
Contributor

enejb commented Feb 2, 2021

Hey 👋 I'll be starting the Sanity-Tests on Android. For each of these I'll be testing on an Samsung Galaxy A8 running Android 9 and testing against build from: wordpress-mobile/WordPress-Android#13927

DarkMode-1

  • Gallery block - Dark mode - steps
  • Shortcode block - Dark mode - steps
  • Media Text block - Dark mode - steps
    Note: The vertical aligmnet options are not inline any more.

Group-1

  • Group - AppenderButton is rendered - steps
  • Group - Deep nesting is possible (iOS only) - steps Didn't test.
  • Group - Check if Group placeholder is visible for the unselected state - steps
  • Group - Check if Group placeholder is render in nested structure - steps
  • Group - Nested block have proper border styling - steps
  • Group - Nested block have proper margins values - steps
  • Group - Breadcrumbs on FloatingToolbar is properly displayed - steps

Spacer-1

  • Spacer block - Spacer is rendered without crash - steps
  • Spacer block - Settings: Control spacer height using the slider - steps
  • Spacer block - Settings: Control spacer height using the text input - steps

Buttons-1

  • Buttons block - Buttons block is added with focused Button with gray background color and white text color - steps

  • Buttons block - Buttons inline appender - steps

  • ???? Buttons block - Add another Button inside Buttons using inserter - steps
    I think this might be working as described. Since I tested this on the column block. But it didn't work quite as expected for me. When I have the "Buttons" (not singular) selected the inverter shows me all the blocks. but if I have a single button selected the inserted shows me just the ability to insert a button block.

  • Buttons block - Button's wrapper grows properly - steps

Buttons-2

  • Buttons block - Removing Button along with Buttons block - steps
  • Buttons block - Removing exactly one Button (when Buttons contain more of them) - steps

Buttons-3

  • Buttons block - Wrapping Buttons - steps
  • Buttons block - Buttons alignment options - steps

Button-6

  • Buttons block - Settings: Open in new tab - steps
  • Buttons block - Settings: Link rel - steps
  • Buttons block - Settings: Link URL - steps
  • Buttons block - Settings: Remove link - steps
  • Buttons block - Settings: Synchronize with button options - steps

Color Settings - 1

  • 👎 Colors Segmented Controls - steps
    I am able to crash the app by switching between the gradient and solid colour really quickly. Eventually the app crashed and I am not able to continue. I have to restart the app.

  • Scroll color palette to start - steps

  • 👎 Scroll color palette to the end - steps
    It doesn't scroll to the end for me

Image - 1

  • Image block - Insert image from device (failing) - TC001
  • Image block - Insert image from device (cancel) - TC002
  • Image block - Add Caption - TC003
  • Image block - Close/Re-open post with an ongoing image upload - TC004
  • Image block - Close post with an ongoing image upload - TC005

Story block - 1

  • Story block - Verify is available in Block Picker - steps
  • Story block - Verify is not available in Block Picker- steps
  • Story block - Add media to empty Story block - steps
  • Story block - Add media to non-empty Story block - steps
  • Story block - Close/Re-open post with an ongoing Story upload - steps

@fluiddot
Copy link
Copy Markdown
Contributor Author

fluiddot commented Feb 3, 2021

After the first round of manual tests we found so far the following issues:

iOS & Android

Quote block splits on new line ✅

Description

Tapping enter on an empty line splits the block.

Next steps

This was checked on the web version and this behaviour is expected so we shouldn't consider this an issue.

Image not copied when pasting formatted text copied from website ✅

Description

The image behind "Join us!" was not copied over but this might be the current case because the How We Work page was recently updated. When copying content from other site like https://wordpress.com/blog/2021/01/25/building-single-page-websites-on-wordpress-com/, the images were also pasted.

Next steps

The image from How We Work page is not copied because it's rendered as a background images so we shouldn't consider this an issue.

Quote Block - Wrong cursor position when merging back ✅

Description

Split by pressing enter to create a new line at the end of citation and merge back pressing backspace -> Cursor jumps back to quote instead of citation.

Next steps

This was checked on the web version and this behaviour is expected so we shouldn't consider this an issue.

Pullquote Block - Wrong cursor position when merging back 🟡

Description

Split by pressing enter to create a new line at the end of citation and merge back pressing backspace -> Pullquote block is selected instead of cursor moving back to citation.

Next steps

This behaviour is not expected, however it's also reproducible on released versions like WPiOS 16.6 and 16.5 and WPAndroid 16.6 so this version is not introducing it.

An issue has been created to address it.

MediaText Block - Vertical alignment options are no longer inlined 🟡

Description

The screenshot of the test case shows the vertical alignment options as inlined, looks like this has been changed in a recent version.

Next steps

If this change was made on purpose we should update the screenshot.

Only iOS

None 🎊 .

Only Android

Text flickering when removing a line on some multiline blocks 🟡

Description

When a line is deleted in some multiline blocks, specifically the Quote, Preformatted and Verse, the text lines position moves some pixels down for a frame and then goes back to the original position.

Next steps

This was checked on latest release candidate version (16.6-rc-2) and this issue is also reproducible there so this version is not introducing it.

An issue has been created to address it.

Colors Segmented Controls crash 🟢 (fixed by @lukewalczak)

Fixed in gutenberg PR 🎊 .

Description

When switching between the gradient and solid colour really quick the app crashes.

Here are the related steps.

Next steps

This was checked on release candidate version (16.6-rc-2) and this issue is also reproducible there so this version is not introducing it.

Since this is producing a crash it would be nice to fix it in this version. An issue has been created to address it as soon as possible.

@fluiddot
Copy link
Copy Markdown
Contributor Author

fluiddot commented Feb 3, 2021

👎 Scroll color palette to the end - steps
It doesn't scroll to the end for me

👋 @enejb I tried to reproduce the issue you commented but so far it's not happening to me, could you try it again?

I did the test on a Samsung Galaxy S20 FE 5G and build 93717.

scroll-color-palette-end-android.mp4

@fluiddot
Copy link
Copy Markdown
Contributor Author

fluiddot commented Feb 4, 2021

Status update

Next step will be to:

@fluiddot
Copy link
Copy Markdown
Contributor Author

fluiddot commented Feb 4, 2021

Android: Writing Flow checklist [2nd round] 🟡

Tested on Samsung Galaxy S20 FE 5G, Android 10 - Build 94309

General

Rich Text Format

  • TC001 - Bold, Italic, strikethrough buttons
  • TC002 - Alignment buttons
  • TC003 - Alignment Split
  • TC004 - Link button works without selection
  • TC005 - Link button works with a selected word
  • TC006 - Adding a link from a copied URL
  • TC007 - Test format detection under the cursor
  • TC008 - Test formatting doesn't remove leading or trailing whitespace
Splitting and merging

Undo / Redo - Test Cases

  • TC001 - Undo/redo block actions
  • TC002 - Undo/redo text
  • TC003 - Undo/redo text format

Result

I found a new issue but it's not related to recent changes, I'm afraid I missed to test the Heading block in the first round for the test case Splitting and merging 😞 .

Extra PRs tested:

@ceyhun
Copy link
Copy Markdown
Contributor

ceyhun commented Feb 4, 2021

iOS Writing Flow checklist [2nd round] 🟡

Tested on iPhone 11 Pro, iOS 14.3, 41557

General

Rich Text Format

  • TC001 - Bold, Italic, strikethrough buttons
  • TC002 - Alignment buttons
  • TC003 - Alignment Split
  • TC004 - Link button works without selection
  • TC005 - Link button works with a selected word
  • TC006 - Adding a link from a copied URL
  • TC007 - Test format detection under the cursor
  • TC008 - Test formatting doesn't remove leading or trailing whitespace
Splitting and merging
  • TC001 - Merge after writing
  • TC002 - Merge after selection
  • TC003 - Merge after deleting text
  • TC004 - Merge after deleting all
  • TC005 - Merge multiple blocks
  • TC006 - Splitting/merge list block

Undo / Redo - Test Cases

  • TC001 - Undo/redo block actions
  • TC002 - Undo/redo text
  • TC003 - Undo/redo text format

Extra PRs tested:

@enejb
Copy link
Copy Markdown
Contributor

enejb commented Feb 4, 2021

👋 @fluiddot. I just tested it again and it looks like both Colour issues (Crash and not advancing to the end) are fixed for me. 🥳

@fluiddot
Copy link
Copy Markdown
Contributor Author

fluiddot commented Feb 5, 2021

Status update

The second round of manual tests is finished, no critical issues have been found and latest changes have been verified 🎊 .

Here are the list of issues found in this release that should be addressed in the future:

Next steps

  • Review and approve the PRs of the release branches.
  • Run the last day steps for merging the release into the WordPress app repositories.

Copy link
Copy Markdown
Contributor

@ceyhun ceyhun left a comment

Choose a reason for hiding this comment

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

Things are looking good :shipit:

@fluiddot fluiddot added this to the 1.46 (16.7) milestone Feb 5, 2021
@fluiddot fluiddot merged commit ef50e91 into trunk Feb 5, 2021
@fluiddot fluiddot deleted the release/1.46.0 branch February 5, 2021 10:41
@fluiddot fluiddot mentioned this pull request Feb 5, 2021
2 tasks
@ceyhun ceyhun restored the release/1.46.0 branch February 11, 2021 15:28
@dcalhoun dcalhoun mentioned this pull request Feb 16, 2021
3 tasks
@fluiddot fluiddot deleted the release/1.46.0 branch February 17, 2021 10:58
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.

9 participants