[RNMobile] Fix drag mode not being enabled when long-pressing over Shortcode block #41155
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
fluiddot
added
[Type] Bug
|
Size Change: +166 B (0%) Total Size: 1.24 MB
ℹ️ View Unchanged
|
geriux
reviewed
May 19, 2022
| } | ||
| } } | ||
| unstableOnFocus={ onFocus } | ||
| __unstableMobileNoFocusOnMount={ true } |
There was a problem hiding this comment.
When adding a Shortcode block, due to this prop the block doesn't autofocus, is this needed to avoid some RichText issue?
There was a problem hiding this comment.
Thanks for pointing it out 🙇 ! Yep, I also noticed this recently when testing, I finally removed the prop 👍 .
|
Using a block-based theme, RichText sets the text color from the theme as a fallback if none is passed down as a prop:
I think we could set the default color here, to avoid this issue, what do you think? |
fluiddot
force-pushed
the
rnmobile/update/shortcode-use-rich-text
branch
from
4ca3475
to
76e58ed
Compare
fluiddot
commented
May 19, 2022
Comment on lines
+34
to
+38
| @objc var disableAutocorrection: Bool = false { | ||
| didSet { | ||
| autocorrectionType = disableAutocorrection ? .no : .default | ||
| } | ||
| } |
There was a problem hiding this comment.
This logic is similar to the RN's TextInput component on iOS (reference).
Comment on lines
+621
to
+629
| @ReactProp(name = "disableAutocorrection", defaultBoolean = false) | ||
| public void disableAutocorrection(final ReactAztecText view, boolean disable) { | ||
| if (disable) { | ||
| view.setInputType((view.getInputType() & ~InputType.TYPE_TEXT_FLAG_AUTO_CORRECT) | InputType.TYPE_TEXT_FLAG_NO_SUGGESTIONS); | ||
| } | ||
| else { | ||
| view.setInputType((view.getInputType() & ~InputType.TYPE_TEXT_FLAG_NO_SUGGESTIONS) | InputType.TYPE_TEXT_FLAG_AUTO_CORRECT); | ||
| } | ||
| } |
There was a problem hiding this comment.
This logic is similar to the RN's TextInput component on Android (reference).
Comment on lines
+1256
to
+1266
| const { | ||
| clientId, | ||
| identifier, | ||
| withoutInteractiveFormatting, | ||
| allowedFormats, | ||
| } = props; | ||
| const { formatTypes } = useFormatTypes( { | ||
| clientId: props.clientId, | ||
| identifier: props.identifier, | ||
| withoutInteractiveFormatting: props.withoutInteractiveFormatting, | ||
| clientId, | ||
| identifier, | ||
| withoutInteractiveFormatting, | ||
| allowedFormats, |
There was a problem hiding this comment.
I found out that the format types weren't taking into account the allowed formats calculated in the RichText component:
gutenberg/packages/block-editor/src/components/rich-text/index.native.js
Lines 194 to 198 in a1e1fdc
This was causing that setting the prop __unstableDisableFormats doesn't hide the text color format button from the toolbar, as well as preventing hiding other format buttons when using the allowedFormats prop.
| } | ||
| } } | ||
| unstableOnFocus={ onFocus } | ||
| __unstableMobileNoFocusOnMount={ true } |
There was a problem hiding this comment.
Thanks for pointing it out 🙇 ! Yep, I also noticed this recently when testing, I finally removed the prop 👍 .
Oh, good catch! Yeah, I think in this case we should set the text color to prevent it to be overridden by the theme colors 👍 . I'll update it as soon as possible. |
This issue has been addressed in af35786.
NOTE: I used the |
geriux
reviewed
May 20, 2022
| { ...forcePlainTextProps } | ||
| { ...fontProps } | ||
| identifier="content" | ||
| tagName="pre" |
There was a problem hiding this comment.
I think this should come from props, currently, it's fine because only the Shortcode block uses it but if we want to use it in other blocks, the tagName would be different, what do you think?
There was a problem hiding this comment.
Yep, good point. In fact, I think that we could even remove the prop to use the default value div (reference). As far as I checked, it works even better than using pre tag 😅 .
| Android | iOS |
|---|---|
ios-shortcode-block.mp4 |
android-shortcode-block.mp4 |
As an additional note, I'd like to share that during implementation, I tried using different tag names like p or span but found some issues:
p: On Android, it doesn't allow to add new lines at the end of the content.
gutenberg/packages/rich-text/src/component/index.native.js
Lines 1095 to 1097 in 51db4bf
span: Automatically wraps thespantags withptags. Besides, adding new lines leads to odd issues where lines are merged or add extra newlines 🙃.
There was a problem hiding this comment.
Sounds good! At first I was wondering how it was showing the monospace font without the pre tag but I checked we are setting the font from the styles so we are good 👍
|
This is working well! Really nice approach to start using RichText in the PlainText component 🤩 I only left one comment, let me know what you think. |
Thanks for the review @geriux ! I've just addressed the comment in the latest commit 0433a92. As described here, I decided to remove the |
geriux
approved these changes
May 20, 2022
geriux
pushed a commit
that referenced
this pull request
May 20, 2022
…ortcode block (#41155) * Add prop for disabling suggestions button * Use allowed formats in format types calculation * Add RichText version to PlainText component * Use experimental version of PlainText in Shortcode block * Add disableAutocorrection prop to RichText * Disable autocorrection in Shortcode block * Update PlainText props in Shortcode block * Use pre as tagName in PlainText * Rename replaceLineBreaks function * Update shortcode block unit tests * Prevent text input focus when selecting Shortcode block * Force text color in Shortcode block * Remove tagName prop from PlainText component
4 tasks
geriux
pushed a commit
that referenced
this pull request
May 23, 2022
* Release script: Update react-native-editor version to 1.76.0 * Release script: Update with changes from 'npm run core preios' * Update Changelog * Release script: Update react-native-editor version to 1.76.1 * Release script: Update with changes from 'npm run core preios' * [Mobile] - BlockList - Add internal onLayout from CellRendererComponent to BlockListItemCell (#41105) * Mobile - BlockList - Pass FlatList internal onLayout through CellRendererComponent * Mobile - Update onLayout naming for BlockListItemCell * [Mobile] - Fix Drag & Drop Chip positioning issue with RTL languages (#41053) * Mobile - Fix - Drag & drop chip issue with RTL languages * Mobile - BlockDraggable - Rename maxWidth variable to contentWidth * Mobile - BlockDraggable - Add custom exiting animation that uses the same functionality as ZoomOutEasyDown but customizing the translateX value taking into account RTL languages. * [RNMobile] Add drag & drop help guide in Help & Support screen (#40961) * Add drag & drop help guide * Update content of help screen related to moving blocks * Update drand-and-drop images * Update styles of heading component of help screen * Add HelpDetailBadge component This component will be rendered in `HelpDetailSectionHeadingText` component via the `badge` prop. * Add NEW badge to move blocks help section * Optimize drag-and-drop images * Add move-blocks icon to Help & Support screen * Update react-native-editor changelog * Add HelpSectionTitle component * Prevent rendering separator on last help topic item * [RNMobile] Fix drag mode not being enabled when long-pressing over Shortcode block (#41155) * Add prop for disabling suggestions button * Use allowed formats in format types calculation * Add RichText version to PlainText component * Use experimental version of PlainText in Shortcode block * Add disableAutocorrection prop to RichText * Disable autocorrection in Shortcode block * Update PlainText props in Shortcode block * Use pre as tagName in PlainText * Rename replaceLineBreaks function * Update shortcode block unit tests * Prevent text input focus when selecting Shortcode block * Force text color in Shortcode block * Remove tagName prop from PlainText component * Mobile - Update changelog * Translate NEW badge in Move blocks help screen * Revert cocoapods version update Co-authored-by: jhnstn <[email protected]> Co-authored-by: Carlos Garcia <[email protected]>
fluiddot
added a commit
that referenced
this pull request
May 27, 2022
* Release script: Update react-native-editor version to 1.76.0 * Release script: Update with changes from 'npm run core preios' * Update Changelog * Release script: Update react-native-editor version to 1.76.1 * Release script: Update with changes from 'npm run core preios' * [Mobile] - BlockList - Add internal onLayout from CellRendererComponent to BlockListItemCell (#41105) * Mobile - BlockList - Pass FlatList internal onLayout through CellRendererComponent * Mobile - Update onLayout naming for BlockListItemCell * [Mobile] - Fix Drag & Drop Chip positioning issue with RTL languages (#41053) * Mobile - Fix - Drag & drop chip issue with RTL languages * Mobile - BlockDraggable - Rename maxWidth variable to contentWidth * Mobile - BlockDraggable - Add custom exiting animation that uses the same functionality as ZoomOutEasyDown but customizing the translateX value taking into account RTL languages. * [RNMobile] Add drag & drop help guide in Help & Support screen (#40961) * Add drag & drop help guide * Update content of help screen related to moving blocks * Update drand-and-drop images * Update styles of heading component of help screen * Add HelpDetailBadge component This component will be rendered in `HelpDetailSectionHeadingText` component via the `badge` prop. * Add NEW badge to move blocks help section * Optimize drag-and-drop images * Add move-blocks icon to Help & Support screen * Update react-native-editor changelog * Add HelpSectionTitle component * Prevent rendering separator on last help topic item * [RNMobile] Fix drag mode not being enabled when long-pressing over Shortcode block (#41155) * Add prop for disabling suggestions button * Use allowed formats in format types calculation * Add RichText version to PlainText component * Use experimental version of PlainText in Shortcode block * Add disableAutocorrection prop to RichText * Disable autocorrection in Shortcode block * Update PlainText props in Shortcode block * Use pre as tagName in PlainText * Rename replaceLineBreaks function * Update shortcode block unit tests * Prevent text input focus when selecting Shortcode block * Force text color in Shortcode block * Remove tagName prop from PlainText component * Mobile - Update changelog * Translate NEW badge in Move blocks help screen * Release script: Update react-native-editor version to 1.76.2 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Ensure post title gets focused when is notified from native side (#41371) * Update react-native-editor changelog * Update Podfile.lock Co-authored-by: jhnstn <[email protected]> Co-authored-by: Gerardo <[email protected]>
geriux
pushed a commit
that referenced
this pull request
Jun 2, 2022
* Release script: Update react-native-editor version to 1.76.0 * Release script: Update with changes from 'npm run core preios' * Update Changelog * Release script: Update react-native-editor version to 1.76.1 * Release script: Update with changes from 'npm run core preios' * [Mobile] - BlockList - Add internal onLayout from CellRendererComponent to BlockListItemCell (#41105) * Mobile - BlockList - Pass FlatList internal onLayout through CellRendererComponent * Mobile - Update onLayout naming for BlockListItemCell * [Mobile] - Fix Drag & Drop Chip positioning issue with RTL languages (#41053) * Mobile - Fix - Drag & drop chip issue with RTL languages * Mobile - BlockDraggable - Rename maxWidth variable to contentWidth * Mobile - BlockDraggable - Add custom exiting animation that uses the same functionality as ZoomOutEasyDown but customizing the translateX value taking into account RTL languages. * [RNMobile] Add drag & drop help guide in Help & Support screen (#40961) * Add drag & drop help guide * Update content of help screen related to moving blocks * Update drand-and-drop images * Update styles of heading component of help screen * Add HelpDetailBadge component This component will be rendered in `HelpDetailSectionHeadingText` component via the `badge` prop. * Add NEW badge to move blocks help section * Optimize drag-and-drop images * Add move-blocks icon to Help & Support screen * Update react-native-editor changelog * Add HelpSectionTitle component * Prevent rendering separator on last help topic item * [RNMobile] Fix drag mode not being enabled when long-pressing over Shortcode block (#41155) * Add prop for disabling suggestions button * Use allowed formats in format types calculation * Add RichText version to PlainText component * Use experimental version of PlainText in Shortcode block * Add disableAutocorrection prop to RichText * Disable autocorrection in Shortcode block * Update PlainText props in Shortcode block * Use pre as tagName in PlainText * Rename replaceLineBreaks function * Update shortcode block unit tests * Prevent text input focus when selecting Shortcode block * Force text color in Shortcode block * Remove tagName prop from PlainText component * Mobile - Update changelog * Translate NEW badge in Move blocks help screen * Release script: Update react-native-editor version to 1.76.2 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Ensure post title gets focused when is notified from native side (#41371) * Update react-native-editor changelog * Release script: Update react-native-editor version to 1.76.3 * Release script: Update with changes from 'npm run core preios' * Bump react-native-reanimated dependency * Update changelog * Update Changelog * Revert removing comma Co-authored-by: jhnstn <[email protected]> Co-authored-by: Carlos Garcia <[email protected]>
fluiddot
added a commit
that referenced
this pull request
Jun 3, 2022
* Release script: Update react-native-editor version to 1.76.0 * Release script: Update with changes from 'npm run core preios' * Update Changelog * Release script: Update react-native-editor version to 1.76.1 * Release script: Update with changes from 'npm run core preios' * [Mobile] - BlockList - Add internal onLayout from CellRendererComponent to BlockListItemCell (#41105) * Mobile - BlockList - Pass FlatList internal onLayout through CellRendererComponent * Mobile - Update onLayout naming for BlockListItemCell * [Mobile] - Fix Drag & Drop Chip positioning issue with RTL languages (#41053) * Mobile - Fix - Drag & drop chip issue with RTL languages * Mobile - BlockDraggable - Rename maxWidth variable to contentWidth * Mobile - BlockDraggable - Add custom exiting animation that uses the same functionality as ZoomOutEasyDown but customizing the translateX value taking into account RTL languages. * [RNMobile] Add drag & drop help guide in Help & Support screen (#40961) * Add drag & drop help guide * Update content of help screen related to moving blocks * Update drand-and-drop images * Update styles of heading component of help screen * Add HelpDetailBadge component This component will be rendered in `HelpDetailSectionHeadingText` component via the `badge` prop. * Add NEW badge to move blocks help section * Optimize drag-and-drop images * Add move-blocks icon to Help & Support screen * Update react-native-editor changelog * Add HelpSectionTitle component * Prevent rendering separator on last help topic item * [RNMobile] Fix drag mode not being enabled when long-pressing over Shortcode block (#41155) * Add prop for disabling suggestions button * Use allowed formats in format types calculation * Add RichText version to PlainText component * Use experimental version of PlainText in Shortcode block * Add disableAutocorrection prop to RichText * Disable autocorrection in Shortcode block * Update PlainText props in Shortcode block * Use pre as tagName in PlainText * Rename replaceLineBreaks function * Update shortcode block unit tests * Prevent text input focus when selecting Shortcode block * Force text color in Shortcode block * Remove tagName prop from PlainText component * Mobile - Update changelog * Translate NEW badge in Move blocks help screen * Release script: Update react-native-editor version to 1.77.0 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Improve text read by screen readers for BottomSheetSelectControl (#41036) Improves the text that's read by screen readers by the BottomSheetSelectControl component, adding extra context and making its purpose clearer. * [RNMobile] Add 'Insert from URL' option to Image block (#40334) * Add 'Insert from URL' option to Video and Image blocks * Update code style from linting * Improve test cases for Media Upload capture options * Fix whitespace issue * Update Media Upload option tests to be asynchronous * Update native image block to use correct image URL * Add error handling for invalid URLs to native Image block * Clear invalid URL error on Image URL success * Fix synchronicity of Media Upload option tests * Add check for URL handler to native Image block picker options * Update code style * Remove Video block from urlSource options Why: to be introduced in a later PR * Remove URL option from Video block for Media Upload test * Use Notice snackbar for native Image block error handling * Update Image/Media Upload code style and helpers * Use getImage to determine if URL is a valid image within Image block * Add loading indicator and isURL check to native Image block URL behavior * Add loading indicator to native Image block media placeholder * Fix whitespace issue in native Image block code style * Reuse native Image block loading indicator * Use undefined dimension attributes for the native Image block URL behavior Co-authored-by: Derek Blank <[email protected]> * [RNMobile] - E2E Simplify heading and lists blocks functions (#40670) * update tests using paragraph, heading and list blocks * fix slash inserter tests to work in ci * lint fixes * wait for ordered list to appear * lint fixes * extra click only on local env * wait to get backspace click reflected * re-add extra click only for local env * add wait to wait for backspace key to be reflected * lint fixes * break function, set position to get list block * lint fixes * use correct params, update function name * lint fixes * make maxIteration a parameter for isElementVisible * update xpath for list block * utilize waitForVisible for isElementVisible * lint fixes * add wait to getNumberOfParagraphBlocks and update xpath for android list block * update edit text xpath to be read from any level Co-authored-by: jos <[email protected]> * Add ruby version file (#41013) Use same version of ruby that is used on WP iOS * [RNMobile] Improvements to Getting Started Guides (#40964) * Update with copy from Gutenberg Mobile Gutenberg Mobile's readme had some copy in the troubleshooting section that wasn't included in Gutenberg's how-to: https://github.com/wordpress-mobile/gutenberg-mobile/edit/trunk/README.md This commit fixes that issue by updating Gutenberg's how-to with the latest from Gutenberg Mobile. * Tweak wording, update inconsistent capitalisation * Tweaks to copy and formatting With this commit, I've tidied up some of the copy, attempting to make the instructions as concise and helpful as possible. I've also set the images to the same width to tidy up the overall look of the document (the images will appear small on mobile devices, but can be tapped on to get the full image). * Set all images to fixed width, for consistency * Rearrange sections to improve overall flow - The "Unit Tests" is moved towards the bottom of the document, so that's it's closer to the instructions for integration tests. the document flows as follows: - The steps for installing Xcode are moved under the "iOS" section. - The overall flow of the document has been updated to the following: Clone Gutenberg > iOS > Android > Tests * Add details about recommended JDK * Add details about Cocoapods * Refine instructions for installing Cocoapods * Note `ffi` may already be installed Some versions of Ruby may not require the `ffi` to be manually installed, which is noted with this commit. * update expected html for file block (#41300) Co-authored-by: jos <[email protected]> * Add waitForVisible() to all blocks (#41126) * add waits to all block - 1st try * fix failing tests * correct if else condition * fix failing cover block test * update spaces * update to use new click helper * wait for blocks to be visible first in getFirstBlock and getLastBlock * remove length as a parameter * update timing for long press since it's failing intermittently in ci * remove deleted param, revert space changes * remove redundant code * exit function once condition is met * increse wait time for long press * remove unneccesary condition, update message, return click value Co-authored-by: jos <[email protected]> * Update CHANGELOG * Revert "update expected html for file block (#41300)" This reverts commit a046a61. * [RNMobile] Ensure post title gets focused when is notified from native side (#41371) * Release script: Update react-native-editor version to 1.77.1 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Bump `react-native-reanimated` version to `2.4.1-wp-3` (#41482) * Update react-native-editor changelog Co-authored-by: jhnstn <[email protected]> Co-authored-by: Gerardo <[email protected]> Co-authored-by: Siobhan <[email protected]> Co-authored-by: Derek Blank <[email protected]> Co-authored-by: Derek Blank <[email protected]> Co-authored-by: Jos <[email protected]> Co-authored-by: jos <[email protected]> Co-authored-by: Jason Johnston <[email protected]>
youknowriad
pushed a commit
that referenced
this pull request
Jun 3, 2022
* Release script: Update react-native-editor version to 1.76.0 * Release script: Update with changes from 'npm run core preios' * Update Changelog * Release script: Update react-native-editor version to 1.76.1 * Release script: Update with changes from 'npm run core preios' * [Mobile] - BlockList - Add internal onLayout from CellRendererComponent to BlockListItemCell (#41105) * Mobile - BlockList - Pass FlatList internal onLayout through CellRendererComponent * Mobile - Update onLayout naming for BlockListItemCell * [Mobile] - Fix Drag & Drop Chip positioning issue with RTL languages (#41053) * Mobile - Fix - Drag & drop chip issue with RTL languages * Mobile - BlockDraggable - Rename maxWidth variable to contentWidth * Mobile - BlockDraggable - Add custom exiting animation that uses the same functionality as ZoomOutEasyDown but customizing the translateX value taking into account RTL languages. * [RNMobile] Add drag & drop help guide in Help & Support screen (#40961) * Add drag & drop help guide * Update content of help screen related to moving blocks * Update drand-and-drop images * Update styles of heading component of help screen * Add HelpDetailBadge component This component will be rendered in `HelpDetailSectionHeadingText` component via the `badge` prop. * Add NEW badge to move blocks help section * Optimize drag-and-drop images * Add move-blocks icon to Help & Support screen * Update react-native-editor changelog * Add HelpSectionTitle component * Prevent rendering separator on last help topic item * [RNMobile] Fix drag mode not being enabled when long-pressing over Shortcode block (#41155) * Add prop for disabling suggestions button * Use allowed formats in format types calculation * Add RichText version to PlainText component * Use experimental version of PlainText in Shortcode block * Add disableAutocorrection prop to RichText * Disable autocorrection in Shortcode block * Update PlainText props in Shortcode block * Use pre as tagName in PlainText * Rename replaceLineBreaks function * Update shortcode block unit tests * Prevent text input focus when selecting Shortcode block * Force text color in Shortcode block * Remove tagName prop from PlainText component * Mobile - Update changelog * Translate NEW badge in Move blocks help screen * Release script: Update react-native-editor version to 1.76.2 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Ensure post title gets focused when is notified from native side (#41371) * Update react-native-editor changelog * Update Podfile.lock Co-authored-by: jhnstn <[email protected]> Co-authored-by: Gerardo <[email protected]>
fluiddot
added a commit
that referenced
this pull request
Jun 13, 2022
* Release script: Update react-native-editor version to 1.76.0 * Release script: Update with changes from 'npm run core preios' * Update Changelog * Release script: Update react-native-editor version to 1.76.1 * Release script: Update with changes from 'npm run core preios' * [Mobile] - BlockList - Add internal onLayout from CellRendererComponent to BlockListItemCell (#41105) * Mobile - BlockList - Pass FlatList internal onLayout through CellRendererComponent * Mobile - Update onLayout naming for BlockListItemCell * [Mobile] - Fix Drag & Drop Chip positioning issue with RTL languages (#41053) * Mobile - Fix - Drag & drop chip issue with RTL languages * Mobile - BlockDraggable - Rename maxWidth variable to contentWidth * Mobile - BlockDraggable - Add custom exiting animation that uses the same functionality as ZoomOutEasyDown but customizing the translateX value taking into account RTL languages. * [RNMobile] Add drag & drop help guide in Help & Support screen (#40961) * Add drag & drop help guide * Update content of help screen related to moving blocks * Update drand-and-drop images * Update styles of heading component of help screen * Add HelpDetailBadge component This component will be rendered in `HelpDetailSectionHeadingText` component via the `badge` prop. * Add NEW badge to move blocks help section * Optimize drag-and-drop images * Add move-blocks icon to Help & Support screen * Update react-native-editor changelog * Add HelpSectionTitle component * Prevent rendering separator on last help topic item * [RNMobile] Fix drag mode not being enabled when long-pressing over Shortcode block (#41155) * Add prop for disabling suggestions button * Use allowed formats in format types calculation * Add RichText version to PlainText component * Use experimental version of PlainText in Shortcode block * Add disableAutocorrection prop to RichText * Disable autocorrection in Shortcode block * Update PlainText props in Shortcode block * Use pre as tagName in PlainText * Rename replaceLineBreaks function * Update shortcode block unit tests * Prevent text input focus when selecting Shortcode block * Force text color in Shortcode block * Remove tagName prop from PlainText component * Mobile - Update changelog * Translate NEW badge in Move blocks help screen * Release script: Update react-native-editor version to 1.77.0 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Improve text read by screen readers for BottomSheetSelectControl (#41036) Improves the text that's read by screen readers by the BottomSheetSelectControl component, adding extra context and making its purpose clearer. * [RNMobile] Add 'Insert from URL' option to Image block (#40334) * Add 'Insert from URL' option to Video and Image blocks * Update code style from linting * Improve test cases for Media Upload capture options * Fix whitespace issue * Update Media Upload option tests to be asynchronous * Update native image block to use correct image URL * Add error handling for invalid URLs to native Image block * Clear invalid URL error on Image URL success * Fix synchronicity of Media Upload option tests * Add check for URL handler to native Image block picker options * Update code style * Remove Video block from urlSource options Why: to be introduced in a later PR * Remove URL option from Video block for Media Upload test * Use Notice snackbar for native Image block error handling * Update Image/Media Upload code style and helpers * Use getImage to determine if URL is a valid image within Image block * Add loading indicator and isURL check to native Image block URL behavior * Add loading indicator to native Image block media placeholder * Fix whitespace issue in native Image block code style * Reuse native Image block loading indicator * Use undefined dimension attributes for the native Image block URL behavior Co-authored-by: Derek Blank <[email protected]> * [RNMobile] - E2E Simplify heading and lists blocks functions (#40670) * update tests using paragraph, heading and list blocks * fix slash inserter tests to work in ci * lint fixes * wait for ordered list to appear * lint fixes * extra click only on local env * wait to get backspace click reflected * re-add extra click only for local env * add wait to wait for backspace key to be reflected * lint fixes * break function, set position to get list block * lint fixes * use correct params, update function name * lint fixes * make maxIteration a parameter for isElementVisible * update xpath for list block * utilize waitForVisible for isElementVisible * lint fixes * add wait to getNumberOfParagraphBlocks and update xpath for android list block * update edit text xpath to be read from any level Co-authored-by: jos <[email protected]> * Add ruby version file (#41013) Use same version of ruby that is used on WP iOS * [RNMobile] Improvements to Getting Started Guides (#40964) * Update with copy from Gutenberg Mobile Gutenberg Mobile's readme had some copy in the troubleshooting section that wasn't included in Gutenberg's how-to: https://github.com/wordpress-mobile/gutenberg-mobile/edit/trunk/README.md This commit fixes that issue by updating Gutenberg's how-to with the latest from Gutenberg Mobile. * Tweak wording, update inconsistent capitalisation * Tweaks to copy and formatting With this commit, I've tidied up some of the copy, attempting to make the instructions as concise and helpful as possible. I've also set the images to the same width to tidy up the overall look of the document (the images will appear small on mobile devices, but can be tapped on to get the full image). * Set all images to fixed width, for consistency * Rearrange sections to improve overall flow - The "Unit Tests" is moved towards the bottom of the document, so that's it's closer to the instructions for integration tests. the document flows as follows: - The steps for installing Xcode are moved under the "iOS" section. - The overall flow of the document has been updated to the following: Clone Gutenberg > iOS > Android > Tests * Add details about recommended JDK * Add details about Cocoapods * Refine instructions for installing Cocoapods * Note `ffi` may already be installed Some versions of Ruby may not require the `ffi` to be manually installed, which is noted with this commit. * update expected html for file block (#41300) Co-authored-by: jos <[email protected]> * Add waitForVisible() to all blocks (#41126) * add waits to all block - 1st try * fix failing tests * correct if else condition * fix failing cover block test * update spaces * update to use new click helper * wait for blocks to be visible first in getFirstBlock and getLastBlock * remove length as a parameter * update timing for long press since it's failing intermittently in ci * remove deleted param, revert space changes * remove redundant code * exit function once condition is met * increse wait time for long press * remove unneccesary condition, update message, return click value Co-authored-by: jos <[email protected]> * Update CHANGELOG * Revert "update expected html for file block (#41300)" This reverts commit a046a61. * [RNMobile] Ensure post title gets focused when is notified from native side (#41371) * Release script: Update react-native-editor version to 1.77.1 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Bump `react-native-reanimated` version to `2.4.1-wp-3` (#41482) * Update react-native-editor changelog * Release script: Update react-native-editor version to 1.78.0 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Bump `react-native-gesture-handler` to version `2.3.2` (#41337) * Update react-native-editor changelog * Remove react-native-editor changelog duplicate entries Co-authored-by: jhnstn <[email protected]> Co-authored-by: Gerardo <[email protected]> Co-authored-by: Siobhan <[email protected]> Co-authored-by: Derek Blank <[email protected]> Co-authored-by: Derek Blank <[email protected]> Co-authored-by: Jos <[email protected]> Co-authored-by: jos <[email protected]> Co-authored-by: Jason Johnston <[email protected]>
jhnstn
added a commit
to jhnstn/gutenberg
that referenced
this pull request
Jun 15, 2022
* Release script: Update react-native-editor version to 1.76.0 * Release script: Update with changes from 'npm run core preios' * Update Changelog * Release script: Update react-native-editor version to 1.76.1 * Release script: Update with changes from 'npm run core preios' * [Mobile] - BlockList - Add internal onLayout from CellRendererComponent to BlockListItemCell (WordPress#41105) * Mobile - BlockList - Pass FlatList internal onLayout through CellRendererComponent * Mobile - Update onLayout naming for BlockListItemCell * [Mobile] - Fix Drag & Drop Chip positioning issue with RTL languages (WordPress#41053) * Mobile - Fix - Drag & drop chip issue with RTL languages * Mobile - BlockDraggable - Rename maxWidth variable to contentWidth * Mobile - BlockDraggable - Add custom exiting animation that uses the same functionality as ZoomOutEasyDown but customizing the translateX value taking into account RTL languages. * [RNMobile] Add drag & drop help guide in Help & Support screen (WordPress#40961) * Add drag & drop help guide * Update content of help screen related to moving blocks * Update drand-and-drop images * Update styles of heading component of help screen * Add HelpDetailBadge component This component will be rendered in `HelpDetailSectionHeadingText` component via the `badge` prop. * Add NEW badge to move blocks help section * Optimize drag-and-drop images * Add move-blocks icon to Help & Support screen * Update react-native-editor changelog * Add HelpSectionTitle component * Prevent rendering separator on last help topic item * [RNMobile] Fix drag mode not being enabled when long-pressing over Shortcode block (WordPress#41155) * Add prop for disabling suggestions button * Use allowed formats in format types calculation * Add RichText version to PlainText component * Use experimental version of PlainText in Shortcode block * Add disableAutocorrection prop to RichText * Disable autocorrection in Shortcode block * Update PlainText props in Shortcode block * Use pre as tagName in PlainText * Rename replaceLineBreaks function * Update shortcode block unit tests * Prevent text input focus when selecting Shortcode block * Force text color in Shortcode block * Remove tagName prop from PlainText component * Mobile - Update changelog * Translate NEW badge in Move blocks help screen * Release script: Update react-native-editor version to 1.76.2 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Ensure post title gets focused when is notified from native side (WordPress#41371) * Update react-native-editor changelog * Release script: Update react-native-editor version to 1.76.3 * Release script: Update with changes from 'npm run core preios' * Bump react-native-reanimated dependency * Update changelog * Update Changelog * Revert removing comma Co-authored-by: jhnstn <[email protected]> Co-authored-by: Carlos Garcia <[email protected]>
jhnstn
added a commit
to jhnstn/gutenberg
that referenced
this pull request
Jun 15, 2022
* Release script: Update react-native-editor version to 1.76.0 * Release script: Update with changes from 'npm run core preios' * Update Changelog * Release script: Update react-native-editor version to 1.76.1 * Release script: Update with changes from 'npm run core preios' * [Mobile] - BlockList - Add internal onLayout from CellRendererComponent to BlockListItemCell (WordPress#41105) * Mobile - BlockList - Pass FlatList internal onLayout through CellRendererComponent * Mobile - Update onLayout naming for BlockListItemCell * [Mobile] - Fix Drag & Drop Chip positioning issue with RTL languages (WordPress#41053) * Mobile - Fix - Drag & drop chip issue with RTL languages * Mobile - BlockDraggable - Rename maxWidth variable to contentWidth * Mobile - BlockDraggable - Add custom exiting animation that uses the same functionality as ZoomOutEasyDown but customizing the translateX value taking into account RTL languages. * [RNMobile] Add drag & drop help guide in Help & Support screen (WordPress#40961) * Add drag & drop help guide * Update content of help screen related to moving blocks * Update drand-and-drop images * Update styles of heading component of help screen * Add HelpDetailBadge component This component will be rendered in `HelpDetailSectionHeadingText` component via the `badge` prop. * Add NEW badge to move blocks help section * Optimize drag-and-drop images * Add move-blocks icon to Help & Support screen * Update react-native-editor changelog * Add HelpSectionTitle component * Prevent rendering separator on last help topic item * [RNMobile] Fix drag mode not being enabled when long-pressing over Shortcode block (WordPress#41155) * Add prop for disabling suggestions button * Use allowed formats in format types calculation * Add RichText version to PlainText component * Use experimental version of PlainText in Shortcode block * Add disableAutocorrection prop to RichText * Disable autocorrection in Shortcode block * Update PlainText props in Shortcode block * Use pre as tagName in PlainText * Rename replaceLineBreaks function * Update shortcode block unit tests * Prevent text input focus when selecting Shortcode block * Force text color in Shortcode block * Remove tagName prop from PlainText component * Mobile - Update changelog * Translate NEW badge in Move blocks help screen * Release script: Update react-native-editor version to 1.77.0 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Improve text read by screen readers for BottomSheetSelectControl (WordPress#41036) Improves the text that's read by screen readers by the BottomSheetSelectControl component, adding extra context and making its purpose clearer. * [RNMobile] Add 'Insert from URL' option to Image block (WordPress#40334) * Add 'Insert from URL' option to Video and Image blocks * Update code style from linting * Improve test cases for Media Upload capture options * Fix whitespace issue * Update Media Upload option tests to be asynchronous * Update native image block to use correct image URL * Add error handling for invalid URLs to native Image block * Clear invalid URL error on Image URL success * Fix synchronicity of Media Upload option tests * Add check for URL handler to native Image block picker options * Update code style * Remove Video block from urlSource options Why: to be introduced in a later PR * Remove URL option from Video block for Media Upload test * Use Notice snackbar for native Image block error handling * Update Image/Media Upload code style and helpers * Use getImage to determine if URL is a valid image within Image block * Add loading indicator and isURL check to native Image block URL behavior * Add loading indicator to native Image block media placeholder * Fix whitespace issue in native Image block code style * Reuse native Image block loading indicator * Use undefined dimension attributes for the native Image block URL behavior Co-authored-by: Derek Blank <[email protected]> * [RNMobile] - E2E Simplify heading and lists blocks functions (WordPress#40670) * update tests using paragraph, heading and list blocks * fix slash inserter tests to work in ci * lint fixes * wait for ordered list to appear * lint fixes * extra click only on local env * wait to get backspace click reflected * re-add extra click only for local env * add wait to wait for backspace key to be reflected * lint fixes * break function, set position to get list block * lint fixes * use correct params, update function name * lint fixes * make maxIteration a parameter for isElementVisible * update xpath for list block * utilize waitForVisible for isElementVisible * lint fixes * add wait to getNumberOfParagraphBlocks and update xpath for android list block * update edit text xpath to be read from any level Co-authored-by: jos <[email protected]> * Add ruby version file (WordPress#41013) Use same version of ruby that is used on WP iOS * [RNMobile] Improvements to Getting Started Guides (WordPress#40964) * Update with copy from Gutenberg Mobile Gutenberg Mobile's readme had some copy in the troubleshooting section that wasn't included in Gutenberg's how-to: https://github.com/wordpress-mobile/gutenberg-mobile/edit/trunk/README.md This commit fixes that issue by updating Gutenberg's how-to with the latest from Gutenberg Mobile. * Tweak wording, update inconsistent capitalisation * Tweaks to copy and formatting With this commit, I've tidied up some of the copy, attempting to make the instructions as concise and helpful as possible. I've also set the images to the same width to tidy up the overall look of the document (the images will appear small on mobile devices, but can be tapped on to get the full image). * Set all images to fixed width, for consistency * Rearrange sections to improve overall flow - The "Unit Tests" is moved towards the bottom of the document, so that's it's closer to the instructions for integration tests. the document flows as follows: - The steps for installing Xcode are moved under the "iOS" section. - The overall flow of the document has been updated to the following: Clone Gutenberg > iOS > Android > Tests * Add details about recommended JDK * Add details about Cocoapods * Refine instructions for installing Cocoapods * Note `ffi` may already be installed Some versions of Ruby may not require the `ffi` to be manually installed, which is noted with this commit. * update expected html for file block (WordPress#41300) Co-authored-by: jos <[email protected]> * Add waitForVisible() to all blocks (WordPress#41126) * add waits to all block - 1st try * fix failing tests * correct if else condition * fix failing cover block test * update spaces * update to use new click helper * wait for blocks to be visible first in getFirstBlock and getLastBlock * remove length as a parameter * update timing for long press since it's failing intermittently in ci * remove deleted param, revert space changes * remove redundant code * exit function once condition is met * increse wait time for long press * remove unneccesary condition, update message, return click value Co-authored-by: jos <[email protected]> * Update CHANGELOG * Revert "update expected html for file block (WordPress#41300)" This reverts commit a046a61. * [RNMobile] Ensure post title gets focused when is notified from native side (WordPress#41371) * Release script: Update react-native-editor version to 1.77.1 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Bump `react-native-reanimated` version to `2.4.1-wp-3` (WordPress#41482) * Update react-native-editor changelog Co-authored-by: jhnstn <[email protected]> Co-authored-by: Gerardo <[email protected]> Co-authored-by: Siobhan <[email protected]> Co-authored-by: Derek Blank <[email protected]> Co-authored-by: Derek Blank <[email protected]> Co-authored-by: Jos <[email protected]> Co-authored-by: jos <[email protected]> Co-authored-by: Jason Johnston <[email protected]>
jhnstn
added a commit
to jhnstn/gutenberg
that referenced
this pull request
Jun 15, 2022
* Release script: Update react-native-editor version to 1.76.0 * Release script: Update with changes from 'npm run core preios' * Update Changelog * Release script: Update react-native-editor version to 1.76.1 * Release script: Update with changes from 'npm run core preios' * [Mobile] - BlockList - Add internal onLayout from CellRendererComponent to BlockListItemCell (WordPress#41105) * Mobile - BlockList - Pass FlatList internal onLayout through CellRendererComponent * Mobile - Update onLayout naming for BlockListItemCell * [Mobile] - Fix Drag & Drop Chip positioning issue with RTL languages (WordPress#41053) * Mobile - Fix - Drag & drop chip issue with RTL languages * Mobile - BlockDraggable - Rename maxWidth variable to contentWidth * Mobile - BlockDraggable - Add custom exiting animation that uses the same functionality as ZoomOutEasyDown but customizing the translateX value taking into account RTL languages. * [RNMobile] Add drag & drop help guide in Help & Support screen (WordPress#40961) * Add drag & drop help guide * Update content of help screen related to moving blocks * Update drand-and-drop images * Update styles of heading component of help screen * Add HelpDetailBadge component This component will be rendered in `HelpDetailSectionHeadingText` component via the `badge` prop. * Add NEW badge to move blocks help section * Optimize drag-and-drop images * Add move-blocks icon to Help & Support screen * Update react-native-editor changelog * Add HelpSectionTitle component * Prevent rendering separator on last help topic item * [RNMobile] Fix drag mode not being enabled when long-pressing over Shortcode block (WordPress#41155) * Add prop for disabling suggestions button * Use allowed formats in format types calculation * Add RichText version to PlainText component * Use experimental version of PlainText in Shortcode block * Add disableAutocorrection prop to RichText * Disable autocorrection in Shortcode block * Update PlainText props in Shortcode block * Use pre as tagName in PlainText * Rename replaceLineBreaks function * Update shortcode block unit tests * Prevent text input focus when selecting Shortcode block * Force text color in Shortcode block * Remove tagName prop from PlainText component * Mobile - Update changelog * Translate NEW badge in Move blocks help screen * Release script: Update react-native-editor version to 1.77.0 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Improve text read by screen readers for BottomSheetSelectControl (WordPress#41036) Improves the text that's read by screen readers by the BottomSheetSelectControl component, adding extra context and making its purpose clearer. * [RNMobile] Add 'Insert from URL' option to Image block (WordPress#40334) * Add 'Insert from URL' option to Video and Image blocks * Update code style from linting * Improve test cases for Media Upload capture options * Fix whitespace issue * Update Media Upload option tests to be asynchronous * Update native image block to use correct image URL * Add error handling for invalid URLs to native Image block * Clear invalid URL error on Image URL success * Fix synchronicity of Media Upload option tests * Add check for URL handler to native Image block picker options * Update code style * Remove Video block from urlSource options Why: to be introduced in a later PR * Remove URL option from Video block for Media Upload test * Use Notice snackbar for native Image block error handling * Update Image/Media Upload code style and helpers * Use getImage to determine if URL is a valid image within Image block * Add loading indicator and isURL check to native Image block URL behavior * Add loading indicator to native Image block media placeholder * Fix whitespace issue in native Image block code style * Reuse native Image block loading indicator * Use undefined dimension attributes for the native Image block URL behavior Co-authored-by: Derek Blank <[email protected]> * [RNMobile] - E2E Simplify heading and lists blocks functions (WordPress#40670) * update tests using paragraph, heading and list blocks * fix slash inserter tests to work in ci * lint fixes * wait for ordered list to appear * lint fixes * extra click only on local env * wait to get backspace click reflected * re-add extra click only for local env * add wait to wait for backspace key to be reflected * lint fixes * break function, set position to get list block * lint fixes * use correct params, update function name * lint fixes * make maxIteration a parameter for isElementVisible * update xpath for list block * utilize waitForVisible for isElementVisible * lint fixes * add wait to getNumberOfParagraphBlocks and update xpath for android list block * update edit text xpath to be read from any level Co-authored-by: jos <[email protected]> * Add ruby version file (WordPress#41013) Use same version of ruby that is used on WP iOS * [RNMobile] Improvements to Getting Started Guides (WordPress#40964) * Update with copy from Gutenberg Mobile Gutenberg Mobile's readme had some copy in the troubleshooting section that wasn't included in Gutenberg's how-to: https://github.com/wordpress-mobile/gutenberg-mobile/edit/trunk/README.md This commit fixes that issue by updating Gutenberg's how-to with the latest from Gutenberg Mobile. * Tweak wording, update inconsistent capitalisation * Tweaks to copy and formatting With this commit, I've tidied up some of the copy, attempting to make the instructions as concise and helpful as possible. I've also set the images to the same width to tidy up the overall look of the document (the images will appear small on mobile devices, but can be tapped on to get the full image). * Set all images to fixed width, for consistency * Rearrange sections to improve overall flow - The "Unit Tests" is moved towards the bottom of the document, so that's it's closer to the instructions for integration tests. the document flows as follows: - The steps for installing Xcode are moved under the "iOS" section. - The overall flow of the document has been updated to the following: Clone Gutenberg > iOS > Android > Tests * Add details about recommended JDK * Add details about Cocoapods * Refine instructions for installing Cocoapods * Note `ffi` may already be installed Some versions of Ruby may not require the `ffi` to be manually installed, which is noted with this commit. * update expected html for file block (WordPress#41300) Co-authored-by: jos <[email protected]> * Add waitForVisible() to all blocks (WordPress#41126) * add waits to all block - 1st try * fix failing tests * correct if else condition * fix failing cover block test * update spaces * update to use new click helper * wait for blocks to be visible first in getFirstBlock and getLastBlock * remove length as a parameter * update timing for long press since it's failing intermittently in ci * remove deleted param, revert space changes * remove redundant code * exit function once condition is met * increse wait time for long press * remove unneccesary condition, update message, return click value Co-authored-by: jos <[email protected]> * Update CHANGELOG * Revert "update expected html for file block (WordPress#41300)" This reverts commit a046a61. * [RNMobile] Ensure post title gets focused when is notified from native side (WordPress#41371) * Release script: Update react-native-editor version to 1.77.1 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Bump `react-native-reanimated` version to `2.4.1-wp-3` (WordPress#41482) * Update react-native-editor changelog * Release script: Update react-native-editor version to 1.78.0 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Bump `react-native-gesture-handler` to version `2.3.2` (WordPress#41337) * Update react-native-editor changelog * Remove react-native-editor changelog duplicate entries Co-authored-by: jhnstn <[email protected]> Co-authored-by: Gerardo <[email protected]> Co-authored-by: Siobhan <[email protected]> Co-authored-by: Derek Blank <[email protected]> Co-authored-by: Derek Blank <[email protected]> Co-authored-by: Jos <[email protected]> Co-authored-by: jos <[email protected]> Co-authored-by: Jason Johnston <[email protected]>
fluiddot
added a commit
that referenced
this pull request
Jun 17, 2022
* Release script: Update react-native-editor version to 1.76.0 * Release script: Update with changes from 'npm run core preios' * Update Changelog * Release script: Update react-native-editor version to 1.76.1 * Release script: Update with changes from 'npm run core preios' * [Mobile] - BlockList - Add internal onLayout from CellRendererComponent to BlockListItemCell (#41105) * Mobile - BlockList - Pass FlatList internal onLayout through CellRendererComponent * Mobile - Update onLayout naming for BlockListItemCell * [Mobile] - Fix Drag & Drop Chip positioning issue with RTL languages (#41053) * Mobile - Fix - Drag & drop chip issue with RTL languages * Mobile - BlockDraggable - Rename maxWidth variable to contentWidth * Mobile - BlockDraggable - Add custom exiting animation that uses the same functionality as ZoomOutEasyDown but customizing the translateX value taking into account RTL languages. * [RNMobile] Add drag & drop help guide in Help & Support screen (#40961) * Add drag & drop help guide * Update content of help screen related to moving blocks * Update drand-and-drop images * Update styles of heading component of help screen * Add HelpDetailBadge component This component will be rendered in `HelpDetailSectionHeadingText` component via the `badge` prop. * Add NEW badge to move blocks help section * Optimize drag-and-drop images * Add move-blocks icon to Help & Support screen * Update react-native-editor changelog * Add HelpSectionTitle component * Prevent rendering separator on last help topic item * [RNMobile] Fix drag mode not being enabled when long-pressing over Shortcode block (#41155) * Add prop for disabling suggestions button * Use allowed formats in format types calculation * Add RichText version to PlainText component * Use experimental version of PlainText in Shortcode block * Add disableAutocorrection prop to RichText * Disable autocorrection in Shortcode block * Update PlainText props in Shortcode block * Use pre as tagName in PlainText * Rename replaceLineBreaks function * Update shortcode block unit tests * Prevent text input focus when selecting Shortcode block * Force text color in Shortcode block * Remove tagName prop from PlainText component * Mobile - Update changelog * Translate NEW badge in Move blocks help screen * Release script: Update react-native-editor version to 1.77.0 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Improve text read by screen readers for BottomSheetSelectControl (#41036) Improves the text that's read by screen readers by the BottomSheetSelectControl component, adding extra context and making its purpose clearer. * [RNMobile] Add 'Insert from URL' option to Image block (#40334) * Add 'Insert from URL' option to Video and Image blocks * Update code style from linting * Improve test cases for Media Upload capture options * Fix whitespace issue * Update Media Upload option tests to be asynchronous * Update native image block to use correct image URL * Add error handling for invalid URLs to native Image block * Clear invalid URL error on Image URL success * Fix synchronicity of Media Upload option tests * Add check for URL handler to native Image block picker options * Update code style * Remove Video block from urlSource options Why: to be introduced in a later PR * Remove URL option from Video block for Media Upload test * Use Notice snackbar for native Image block error handling * Update Image/Media Upload code style and helpers * Use getImage to determine if URL is a valid image within Image block * Add loading indicator and isURL check to native Image block URL behavior * Add loading indicator to native Image block media placeholder * Fix whitespace issue in native Image block code style * Reuse native Image block loading indicator * Use undefined dimension attributes for the native Image block URL behavior Co-authored-by: Derek Blank <[email protected]> * [RNMobile] - E2E Simplify heading and lists blocks functions (#40670) * update tests using paragraph, heading and list blocks * fix slash inserter tests to work in ci * lint fixes * wait for ordered list to appear * lint fixes * extra click only on local env * wait to get backspace click reflected * re-add extra click only for local env * add wait to wait for backspace key to be reflected * lint fixes * break function, set position to get list block * lint fixes * use correct params, update function name * lint fixes * make maxIteration a parameter for isElementVisible * update xpath for list block * utilize waitForVisible for isElementVisible * lint fixes * add wait to getNumberOfParagraphBlocks and update xpath for android list block * update edit text xpath to be read from any level Co-authored-by: jos <[email protected]> * Add ruby version file (#41013) Use same version of ruby that is used on WP iOS * [RNMobile] Improvements to Getting Started Guides (#40964) * Update with copy from Gutenberg Mobile Gutenberg Mobile's readme had some copy in the troubleshooting section that wasn't included in Gutenberg's how-to: https://github.com/wordpress-mobile/gutenberg-mobile/edit/trunk/README.md This commit fixes that issue by updating Gutenberg's how-to with the latest from Gutenberg Mobile. * Tweak wording, update inconsistent capitalisation * Tweaks to copy and formatting With this commit, I've tidied up some of the copy, attempting to make the instructions as concise and helpful as possible. I've also set the images to the same width to tidy up the overall look of the document (the images will appear small on mobile devices, but can be tapped on to get the full image). * Set all images to fixed width, for consistency * Rearrange sections to improve overall flow - The "Unit Tests" is moved towards the bottom of the document, so that's it's closer to the instructions for integration tests. the document flows as follows: - The steps for installing Xcode are moved under the "iOS" section. - The overall flow of the document has been updated to the following: Clone Gutenberg > iOS > Android > Tests * Add details about recommended JDK * Add details about Cocoapods * Refine instructions for installing Cocoapods * Note `ffi` may already be installed Some versions of Ruby may not require the `ffi` to be manually installed, which is noted with this commit. * update expected html for file block (#41300) Co-authored-by: jos <[email protected]> * Add waitForVisible() to all blocks (#41126) * add waits to all block - 1st try * fix failing tests * correct if else condition * fix failing cover block test * update spaces * update to use new click helper * wait for blocks to be visible first in getFirstBlock and getLastBlock * remove length as a parameter * update timing for long press since it's failing intermittently in ci * remove deleted param, revert space changes * remove redundant code * exit function once condition is met * increse wait time for long press * remove unneccesary condition, update message, return click value Co-authored-by: jos <[email protected]> * Update CHANGELOG * Revert "update expected html for file block (#41300)" This reverts commit a046a61. * [RNMobile] Ensure post title gets focused when is notified from native side (#41371) * Release script: Update react-native-editor version to 1.77.1 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Bump `react-native-reanimated` version to `2.4.1-wp-3` (#41482) * Update react-native-editor changelog * Release script: Update react-native-editor version to 1.78.0 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Bump `react-native-gesture-handler` to version `2.3.2` (#41337) * Update react-native-editor changelog * Release script: Update react-native-editor version to 1.78.1 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Gallery block: Re-introduce `v1` (#41533) * Fix isGalleryV2Enabled calculation for the native version * Update comment in isGalleryV2Enabled function Co-authored-by: David Calhoun <[email protected]> Co-authored-by: David Calhoun <[email protected]> * [RNMobile] Fix missing translations for locales that include region (only on Android) (#41685) * Update CHANGELOG Co-authored-by: jhnstn <[email protected]> Co-authored-by: Gerardo <[email protected]> Co-authored-by: Carlos Garcia <[email protected]> Co-authored-by: Derek Blank <[email protected]> Co-authored-by: Derek Blank <[email protected]> Co-authored-by: Jos <[email protected]> Co-authored-by: jos <[email protected]> Co-authored-by: Jason Johnston <[email protected]> Co-authored-by: David Calhoun <[email protected]>
dcalhoun
added a commit
that referenced
this pull request
Jul 8, 2022
* Release script: Update react-native-editor version to 1.76.0 * Release script: Update with changes from 'npm run core preios' * Update Changelog * Release script: Update react-native-editor version to 1.76.1 * Release script: Update with changes from 'npm run core preios' * [Mobile] - BlockList - Add internal onLayout from CellRendererComponent to BlockListItemCell (#41105) * Mobile - BlockList - Pass FlatList internal onLayout through CellRendererComponent * Mobile - Update onLayout naming for BlockListItemCell * [Mobile] - Fix Drag & Drop Chip positioning issue with RTL languages (#41053) * Mobile - Fix - Drag & drop chip issue with RTL languages * Mobile - BlockDraggable - Rename maxWidth variable to contentWidth * Mobile - BlockDraggable - Add custom exiting animation that uses the same functionality as ZoomOutEasyDown but customizing the translateX value taking into account RTL languages. * [RNMobile] Add drag & drop help guide in Help & Support screen (#40961) * Add drag & drop help guide * Update content of help screen related to moving blocks * Update drand-and-drop images * Update styles of heading component of help screen * Add HelpDetailBadge component This component will be rendered in `HelpDetailSectionHeadingText` component via the `badge` prop. * Add NEW badge to move blocks help section * Optimize drag-and-drop images * Add move-blocks icon to Help & Support screen * Update react-native-editor changelog * Add HelpSectionTitle component * Prevent rendering separator on last help topic item * [RNMobile] Fix drag mode not being enabled when long-pressing over Shortcode block (#41155) * Add prop for disabling suggestions button * Use allowed formats in format types calculation * Add RichText version to PlainText component * Use experimental version of PlainText in Shortcode block * Add disableAutocorrection prop to RichText * Disable autocorrection in Shortcode block * Update PlainText props in Shortcode block * Use pre as tagName in PlainText * Rename replaceLineBreaks function * Update shortcode block unit tests * Prevent text input focus when selecting Shortcode block * Force text color in Shortcode block * Remove tagName prop from PlainText component * Mobile - Update changelog * Translate NEW badge in Move blocks help screen * Release script: Update react-native-editor version to 1.77.0 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Improve text read by screen readers for BottomSheetSelectControl (#41036) Improves the text that's read by screen readers by the BottomSheetSelectControl component, adding extra context and making its purpose clearer. * [RNMobile] Add 'Insert from URL' option to Image block (#40334) * Add 'Insert from URL' option to Video and Image blocks * Update code style from linting * Improve test cases for Media Upload capture options * Fix whitespace issue * Update Media Upload option tests to be asynchronous * Update native image block to use correct image URL * Add error handling for invalid URLs to native Image block * Clear invalid URL error on Image URL success * Fix synchronicity of Media Upload option tests * Add check for URL handler to native Image block picker options * Update code style * Remove Video block from urlSource options Why: to be introduced in a later PR * Remove URL option from Video block for Media Upload test * Use Notice snackbar for native Image block error handling * Update Image/Media Upload code style and helpers * Use getImage to determine if URL is a valid image within Image block * Add loading indicator and isURL check to native Image block URL behavior * Add loading indicator to native Image block media placeholder * Fix whitespace issue in native Image block code style * Reuse native Image block loading indicator * Use undefined dimension attributes for the native Image block URL behavior Co-authored-by: Derek Blank <[email protected]> * [RNMobile] - E2E Simplify heading and lists blocks functions (#40670) * update tests using paragraph, heading and list blocks * fix slash inserter tests to work in ci * lint fixes * wait for ordered list to appear * lint fixes * extra click only on local env * wait to get backspace click reflected * re-add extra click only for local env * add wait to wait for backspace key to be reflected * lint fixes * break function, set position to get list block * lint fixes * use correct params, update function name * lint fixes * make maxIteration a parameter for isElementVisible * update xpath for list block * utilize waitForVisible for isElementVisible * lint fixes * add wait to getNumberOfParagraphBlocks and update xpath for android list block * update edit text xpath to be read from any level Co-authored-by: jos <[email protected]> * Add ruby version file (#41013) Use same version of ruby that is used on WP iOS * [RNMobile] Improvements to Getting Started Guides (#40964) * Update with copy from Gutenberg Mobile Gutenberg Mobile's readme had some copy in the troubleshooting section that wasn't included in Gutenberg's how-to: https://github.com/wordpress-mobile/gutenberg-mobile/edit/trunk/README.md This commit fixes that issue by updating Gutenberg's how-to with the latest from Gutenberg Mobile. * Tweak wording, update inconsistent capitalisation * Tweaks to copy and formatting With this commit, I've tidied up some of the copy, attempting to make the instructions as concise and helpful as possible. I've also set the images to the same width to tidy up the overall look of the document (the images will appear small on mobile devices, but can be tapped on to get the full image). * Set all images to fixed width, for consistency * Rearrange sections to improve overall flow - The "Unit Tests" is moved towards the bottom of the document, so that's it's closer to the instructions for integration tests. the document flows as follows: - The steps for installing Xcode are moved under the "iOS" section. - The overall flow of the document has been updated to the following: Clone Gutenberg > iOS > Android > Tests * Add details about recommended JDK * Add details about Cocoapods * Refine instructions for installing Cocoapods * Note `ffi` may already be installed Some versions of Ruby may not require the `ffi` to be manually installed, which is noted with this commit. * update expected html for file block (#41300) Co-authored-by: jos <[email protected]> * Add waitForVisible() to all blocks (#41126) * add waits to all block - 1st try * fix failing tests * correct if else condition * fix failing cover block test * update spaces * update to use new click helper * wait for blocks to be visible first in getFirstBlock and getLastBlock * remove length as a parameter * update timing for long press since it's failing intermittently in ci * remove deleted param, revert space changes * remove redundant code * exit function once condition is met * increse wait time for long press * remove unneccesary condition, update message, return click value Co-authored-by: jos <[email protected]> * Update CHANGELOG * Revert "update expected html for file block (#41300)" This reverts commit a046a61. * [RNMobile] Ensure post title gets focused when is notified from native side (#41371) * Release script: Update react-native-editor version to 1.77.1 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Bump `react-native-reanimated` version to `2.4.1-wp-3` (#41482) * Update react-native-editor changelog * Release script: Update react-native-editor version to 1.78.0 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Bump `react-native-gesture-handler` to version `2.3.2` (#41337) * Update react-native-editor changelog * Release script: Update react-native-editor version to 1.78.1 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Gallery block: Re-introduce `v1` (#41533) * Fix isGalleryV2Enabled calculation for the native version * Update comment in isGalleryV2Enabled function Co-authored-by: David Calhoun <[email protected]> Co-authored-by: David Calhoun <[email protected]> * [RNMobile] Fix missing translations for locales that include region (only on Android) (#41685) * Update CHANGELOG * Release script: Update react-native-editor version to 1.79.0 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Add integration tests to cover Drag & Drop functionality (#41364) * Add testID prop to Draggable components * Add unit tests for Draggable component * Set draggingId shared value before enable dragging This change is required for testing, otherwise the dragging id is not passed when the dragging gesture begins. * Mock generateHapticFeedback function * Add testID to draggable chip component * Add testID to BlockDraggable component * Add test helpers for BlockDraggable component Additionally, helpers related to fake timers have been added and updated in the global helpers file. * Add drag and drop integration tests * Update react-native-aztec mock to use AztecInputState * [Mobile] - Add E2E tests for the Drag & Drop blocks feature (#41368) * Mobile - Add E2E tests for the Drag & Drop blocks feature * Update longPress action * Use clickIfClickable * Use longPress instead of press * Fix clipboard typo * Add setClipboard and clearClipboard helpers * Mobile - Skip some of the Drag & Drop E2E on iOS (#41529) * Mobile - Skip some of the Drag & Drop E2E on iOS * Replace if condition to use onlyOnAndroid wrapper instead * where it can change to use clickIfClickable(), add new param for waitForVisible() to control return value (#41367) Co-authored-by: jos <[email protected]> * [RNMobile] Add 'Insert from URL' option to Video block (#41493) * Add onSelectURL function to mobile Video block edit * Add notices store to mobile Video block * Display Insert from URL in Video block media options menu * Allow native video block to use Embed block for embeddable URLs * Update native Video block src onSelectURL attribute * Update CHANGELOG * Upgrade Aztec to v1.5.9 (#41828) * fix: Image block preserves alt text from media library (#41839) * fix: Image block preserves alt text from media library When the alt text for a media item is present in the media library, that value should be copied into the Image block when inserted. This behavior avoids the need to re-enter the alt text for each image inserted into the post content. * fix: Allow Media with and without alt text Previous code required an alt text value, which broken existing code. This overrides the method to support both contexts. It also updates existing code to use the same method of generating media throughout the source. * test: Update alt text for demo editor test data Consistently setting the alt text for all platforms will likely help avoid confusion. * test: Update outdated test fixture data The demo editor now returns an alt text for the test image media. * docs: Update changelog * [RNMobile] Implement recovery option for invalid blocks (#41988) If a block's validation fails within the mobile app, users are shown an error but not provided with any options for recovering the block. This PR introduces an option for users to attempt block recovery. They'll be prompted to tap on a block if they wish to attempt recovery. * docs: Update change log * [RNMobile] Upgrades Gradle to 7.4.2 & Android Gradle Plugin to 7.2.1 (#42136) * Update Gradle to 7.4.1 * Update AGP to 7.2.1 * Update Aztec-Android version to v1.6.0 (#42243) Co-authored-by: jhnstn <[email protected]> Co-authored-by: Gerardo <[email protected]> Co-authored-by: Carlos Garcia <[email protected]> Co-authored-by: Siobhan <[email protected]> Co-authored-by: Derek Blank <[email protected]> Co-authored-by: Derek Blank <[email protected]> Co-authored-by: Jos <[email protected]> Co-authored-by: jos <[email protected]> Co-authored-by: Jason Johnston <[email protected]> Co-authored-by: Oguz Kocer <[email protected]>
dcalhoun
added a commit
that referenced
this pull request
Jul 20, 2022
* Release script: Update react-native-editor version to 1.76.0 * Release script: Update with changes from 'npm run core preios' * Update Changelog * Release script: Update react-native-editor version to 1.76.1 * Release script: Update with changes from 'npm run core preios' * [Mobile] - BlockList - Add internal onLayout from CellRendererComponent to BlockListItemCell (#41105) * Mobile - BlockList - Pass FlatList internal onLayout through CellRendererComponent * Mobile - Update onLayout naming for BlockListItemCell * [Mobile] - Fix Drag & Drop Chip positioning issue with RTL languages (#41053) * Mobile - Fix - Drag & drop chip issue with RTL languages * Mobile - BlockDraggable - Rename maxWidth variable to contentWidth * Mobile - BlockDraggable - Add custom exiting animation that uses the same functionality as ZoomOutEasyDown but customizing the translateX value taking into account RTL languages. * [RNMobile] Add drag & drop help guide in Help & Support screen (#40961) * Add drag & drop help guide * Update content of help screen related to moving blocks * Update drand-and-drop images * Update styles of heading component of help screen * Add HelpDetailBadge component This component will be rendered in `HelpDetailSectionHeadingText` component via the `badge` prop. * Add NEW badge to move blocks help section * Optimize drag-and-drop images * Add move-blocks icon to Help & Support screen * Update react-native-editor changelog * Add HelpSectionTitle component * Prevent rendering separator on last help topic item * [RNMobile] Fix drag mode not being enabled when long-pressing over Shortcode block (#41155) * Add prop for disabling suggestions button * Use allowed formats in format types calculation * Add RichText version to PlainText component * Use experimental version of PlainText in Shortcode block * Add disableAutocorrection prop to RichText * Disable autocorrection in Shortcode block * Update PlainText props in Shortcode block * Use pre as tagName in PlainText * Rename replaceLineBreaks function * Update shortcode block unit tests * Prevent text input focus when selecting Shortcode block * Force text color in Shortcode block * Remove tagName prop from PlainText component * Mobile - Update changelog * Translate NEW badge in Move blocks help screen * Release script: Update react-native-editor version to 1.77.0 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Improve text read by screen readers for BottomSheetSelectControl (#41036) Improves the text that's read by screen readers by the BottomSheetSelectControl component, adding extra context and making its purpose clearer. * [RNMobile] Add 'Insert from URL' option to Image block (#40334) * Add 'Insert from URL' option to Video and Image blocks * Update code style from linting * Improve test cases for Media Upload capture options * Fix whitespace issue * Update Media Upload option tests to be asynchronous * Update native image block to use correct image URL * Add error handling for invalid URLs to native Image block * Clear invalid URL error on Image URL success * Fix synchronicity of Media Upload option tests * Add check for URL handler to native Image block picker options * Update code style * Remove Video block from urlSource options Why: to be introduced in a later PR * Remove URL option from Video block for Media Upload test * Use Notice snackbar for native Image block error handling * Update Image/Media Upload code style and helpers * Use getImage to determine if URL is a valid image within Image block * Add loading indicator and isURL check to native Image block URL behavior * Add loading indicator to native Image block media placeholder * Fix whitespace issue in native Image block code style * Reuse native Image block loading indicator * Use undefined dimension attributes for the native Image block URL behavior Co-authored-by: Derek Blank <[email protected]> * [RNMobile] - E2E Simplify heading and lists blocks functions (#40670) * update tests using paragraph, heading and list blocks * fix slash inserter tests to work in ci * lint fixes * wait for ordered list to appear * lint fixes * extra click only on local env * wait to get backspace click reflected * re-add extra click only for local env * add wait to wait for backspace key to be reflected * lint fixes * break function, set position to get list block * lint fixes * use correct params, update function name * lint fixes * make maxIteration a parameter for isElementVisible * update xpath for list block * utilize waitForVisible for isElementVisible * lint fixes * add wait to getNumberOfParagraphBlocks and update xpath for android list block * update edit text xpath to be read from any level Co-authored-by: jos <[email protected]> * Add ruby version file (#41013) Use same version of ruby that is used on WP iOS * [RNMobile] Improvements to Getting Started Guides (#40964) * Update with copy from Gutenberg Mobile Gutenberg Mobile's readme had some copy in the troubleshooting section that wasn't included in Gutenberg's how-to: https://github.com/wordpress-mobile/gutenberg-mobile/edit/trunk/README.md This commit fixes that issue by updating Gutenberg's how-to with the latest from Gutenberg Mobile. * Tweak wording, update inconsistent capitalisation * Tweaks to copy and formatting With this commit, I've tidied up some of the copy, attempting to make the instructions as concise and helpful as possible. I've also set the images to the same width to tidy up the overall look of the document (the images will appear small on mobile devices, but can be tapped on to get the full image). * Set all images to fixed width, for consistency * Rearrange sections to improve overall flow - The "Unit Tests" is moved towards the bottom of the document, so that's it's closer to the instructions for integration tests. the document flows as follows: - The steps for installing Xcode are moved under the "iOS" section. - The overall flow of the document has been updated to the following: Clone Gutenberg > iOS > Android > Tests * Add details about recommended JDK * Add details about Cocoapods * Refine instructions for installing Cocoapods * Note `ffi` may already be installed Some versions of Ruby may not require the `ffi` to be manually installed, which is noted with this commit. * update expected html for file block (#41300) Co-authored-by: jos <[email protected]> * Add waitForVisible() to all blocks (#41126) * add waits to all block - 1st try * fix failing tests * correct if else condition * fix failing cover block test * update spaces * update to use new click helper * wait for blocks to be visible first in getFirstBlock and getLastBlock * remove length as a parameter * update timing for long press since it's failing intermittently in ci * remove deleted param, revert space changes * remove redundant code * exit function once condition is met * increse wait time for long press * remove unneccesary condition, update message, return click value Co-authored-by: jos <[email protected]> * Update CHANGELOG * Revert "update expected html for file block (#41300)" This reverts commit a046a61. * [RNMobile] Ensure post title gets focused when is notified from native side (#41371) * Release script: Update react-native-editor version to 1.77.1 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Bump `react-native-reanimated` version to `2.4.1-wp-3` (#41482) * Update react-native-editor changelog * Release script: Update react-native-editor version to 1.78.0 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Bump `react-native-gesture-handler` to version `2.3.2` (#41337) * Update react-native-editor changelog * Release script: Update react-native-editor version to 1.78.1 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Gallery block: Re-introduce `v1` (#41533) * Fix isGalleryV2Enabled calculation for the native version * Update comment in isGalleryV2Enabled function Co-authored-by: David Calhoun <[email protected]> Co-authored-by: David Calhoun <[email protected]> * [RNMobile] Fix missing translations for locales that include region (only on Android) (#41685) * Update CHANGELOG * Release script: Update react-native-editor version to 1.79.0 * Release script: Update with changes from 'npm run core preios' * [RNMobile] Add integration tests to cover Drag & Drop functionality (#41364) * Add testID prop to Draggable components * Add unit tests for Draggable component * Set draggingId shared value before enable dragging This change is required for testing, otherwise the dragging id is not passed when the dragging gesture begins. * Mock generateHapticFeedback function * Add testID to draggable chip component * Add testID to BlockDraggable component * Add test helpers for BlockDraggable component Additionally, helpers related to fake timers have been added and updated in the global helpers file. * Add drag and drop integration tests * Update react-native-aztec mock to use AztecInputState * [Mobile] - Add E2E tests for the Drag & Drop blocks feature (#41368) * Mobile - Add E2E tests for the Drag & Drop blocks feature * Update longPress action * Use clickIfClickable * Use longPress instead of press * Fix clipboard typo * Add setClipboard and clearClipboard helpers * Mobile - Skip some of the Drag & Drop E2E on iOS (#41529) * Mobile - Skip some of the Drag & Drop E2E on iOS * Replace if condition to use onlyOnAndroid wrapper instead * where it can change to use clickIfClickable(), add new param for waitForVisible() to control return value (#41367) Co-authored-by: jos <[email protected]> * [RNMobile] Add 'Insert from URL' option to Video block (#41493) * Add onSelectURL function to mobile Video block edit * Add notices store to mobile Video block * Display Insert from URL in Video block media options menu * Allow native video block to use Embed block for embeddable URLs * Update native Video block src onSelectURL attribute * Update CHANGELOG * Upgrade Aztec to v1.5.9 (#41828) * fix: Image block preserves alt text from media library (#41839) * fix: Image block preserves alt text from media library When the alt text for a media item is present in the media library, that value should be copied into the Image block when inserted. This behavior avoids the need to re-enter the alt text for each image inserted into the post content. * fix: Allow Media with and without alt text Previous code required an alt text value, which broken existing code. This overrides the method to support both contexts. It also updates existing code to use the same method of generating media throughout the source. * test: Update alt text for demo editor test data Consistently setting the alt text for all platforms will likely help avoid confusion. * test: Update outdated test fixture data The demo editor now returns an alt text for the test image media. * docs: Update changelog * [RNMobile] Implement recovery option for invalid blocks (#41988) If a block's validation fails within the mobile app, users are shown an error but not provided with any options for recovering the block. This PR introduces an option for users to attempt block recovery. They'll be prompted to tap on a block if they wish to attempt recovery. * docs: Update change log * [RNMobile] Upgrades Gradle to 7.4.2 & Android Gradle Plugin to 7.2.1 (#42136) * Update Gradle to 7.4.1 * Update AGP to 7.2.1 * Update Aztec-Android version to v1.6.0 (#42243) * Release script: Update react-native-editor version to 1.79.1 * Release script: Update with changes from 'npm run core preios' * fix: Workaround Android 12 webview crash (#42517) * fix: Workaround Android 12 webview crash A crash occurred due to combining webviews with Android 12's new scroll overflow behavior. Applying `opacity: 0.99` prevents the crash from occurring. This change should be viewed as a temporary fix while we await a more appropriate solution for the `react-native-webview` library. react-native-webview/react-native-webview#1915 (comment) * docs: Add change log entry Co-authored-by: jhnstn <[email protected]> Co-authored-by: Gerardo <[email protected]> Co-authored-by: Carlos Garcia <[email protected]> Co-authored-by: Siobhan <[email protected]> Co-authored-by: Derek Blank <[email protected]> Co-authored-by: Derek Blank <[email protected]> Co-authored-by: Jos <[email protected]> Co-authored-by: jos <[email protected]> Co-authored-by: Jason Johnston <[email protected]> Co-authored-by: Oguz Kocer <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What?
Fixes the drag mode not being enabled due to the use of
PlainTextcomponent in the Shortcode block.Why?
This PR will address the issue described in wordpress-mobile/gutenberg-mobile#4853, specifically the one related to the Shortcode block. In the future, we could use the same approach for the rest of the affected blocks.
How?
The fix consists in using the
RichTextcomponent by disabling all formatting options for rendering plain text via thePlainTextcomponent, similarly as it's done experimentally in the web version (reference). In order to achieve this, I also had to apply the following modifications in theRichTextcomponent:disableSuggestionsprop to be able to disable the suggestion button in the formatting toolbar.disableAutocorrectionprop to be able to disable auto-correction settings of the text input. This change also implied updating the native code ofreact-native-aztecpackage.allowedFormatsvalue touseFormatTypeshook.Testing Instructions
Drag mode is enabled when long-pressing the content
Default long-press behavior when editing text
Shortcode block
Check enable/disable suggestions setting in
RichText@symbol) is displayed in the formatting toolbar.Check enable/disable Auto-correction setting in
RichTextmorninnis converted tomorning).Check other blocks using
PlainTextScreenshots or screencast
android-shortcode-block.mp4
ios-shortcode-block.mp4