Skip to content

[RNMobile] Update "add block" button style in default editor view#4705

Merged
SiobhyB merged 50 commits into
trunkfrom
version-toolkit/gutenberg/rnmobile/update/default-block-inserter-button
Apr 13, 2022
Merged

[RNMobile] Update "add block" button style in default editor view#4705
SiobhyB merged 50 commits into
trunkfrom
version-toolkit/gutenberg/rnmobile/update/default-block-inserter-button

Conversation

@wpmobilebot

@wpmobilebot wpmobilebot commented Mar 29, 2022

Copy link
Copy Markdown
Collaborator

Related PRs

Description

This PR is generated by version-toolkit to downstream the changes for gutenberg submodule. Please refer to the Gutenberg PR as the central PR with the most up-to-date description and testing steps for the proposed changes.

@peril-wordpress-mobile

peril-wordpress-mobile Bot commented Mar 29, 2022

Copy link
Copy Markdown

Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job!

@SiobhyB SiobhyB changed the title [WIP] [RNMobile] Update block inserter button [RNMobile] Update block inserter button Mar 29, 2022
@SiobhyB SiobhyB changed the title [RNMobile] Update block inserter button [RNMobile] Update "add block" button style in default editor view Mar 29, 2022

@jhnstn jhnstn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Comment on lines -161 to -163
<string name="gutenberg_native_failed_to_insert_audio_file" tools:ignore="UnusedResources">Failed to insert audio file.</string>
<string name="gutenberg_native_failed_to_insert_audio_file_please_tap_for_options" tools:ignore="UnusedResources">Failed to insert audio file. Please tap for options.</string>
<string name="gutenberg_native_failed_to_insert_media" tools:ignore="UnusedResources">Failed to insert media.</string>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These strings were removed in WordPress/gutenberg#39694, so the change is valid 👍 .

@fluiddot fluiddot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved via WordPress/gutenberg#39726 (review).

NOTE: We might consider updating the iOS bundle with the latest changes.

@SiobhyB SiobhyB enabled auto-merge April 13, 2022 10:47
@SiobhyB SiobhyB merged commit c08ac32 into trunk Apr 13, 2022
@SiobhyB SiobhyB deleted the version-toolkit/gutenberg/rnmobile/update/default-block-inserter-button branch April 13, 2022 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants