Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
Already on GitHub? Sign in to your account
Editor: Try a fixed block toolbar #2148
Conversation
youknowriad
self-assigned this
Aug 2, 2017
youknowriad
requested a review
from jasmussen
Aug 2, 2017
codecov
bot
commented
Aug 2, 2017
•
Codecov Report
@@ Coverage Diff @@
## master #2148 +/- ##
==========================================
- Coverage 26.54% 26.53% -0.01%
==========================================
Files 157 157
Lines 4853 4854 +1
Branches 818 818
==========================================
Hits 1288 1288
- Misses 3012 3013 +1
Partials 553 553
Continue to review full report at Codecov.
|
|
Hah, nice, interestingly I just explored something similar with the toolbar in #2151. I like this more than I thought I would. I'd like to chew on it for a bit, and perhaps also run it by @afercia. But my gut reaction is positive. CC: @melchoyce @karmatosed Edit: we'd probably want some centering going on. |
I was going to say the same in your PR :) |
|
This also has the advantage of reducing the mental-shift when switching from the existing editor (toolbar still at the top) |
|
The primary disadvantage, potentially, could be that some of the context is lost a little bit. This could definitely affect the tie-in to customization, so pinging @melchoyce and @westonruter here. Also @mtias. |
Under the hood, this toolbar can be placed wherever we want in a different editor. For example, the customization could decide to include it above the blocks, or in a sidebar... |
jasmussen
referenced this pull request
Aug 2, 2017
Open
Consider moving cog and trash to quick toolbar #2156
|
I don't dislike this (my original concepts when we were starting had a similar concept but at the bottom). But I think it's trying to address a problem that is not of visual design, but of interaction patterns. The UI should not show when you are in "writing" mode, thus not obscuring anything. The problem is how we define "writing" mode. It sounds like Helen got to a situation where hitting "enter" was showing the UI. That is a bug and we should fix it. Hitting "enter" when you are writing should never show UI. The other problem is with mousemoves. Perhaps we should disable the functionality that shows the UI when moving the mouse, or make sure that you hover the toolbar area, or add a delay. I think we should focus on these very evident interaction shortcomings and fix them before making any call regarding the visual design. |
mtias
added the
Chrome
label
Aug 2, 2017
StaggerLeee
commented
Aug 2, 2017
|
I dont like it. All editors similar to Gutenberg show toolbar where it is now. Do not overcomplicate it. |
|
I'd tend to agree with @mtias. Maybe worth considering to better define what "writing mode" is. I'd also consider a shortcut to make the toolbar appear and receive focus on demand, see #552. The big problem I see with a fixed toolbar is keyboard interaction. Where the toolbar is placed exactly in the source order? How I move focus to the toolbar using only the keyboard? |
|
I have for a while had a lot of issues with the current toolbar because of a simple app I use called PopClip (it has a lot of things that help me interact with content, accessibility things and also just things that make life easier), this is the experience I get: Now I don't suggest we design for my not so common case, but it has always meant the toolbar was a bit of a bothersome issue for me. Flipping the thought process, I also prefer design wise having as much as possible context to actions. I can see that being lost the further things get from the content. One thing I really like that I think covers best of both worlds is a 'shadowing toolbars'. These appear where you need, when you need. A basic level of this is one that follows you around at the top, away from the blocks. A more complex is similar to what @mtias is saying where it appears just when needed. A little caution here though as to a lot of people saying what a user will or won't do, without drawing on our examples we have in tests and feedback. Maybe we can focus on the input we have right now and a solution which solves this. If we don't have enough, lets run some tests. We could also usability test some different options, we have that option. |
I'd love to see some user testing sessions include also at least: 1 keyboard user, 1 screen reader user, 1 speech recognition software user |
|
@afercia as would I. Can you help us recruit them? |
|
Worth noting Ghost use this approach but with a toolbar at the bottom. |
|
Another small advantage I'm finding here is the simplicity of the technical solution. Our actual solution is not that simple, it requires us to know if the mouse is moving, if we're typing, if we stopped typing... while this is way simpler, we only need to know the selected block. Experience shows that often the simplest solutions are the best ones. |
|
@youknowriad not sure I agree, because I'd still like the UI to not be visible when I'm typing even if it's fixed top/bottom. |
@karmatosed will ask around :) |
@youknowriad I'd like to move a lot of things at the bottom |
|
I imagine the inspector will end up being a lot more important than floating toolbars when it comes to customization and layout, with the exception of page/post content (which we might explore making editable via Gutenberg at first anyway). I do find the current implementation a little clunky, so I think @mtias's suggestions are all thing we should try, for sure. However, I also wouldn't object to us trying out a docked toolbar, maybe more like this with it fixed and centered with the content, either on the top or the bottom: (Though I do worry it could get lost on the bottom) |
This was referenced Aug 4, 2017
rianrietveld
commented
Aug 7, 2017
|
User test results #2259 |
@karmatosed @rianrietveld is going to ping you about this, she has some volunteers :) |
This was referenced Aug 8, 2017
|
What about this PR? Should we close it? I really think we should go this road (whether show a fixed toolbar at the top or the bottom, but without centring it IMO) but this is probably not a priority right now and we could revisit later. |
|
I think we should keep it open! I still like it. I don't yet know if it's a perfect fit for us. Even if we need to re-do the PR down the road if we decide to do it (because of rebases), I think it's worth having this open, as it communicates yes, we are considering this. Which we are ;) |
|
Another argument for this is that we could decide to always hide the block surroundings (borders, arrows, delete, and cog buttons) unless the block is hovered. |
|
There's also good feedback in this analysis: https://go.tinymce.com/blog/gutenberg-editor-success-lies-timing/ — which would be partially addressed by a docked toolbar. |
|
CC: @annaephox |
|
I pushed this exploration a bit further hiding the surroundings unless we hover a block. |
|
I'm still digging this, even though I also think the current toolbar can work for us. In other words, if popular opinion and tests suggest this is it, then I'm all onboard. I do think, though, that we'll probably want to always show the toolbar, even if nothing is selected. We'll want to think about what to show there instead — a tip? Seems like we should be able to solve this in a clean way. Alternately, we'd want to test whether it was sensible that the toolbar expands like to does — we might even want to add a little animation so it very quickly appears on click (like .1s appear animation). Finally, I think we'd want to center the toolbar items in the box. Like provide a 700px maxwidth to a centered container. Nice work! |


youknowriad commentedAug 2, 2017
•
edited
People are complaining about the block toolbar showing up constantly and disturbing their writing flow, this PR explores the possibility to use a fixed a block toolbar at the top. And its content change depending on the selected block.
Not done yet: