-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Adds a modal version of the block inserter only in Distraction Free mode #44932
Conversation
Size Change: +280 B (0%) Total Size: 1.27 MB
ℹ️ View Unchanged
|
On refactoring:
|
b367b71
to
eaccab4
Compare
Hi @draganescu - I am a total fan of the distraction-free mode and I tested this inserter modal. Here are my first thoughts. I am a bit concerned to have the same UI asset (blue button with the cross) do different things depending on the active mode. I also have a hunch that a person being intrigued by Distraction-free mode would also prefer keyboard navigation, so they don't need to change controls (Mouse/trackpad to keyboard and back) . Using the Inserter from the Top Toolbar using-blue-inserter-distraction-free.mp4Using the Slash command. If I use the Slash command, I get there much, much faster
This is the video for using the slash command. Maybe the inserter should not be available via the Distraction-free mode at all? I would love to see a little Welcome Guide that would explain
I would even be as radical, and remove placeholder text for the new paragraph "Type / to choose a block". |
I think this approach works — it keeps the editing experience tailored for publishing instead of invoking the whole block inserter. Probably as a follow-up, but I wonder if we should surface blocks related to writing (purposely ignoring the paragraph block) such as:
|
Thinking about this more: is this a necessary addition to Distraction Free mode? I'm leaning towards not. We have the empty block inserter, and the backslash hint for empty paragraph blocks. I don't think we need to introduce another method for inserting blocks. I'd say focusing on the top toolbar improvements will yield a better experience. |
I think @mtias had a strong opinion about having the inserter available. In the meantime I'll rebase this. |
We could use the command palette (#48457) for inserting blocks when that's coming together. Adding blocks could certainly be a use case within that experience. |
Yes I had the same thinking line, I even thought this modal inserter could be the start of that command palette. |
We do have this early POC for the command palette: #49152 |
Top toolbar is improved but alas still incompatible with distraction free mode (because it still has to be rendered close to the block in the dom). |
I will close this for the idea to insert blocks from the command palette. |
What?
Continues the work in #41740.
Why?
To have a version of the inserter specific to distraction free mode. It was one of the remaining items in the original PR:
How?
Adds a modified version of the quick inserter into a modal.
Testing Instructions
Screenshots or screencast
modal-inserter-no-focus.mp4
To do
access + I
)