Skip to content

Conversation

dcalhoun
Copy link
Member

@dcalhoun dcalhoun commented Oct 6, 2025

Description

Close CMM-824. Ref CMM-444. Ref wordpress-mobile/GutenbergKit#129.

Integrates wordpress-mobile/GutenbergKit#186.

Disabling the native header navigation respects GutenbergKit's web-based
modal dialogs, where focus should remain on the dialog to mitigate
confusion from surrounding UI elements.

The hardware back button now dismisses GutenbergKit dialogs first,
before performing the default Android action.

Testing instructions

1. Web modal dialogs disable native header navigation

  1. Open the experimental editor.
  2. Type some text.
  3. Open the block inserter.
  4. Verify all native header navigation UI is disabled.
  5. Close the block inserter.
  6. Verify all native header navigation UI is enabled.

2. Android hardware back button dismisses web dialogs

  1. Open the experimental editor.
  2. Open a modal dialog—e.g., block inserter, block inspector.
  3. Press the hardware back button.
  4. Verify the web dialog is dismissed.
  5. Press the hardware back button.
  6. Verify the editor is dismissed.

Disabling the native header navigation respects GutenbergKit's web-based
modal dialogs, where focus should remain on the dialog to mitigate
confusion from surrounding UI elements.

The hardware back button now dismisses GutenbergKit dialogs first,
before performing the default Android action.
@dcalhoun dcalhoun added this to the 26.3 milestone Oct 6, 2025
@dcalhoun dcalhoun added [Type] Enhancement Posting/Editing Gutenberg Editing and display of Gutenberg blocks. labels Oct 6, 2025
@wpmobilebot
Copy link
Contributor

wpmobilebot commented Oct 6, 2025

Project dependencies changes

list
! Upgraded Dependencies
org.wordpress.gutenbergkit:android:v0.9.0, (changed from v0.8.1)
tree
 +--- project :libs:editor
-|    \--- org.wordpress.gutenbergkit:android:v0.8.1
+|    \--- org.wordpress.gutenbergkit:android:v0.9.0
-\--- org.wordpress.gutenbergkit:android:v0.8.1 (*)
+\--- org.wordpress.gutenbergkit:android:v0.9.0 (*)

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Oct 6, 2025

WordPress📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress WordPress
FlavorJalapeno
Build TypeDebug
Versionpr22260-0665462
Commit0665462
Direct Downloadwordpress-prototype-build-pr22260-0665462.apk
Note: Google Login is not supported on these builds.

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Oct 6, 2025

Jetpack📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack Jetpack
FlavorJalapeno
Build TypeDebug
Versionpr22260-0665462
Commit0665462
Direct Downloadjetpack-prototype-build-pr22260-0665462.apk
Note: Google Login is not supported on these builds.

@dcalhoun dcalhoun marked this pull request as ready for review October 6, 2025 20:25
@dcalhoun dcalhoun requested a review from a team as a code owner October 6, 2025 20:25
@dcalhoun dcalhoun requested review from adalpari and removed request for a team October 6, 2025 20:25
Copy link
Contributor

@adalpari adalpari left a comment

Choose a reason for hiding this comment

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

LGTM and works as expected! :shipit:

@dcalhoun dcalhoun enabled auto-merge (squash) October 7, 2025 18:54
Copy link

sonarqubecloud bot commented Oct 7, 2025

@dcalhoun dcalhoun merged commit 141d3cc into trunk Oct 7, 2025
23 of 26 checks passed
@dcalhoun dcalhoun deleted the feat/gutenberg-kit-modals-disable-navigation-ui branch October 7, 2025 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Gutenberg Editing and display of Gutenberg blocks. Posting/Editing [Type] Enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants