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

Consider moving cog and trash to quick toolbar #2156

Closed
jasmussen opened this issue Aug 2, 2017 · 11 comments
Closed

Consider moving cog and trash to quick toolbar #2156

jasmussen opened this issue Aug 2, 2017 · 11 comments
Labels
[Priority] Low Used to indicate that the issue at hand isn't a top priority to address and can be handled later [Type] Question Questions about the design or development of the editor.

Comments

@jasmussen
Copy link
Contributor

Right now the cog and trash button float to the right of the block for two reasons:

  1. to free up space in the quick toolbar
  2. to be available on hover

1 is spurious as we've had to move the cog and trash buttons (as well as the mover) into an ellipsis on mobile regardless.

  1. is what we should discuss. There's been discussion recently about the weight of the block UI. And completely granted, it's a complex silhuette for just writing text.

Recent PRs all take stabs at improving this:

While these all improve things, it seems like it's still a heavy silhuette with floating buttons left and right of a button. By moving the cog and trash buttons to the quick toolbar, we can both get closer to the mobile UI, and we can reducie the silhuette, all at the cost of having to click a block before you can delete it. Since multi select makes it easy to delete blocks, and since backspacing in an empty block deletes them, this feels like a fair tradeoff to me. Compare:

screen shot 2017-08-02 at 13 52 14

screen shot 2017-08-02 at 13 52 20

(Those screenshots assume the ligthweight block hover design gets merged)

CC: @mtias @melchoyce

@jasmussen jasmussen added Design [Type] Question Questions about the design or development of the editor. labels Aug 2, 2017
@afercia
Copy link
Contributor

afercia commented Aug 2, 2017

+1 for trying this, as it would also improve the current focus order. That always puzzles me: move up, move down, cog, trash, and then the toolbar.

@jasmussen
Copy link
Contributor Author

Here's a mockup that collapses the toolbar, and adds the trash and cog to it also:

screen shot 2017-08-02 at 14 35 38

Here's a before and after for text:

before

after

@StaggerLeee
Copy link

StaggerLeee commented Aug 2, 2017

Move sorting arrows to the block toolbar too ? You can have them as separate toolbar group with few pixels margin.

I wrote once here, padding at left for all blocks, and no padding at the right is bad visual solution.
Those 2 buttons at the right just made good visual balance.

I know you can solve it for all devices. because just this padding will give You about much enough place in toolbar.

Problem is sorting arrows needed hover over, toolbar needs click on block. (Arrows toolbar group showed on hover, other toolbar buttons on click ?) Do not know how it could be solved. Know only I dont like this ugly spacing at the left of all blocks.

Maybe to follow this logic:

  • When blocks are hovered show only toolbar with sorting arrows, centered.
  • When block is clicked you can assume User IS NOT interested for sorting, but editing. Hide sorting toolbar arrow, show normal toolbar.

@karmatosed
Copy link
Member

karmatosed commented Aug 2, 2017

+1 to having this. I would caution though putting everything there. I am not convinced adding the sorting makes as much sense. My thinking is that isn't to do with the block itself as much as the placement within the document.

@mtias
Copy link
Member

mtias commented Aug 2, 2017

Thanks for exploring. I don't like bundling things in the toolbar very much (I wish we could find another place for the block switcher) as it makes the toolbar feel heavier to me, and it's not clear the level of abstractions that the various item groups reside on. As mentioned in another issue, I'd like us to focus on improving the writing interactions first before trying to make larger changes to the UI of blocks, because I think a lot of the issues at the moment come from seeing any UI when you would rather see none (because you are writing).

@jasmussen jasmussen 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 4, 2017
@jasmussen
Copy link
Contributor Author

See also discussion here: #2155 (comment)

Another option is to move the cog to the quick toolbar, and leave the trash button where it is. I agree this is lower priority than the text interactions and milestones. But it feels like it would help solve a hierarchy issue: block position and state actions on the side (and available on hover), block content options grouped in the quick toolbar.

@hedgefield
Copy link

Great ideas @jasmussen. I did some similar explorations recently - I repurposed the cog icon to open the toolbar (and the block settings in the sidebar), that will keep a lot of the toolbar clutter behind a user decision. Then I attached the toolbar to the block itself and lifted both up a bit more as a whole.

miniblock1

miniblock2

The absence of the sorting arrows is part of a different UX thing I was trying, so ignore that.

@jasmussen
Copy link
Contributor Author

Thanks for the mockup, @hedgefield. This reminds me a lot of a UI that @iseulde tried also. In the end we didn't go with that, because so long as we treat the basic elements as individual blocks (p, ul, ol, h, etc) it seems like we should be careful in making the formatting controls harder to get to.

I agree that there are some issues in timing when these appear, how they appear, and where they appear. But it seems like at the very least you should be able to shift-select something and have them appear, without having to tab to a button to invoke it.

@hedgefield
Copy link

That makes sense. I do agree the things you mention would make more of an impact than making it a user action. I've seen it mentioned a few times before, is there a dedicated place or person exploring the timing stuff somewhere?

It might go too far to make something like this an editor toggle, you know, offering people a choice of how they want it to behave, but it might be something we could investigate down the line, if we come across a feature that is problematic more for personal preferences than because we couldn't make sense of the UX for some reason.

For this, I like your last suggestion a lot. Like you say, it makes sense to separate state from content. So just the arrows and the trashcan outside the toolbar.

@jasmussen
Copy link
Contributor Author

There are a couple of PRs here with some discussion ongoing:

#2884
#2148

@jasmussen
Copy link
Contributor Author

This is done 🎉

ceyhun pushed a commit that referenced this issue Apr 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Priority] Low Used to indicate that the issue at hand isn't a top priority to address and can be handled later [Type] Question Questions about the design or development of the editor.
Projects
None yet
Development

No branches or pull requests

6 participants