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

Multi-Select: hide grouping controls for when selecting text only blocks #40063

Open
annezazu opened this issue Apr 6, 2022 · 5 comments
Open
Labels
[Feature] Block Multi Selection The ability to select and manipulate multiple blocks Needs Design Feedback Needs general design feedback. [Type] Enhancement A suggestion for improvement.

Comments

@annezazu
Copy link
Contributor

annezazu commented Apr 6, 2022

What problem does this address?

When using multi-select, the block toolbar still displays the tooling to group/row/stack blocks:

multi.select.grouping.mov

This was found during the 6.0 walkthrough.

What is your proposed solution?

When using multi select with text only blocks (header, paragraph, etc), hide the controls for group/row/stack options:

Screen Shot 2022-04-05 at 6 10 21 PM

@annezazu annezazu added [Type] Enhancement A suggestion for improvement. [Feature] Block Multi Selection The ability to select and manipulate multiple blocks labels Apr 6, 2022
@andrewserong
Copy link
Contributor

Interesting idea! Why do we want to hide the controls when text blocks are selected? Grouping text blocks can be useful if there are paragraphs and headings that someone wants to highlight together:

highlighting-paragraphs

Or was it more about if text is only partially selected, then we should hide it 🤔

Apologies if the context is clearer from the 6.0 walkthrough, I didn't manage to catch it! 🙂

@annezazu
Copy link
Contributor Author

annezazu commented Apr 6, 2022

Ah to be clear, this is to hide the controls when using the new multi-select feature where you can select text across multiple blocks. At least, this was my interpretation as having all of these controls while using multi select feels like overkill.

Current:

Screen Shot 2022-04-06 at 10 45 47 AM

Proposed:

Screen Shot 2022-04-06 at 10 45 57 AM

@andrewserong
Copy link
Contributor

Thanks for clarifying! I see, yes, looks like we've got two slightly different use cases to look after:

  1. As a user, I want to select multiple text blocks to group them / re-arrange them in a particular way (add spacing / background colours, etc).
  2. As a user, I'm editing the content I'm currently writing, and I only want the controls relevant to me as I copy / paste / cut / rearrange text.

Adding some kind of logic that hides the controls when there's a partial selection of text within heading / paragraph blocks sounds good to me 👍

@ntsekouras
Copy link
Contributor

ntsekouras commented Apr 11, 2022

Someone could argue that the block toolbar and block actions menu are more about handling blocks as whole.. This is the approach we will take regarding the Copy button in block actions, even if we have partial selection. Personally I don't have a strong opinion on this, but I'm leaning to leaving it as is.

@andrewserong
Copy link
Contributor

Personally I don't have a strong opinion on this, but I'm leaning to leaving it as is.

I think I'm leaning slightly that way, too, since it can be difficult to predict intent just from the existence of a partial selection.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Block Multi Selection The ability to select and manipulate multiple blocks Needs Design Feedback Needs general design feedback. [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

4 participants