[RNMobile] Social Icons: Ensure inactive icons are visible with block-based themes

[wpmobilebot]

Related PRs

• [RNMobile] Social Icons: Ensure inactive icons are visible with block-based themes  WordPress/gutenberg#55398 by @SiobhyB

Description

This PR is generated by version-toolkit to downstream the changes for gutenberg submodule.

[geriux]

Hey @SiobhyB 👋 Since we are updating how the social icons get rendered, we'd need to update the visual tests snapshots before merging this PR.

[SiobhyB]

@geriux, thank you! I'm currently experiencing some issues running the UI tests but will update here when I've updated the snapshots.

[geriux]

@geriux, thank you! I'm currently experiencing some issues running the UI tests but will update here when I've updated the snapshots.

No problem! Let me know if I can help with those issues!

[SiobhyB]

@geriux, screenshots are updated now :)

[geriux]

@geriux, screenshots are updated now :)

Thank you!

It looks like this PR is not referencing the latest changes in Gutenberg trunk where we made changes for Appium 2. Since this PR is up to date with Gutenberg Mobile's trunk it'd be ideal to sync them so we can run the full tests before merging.

Maybe it'd be easier to merge WordPress/gutenberg#55398 and update the Gutenberg reference with the merge commit in this PR. What do you think?

[SiobhyB]

Maybe it'd be easier to merge WordPress/gutenberg#55398 and update the Gutenberg reference with the merge commit in this PR. What do you think?

Yes, of course, I've gone ahead to begin the merge waterfall now.

[geriux]

LGTM! Tests passed correctly in CI 🚀 Nice work!