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

Revert "Honor the Disable Visual Editor setting (#12000)" #12151

Merged
merged 3 commits into from
Nov 21, 2018

Conversation

noisysocks
Copy link
Member

Fixes #12148 by reverting #12000.

From #12148 (comment):

I suspect that #12000 was the wrong approach here: rather than falling back to classic when rich editing is disabled, we should still load the block editor, but default to the code editor view, or perhaps a HTML block.

The fix for this should also include e2e tests confirming that CPTs that don't work in the block editor won't try to load it.

@noisysocks noisysocks added [Type] Bug An existing feature does not function as intended [Priority] High Used to indicate top priority items that need quick attention [Type] Regression Related to a regression in the latest release labels Nov 21, 2018
@gziolo
Copy link
Member

gziolo commented Nov 21, 2018

Seems like it will fix the reported issue. However, we still need to find a way to fix the original issue which we are about to revert.

@gziolo gziolo added this to the 4.6 milestone Nov 21, 2018
@youknowriad
Copy link
Contributor

I pushed a commit adding the RichText support in Gutenberg itself.

@gziolo
Copy link
Member

gziolo commented Nov 21, 2018

Code in 97ffd1b looks good, I'm testing it.

@youknowriad youknowriad modified the milestones: 4.6, 4.5.1 Nov 21, 2018
Copy link
Member

@gziolo gziolo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gooood, let's :shipit:

@youknowriad youknowriad merged commit 278b322 into master Nov 21, 2018
@youknowriad youknowriad deleted the revert/honor-the-disable-visual-editor-setting branch November 21, 2018 08:16
</div>
{ isRichEditingEnabled && (
<div className="edit-post-text-editor__toolbar">
<h2>{ __( 'Editing Code' ) }</h2>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@youknowriad Should this headline maybe still be displayed (without the button)? See #12182 for a related report.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure no strong opinion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Priority] High Used to indicate top priority items that need quick attention [Type] Bug An existing feature does not function as intended [Type] Regression Related to a regression in the latest release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Blank page when editing custom post type
4 participants