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

Prevent Classic Blocks from being converted to Shared Blocks #5941

Closed
noisysocks opened this issue Apr 3, 2018 · 11 comments
Closed

Prevent Classic Blocks from being converted to Shared Blocks #5941

noisysocks opened this issue Apr 3, 2018 · 11 comments
Assignees
Labels
[Feature] Synced Patterns Related to synced patterns (formerly reusable blocks) [Type] Bug An existing feature does not function as intended
Milestone

Comments

@noisysocks
Copy link
Member

I was watching a talk about Gutenberg at WordPress Sydney and, during a live demo, I noticed the presenter experience some confusion while attempting to migrate a pre-Gutenberg post to use Gutenberg blocks. Here's what he did:

  1. He installed the Gutenberg plugin
  2. He edited an existing post
  3. He selected the Classic block
  4. He clicked on the meatballs icon to open the More options menu:

screen shot 2018-04-03 at 13 24 01

  1. He immediately moved his cursor to point at Convert to Shared Block
  2. He paused for a moment 🤔
  3. He then slowly moved his cursor up to Convert to blocks and clicked it

It seems to me that these two menu options are worded very similarly and that this is what was caused the presenter to have his flow of thought interrupted.

Any ideas on how we could avoid this confusion?

cc. @jasmussen @karmatosed

@noisysocks noisysocks added [Type] Question Questions about the design or development of the editor. Design labels Apr 3, 2018
@jasmussen
Copy link
Contributor

Given that the classic block is set to look drastically different once #4926 lands, perhaps we can leverage that visual to change the verbiage slightly. Something in the vein of "Convert from Classic" or something. Perhaps "Update to Modern Editor"? Perhaps this ties into #4681? What is the name of our editor, if not just "Editor"? CC: @mor10 in case you have time to input as well.

@mtias
Copy link
Member

mtias commented Apr 3, 2018

What about removing the ability to make a Classic block reusable? Doesn't seem worth the added cognitive burden to me. This is supposed to be a fallback block primarily.

@jasmussen
Copy link
Contributor

What about removing the ability to make a Classic block reusable? Doesn't seem worth the added cognitive burden to me. This is supposed to be a fallback block primarily.

Yeah that makes total sense. Especially when seen together with (as mentioned), #4926 — it's less of a block and more of an instance of the old post.

@karmatosed
Copy link
Member

What about removing the ability to make a Classic block reusable?

+1 for this. Good catch @noisysocks on a trip point in the experience.

@karmatosed karmatosed removed the [Type] Question Questions about the design or development of the editor. label Apr 3, 2018
@mor10
Copy link
Contributor

mor10 commented Apr 3, 2018

Question: will this block be available to be applied to new content, or is it purely to transition "old" content in a non-breaking way?

If the former, it should be called "Traditional Editor" or something similar that relates it to typical word processing applications like MS Word from which it borrows it's toolbar. "Classic" assumes the user is familiar with the 'old' way of doing things which only applies to existing users. New users won't know what "classic" means.

If the latter, and the bolck only surfaces if you have old content, it should be called something like "Legacy Content" to make it clear it is not a block in a modern sense but a wrapper for old content.

@noisysocks
Copy link
Member Author

noisysocks commented Apr 4, 2018

What about removing the ability to make a Classic block reusable?

👌 sounds good to me. I've done this in #5970 using an approach first suggested in #4722.

If the latter, and the bolck only surfaces if you have old content, it should be called something like "Legacy Content" to make it clear it is not a block in a modern sense but a wrapper for old content.

I wouldn't mind removing the Classic block from the inserter and renaming it to be Legacy or similar. I reckon we should create a seperate issue for us to discuss this in more detail, though 🙂

@mtias
Copy link
Member

mtias commented Apr 4, 2018

@noisysocks we should still probably not make this a public API as we cannot ensure blocks are not reused through a nesting mechanism.

@noisysocks
Copy link
Member Author

@noisysocks we should still probably not make this a public API as we cannot ensure blocks are not reused through a nesting mechanism.

Good point. Do you think we should just hard code this restriction instead?

Slightly unrelated, but it feels weird that a Classic block could be nested into another block.

@karmatosed karmatosed removed the Design label Apr 24, 2018
@danielbachhuber
Copy link
Member

@noisysocks @mtias Does this issue need any further work or can it be closed?

@danielbachhuber danielbachhuber added the [Status] Needs More Info Follow-up required in order to be actionable. label May 18, 2018
@noisysocks
Copy link
Member Author

It's still an (admittedly very minor) issue. I think preventing Classic blocks from being made into a shared blocks is the right approach. I'll revive #5970 but name it supports._sharing to emphasise that it's a non-public API.

@danielbachhuber danielbachhuber removed the [Status] Needs More Info Follow-up required in order to be actionable. label Jun 27, 2018
@danielbachhuber danielbachhuber changed the title UX: Confusion between 'Convert to blocks' and 'Convert to Shared Block' Prevent Classic Blocks from being converted to Shared Blocks Jun 27, 2018
@danielbachhuber danielbachhuber added [Type] Bug An existing feature does not function as intended [Feature] Synced Patterns Related to synced patterns (formerly reusable blocks) labels Jun 27, 2018
@danielbachhuber
Copy link
Member

I've filed this issue against Merge Proposal: Editor to make sure this behavior is addressed before Gutenberg is too public.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Synced Patterns Related to synced patterns (formerly reusable blocks) [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants