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

Audio and Video blocks: the toolbar "Edit" button disappears or re-appears in a different position #16048

Closed
afercia opened this issue Jun 7, 2019 · 6 comments
Labels
[Block] Audio Affects the Audio Block [Block] Video Affects the Video Block [Feature] Media Anything that impacts the experience of managing media [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Type] Bug An existing feature does not function as intended

Comments

@afercia
Copy link
Contributor

afercia commented Jun 7, 2019

Noticed in the Audio and Video blocks.

When clicking on the "Edit" button in the block toolbar, the button disappears. Depending on the interaction mode (mouse, keyboard) I've noticed different behaviors that I wasn't able to reproduce consistently. Also, not sure if there's a difference between Gutenberg in core (WordPress 5.3-alpha) or the plugin (currently 5.8.0).

Anyways, then the button disappears:

  • there's a focus loss: this shouldn't happen as it's a terrible experience for keyboard users and assistive technology users
  • there's no way to switch back from the block "placeholder" to the view with the embedded media other than:
    • click on "Insert from URL"
    • the embedded media URL is still there
    • click "Apply"

At this point, the "Edit" button re-appears, sometimes in the same location and sometimes... at the beginning of the toolbar. Couldn't reproduce consistently.

To clarify, I made two animated GIFs. To reproduce, sometimes the disappearing doesn't happen the first time "Edit" is clicked, so try multiple times.

Gutenberg version included in wp trunk 5.3:

audio edit disappearing

Gutenberg plugin 5.8.0:

audio edit disappearing plugin

@afercia afercia added [Type] Bug An existing feature does not function as intended [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Feature] Media Anything that impacts the experience of managing media [Block] Video Affects the Video Block [Block] Audio Affects the Audio Block labels Jun 7, 2019
@afercia
Copy link
Contributor Author

afercia commented Jun 7, 2019

Related:
#15518 (review)
#15516 (comment)

@afercia
Copy link
Contributor Author

afercia commented Jun 29, 2019

@draganescu hi! Do I recall correctly you fixed this?

@draganescu
Copy link
Contributor

Hi @afercia it is work in progress. I've been away for the past 2 weeks. I am working on implementing a component that should replace / refactor this whole edit flow thing on all the affected components here:

#16200

@afercia
Copy link
Contributor Author

afercia commented Jul 16, 2019

Thanks @draganescu !

@swissspidy
Copy link
Member

This reminds me of #15641.

@Soean
Copy link
Member

Soean commented Jan 13, 2020

We now use the MediaReplaceFlow component in all media related blocks. See #19162

@Soean Soean closed this as completed Jan 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Audio Affects the Audio Block [Block] Video Affects the Video Block [Feature] Media Anything that impacts the experience of managing media [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

4 participants