Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mobile: Keep Settings Sheet Open After Setting a Featured Image via the Image Block #33821

Open
SiobhyB opened this issue Aug 2, 2021 · 0 comments
Labels
Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) [Priority] Low Used to indicate that the issue at hand isn't a top priority to address and can be handled later [Type] Enhancement A suggestion for improvement.

Comments

@SiobhyB
Copy link
Contributor

SiobhyB commented Aug 2, 2021

Description

When a user navigates to the image block's settings and taps the button to "Set/Remove as Featured", the settings sheet is dismissed. This could be viewed as a bit jarring and, as we've iterated on the design of the "Set/Remove as Featured" button, it'd be preferable to keep the user in the settings sheet when they tap it.

As discussed here, this won't be as simple as removing the call to closeSettingsBottomSheet from here in the code due to the fact we display a confirmation dialog when users already have a featured image set.

The bug described at #29672 means that, by default, the app will crash on iOS if we attempt to display the settings and the confirmation dialog at the same time. We could look into the workaround mentioned in the comments of that GitHub issue or consider removing the confirmation dialog altogether.

(As a side note, this would likely tie in nicely with a suggested enhancement for improving the confirmation dialog at #30410.)

Step-by-step reproduction instructions

  • In the app, navigate to HomeBlog Posts.
  • Add a new image block and upload an image to it.
  • Tap the image block's cog/gear icon to access its settings and then tap the Set/Remove as Featured button at the bottom.

Expected behaviour

To remain within the block's settings panel, where I can continue to adjust settings as needed and read the footer message, which is specific to the featured image changes. As mentioned here by @iamthomasbishop, this behaviour would make the most sense.

Actual behaviour

Tapping on the Set/Remove as Featured button takes you out of the settings panel.

Screenshots or screen recording (optional)

Screen.Recording.2021-08-02.at.15.13.27.mov
@SiobhyB SiobhyB added [Type] Enhancement A suggestion for improvement. Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) labels Aug 2, 2021
@mchowning mchowning added the [Priority] Low Used to indicate that the issue at hand isn't a top priority to address and can be handled later label Aug 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) [Priority] Low Used to indicate that the issue at hand isn't a top priority to address and can be handled later [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

2 participants