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

Selecting multiple non-text blocks using the shift key is no longer possible #40636

Closed
ndiego opened this issue Apr 27, 2022 · 5 comments · Fixed by #40687
Closed

Selecting multiple non-text blocks using the shift key is no longer possible #40636

ndiego opened this issue Apr 27, 2022 · 5 comments · Fixed by #40687
Assignees
Labels
General Interface Parts of the UI which don't fall neatly under other labels. [Priority] High Used to indicate top priority items that need quick attention [Type] Bug An existing feature does not function as intended [Type] Regression Related to a regression in the latest release

Comments

@ndiego
Copy link
Member

ndiego commented Apr 27, 2022

Description

Previously, you could hold down the shift key and select multiple blocks at once. This still works if you "start" with a paragraph or similar text-based block. However, if you try and select two non-text blocks using the shift key, such as two images, it is no longer possible. This is a bit hard to explain, so see the video below.

I believe this may be related to the new functionality that allows you to select text across multiple blocks. As such, we should really try and get this resolved for 6.0.

Step-by-step reproduction instructions

  1. Use the latest version of Gutenberg and 6.0 Beta 3 and Twenty Twenty-Two
  2. Add two image blocks to the Editor
  3. Hold down the shift key and try to select both, it won't work
  4. Now revert to 5.9.3 and disable Gutenberg
  5. Add two image blocks to the Editor
  6. Hold down the shift key and try to select both, it will work

Screenshots, screen recording, code snippet

multiselect.mp4

Environment info

  • WordPress 6.0 Beta 3
  • Gutenberg 13.0

Please confirm that you have searched existing issues in the repo.

Yes

Please confirm that you have tested with all plugins deactivated except Gutenberg.

Yes

@ndiego ndiego added [Type] Bug An existing feature does not function as intended Needs Testing Needs further testing to be confirmed. labels Apr 27, 2022
@ndiego ndiego added the General Interface Parts of the UI which don't fall neatly under other labels. label Apr 27, 2022
@ndiego
Copy link
Member Author

ndiego commented Apr 27, 2022

@annezazu @priethor @gziolo @adamziel sorry for the ping, but wanted to get more eyes on this given how tight we are getting on time.

@annezazu
Copy link
Contributor

Testing with just WordPress 6.0 beta 3 and can confirm the above. Here are some specific use cases to save some time:

  • Selecting an image + holding down shift > can only select other text blocks (can't select another image, cover, or group block)
  • Selecting a group block + holding down shift > can only select other text blocks (can't select another image, cover, or group block)
  • Selecting a cover block at the bottom of List view + holding down shift + selecting a paragraph at the top of list view > selects everything in between.

Reverting back to 5.9.3, this is the prior behavior -- note though that I could replicate the above problem before reloading the page 🤔 Not sure what's going on there:

5.9.3.mov

@gziolo gziolo added [Type] Regression Related to a regression in the latest release [Priority] High Used to indicate top priority items that need quick attention and removed Needs Testing Needs further testing to be confirmed. labels Apr 27, 2022
@gziolo gziolo moved this from Triage to Todo in WordPress 6.0 Editor Tasks Apr 27, 2022
@fabiankaegy
Copy link
Member

Could this possibly be related to the changes in #40604 ?

@talldan
Copy link
Contributor

talldan commented Apr 27, 2022

Could this possibly be related to the changes in #40604 ?

I just gave the code from before that commit a test, and the bug was still present, so I don't think it's related to that.

From what I can tell, multi-selecting blocks now only seems to work when clicking on text.

@priethor
Copy link
Contributor

Thanks for the thorough testing and detailed instructions. I did some of my own and I agree it seems related to the partial block multiselection.

One extra use test case:

  • Partially select text in a paragraph -> Shift+click on an Image/Cover block that has an image -> it doesn't work
  • Partially select text in a paragraph -> Shift+click on an empty Image/Cover block -> it works

It's also worth mentioning that selecting by dragging instead of using Shift works, as well as using Shift in the list view after selecting partial text, so it only seems related to using Shift in the canvas.

@github-actions github-actions bot added the [Status] In Progress Tracking issues with work in progress label Apr 28, 2022
Repository owner moved this from Todo to Done in WordPress 6.0 Editor Tasks Apr 29, 2022
@priethor priethor removed the [Status] In Progress Tracking issues with work in progress label May 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
General Interface Parts of the UI which don't fall neatly under other labels. [Priority] High Used to indicate top priority items that need quick attention [Type] Bug An existing feature does not function as intended [Type] Regression Related to a regression in the latest release
Projects
No open projects
Archived in project
Development

Successfully merging a pull request may close this issue.

7 participants