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

Meta Boxes: Don't hide disabled meta boxes by modifying DOM #12628

Merged
merged 1 commit into from
Dec 9, 2018

Conversation

noisysocks
Copy link
Member

@noisysocks noisysocks commented Dec 5, 2018

Fixes #12571.

Hiding disabled meta boxes by setting element.style.display = 'none' interferes with plugins like ACF which rely on being able to show and hide meta boxes using $.hide() and $.show().

Hiding the meta box using a new .edit-post-meta-boxes-area .is-hidden class ensures that we don't interfere with third party code.

Testing

  1. Register a meta box which hides itself using JavaScript:
<?php

add_action( 'add_meta_boxes', 'test_add_meta_boxes' );

function test_add_meta_boxes() {
	add_meta_box( 'test-meta-box', 'Test Meta Box', 'test_render_meta_box' );
}

function test_render_meta_box() {
	?>
	This should be hidden!
	<script>
	( function( $ ) {
		$( '#test-meta-box' ).hide();
	} )( jQuery );	
	</script>
	<?php
}
  1. Create a new post. The meta box should be hidden.

  2. Open Show more tools & options and select Options.

  3. Toggle the meta box off and on. The meta box should remain hidden.

I've also tested this by repeating the steps above with an additional non-hidden meta box registered, and by installing ACF and attempting to recreate the scenario described in #12571.

Hiding disabled meta boxes by setting `element.style.display = 'none'`
interferes with plugins like ACF which rely on being able to show and
hide meta boxes using `$.hide()` and `$.show()`.

Hiding the meta box using a new `.edit-post-meta-boxes-area .is-hidden`
class ensures that we don't interfere with third party code.
@noisysocks noisysocks added [Type] Bug An existing feature does not function as intended [Feature] Meta Boxes A draggable box shown on the post editing screen labels Dec 5, 2018
@noisysocks noisysocks added this to the 4.7 milestone Dec 5, 2018
@aduth
Copy link
Member

aduth commented Dec 6, 2018

One thing I notice is that there's still a box, albeit empty, where the meta box lives in the page:

image

@noisysocks
Copy link
Member Author

@aduth: Hm, that's tough to address. Any ideas?

We already have styling which will remove the border and padding on the container when there is no content. But in this case there is content—it's just that it's hidden by third party JavaScript.

The empty container is correctly not shown when meta boxes are hidden using removeEditorPanel(), toggleEditorPanel(), or remove_meta_box(). I think that, going forward, third parties should use these APIs instead of manually $.show()ing and $.hide()ing meta boxes.

@mtias
Copy link
Member

mtias commented Dec 6, 2018

I am not sure this can be made 100% smooth considering the ad hoc effect of directly hiding from the DOM will clash with the state → view model.

@aduth
Copy link
Member

aduth commented Dec 6, 2018

I was curious, and so checked how it appeared prior to #10676 (git co cbf425b~1), which is identical to how it appears now in this branch:

image

I am not sure this can be made 100% smooth considering the ad hoc effect of directly hiding from the DOM will clash with the state → view model.

I'd tend to agree. Even with these changes, there's going to be the potential for conflicts if a plugin manipulates the DOM directly. In that sense, it'd be more about getting it into the "expected" state from what had prompted #12571 to be raised as a regression.

Hm, that's tough to address. Any ideas?

My initial thought would be to reorient the styling to apply to the individual meta boxes, such that if they were hidden, there'd be no lingering artifacts. This is mostly possible, but the border in particular is hard to recreate. I'd thought applying it in a :first-child selector, but if the first child is the hidden meta box, the border wouldn't be shown at all (notably, even if other visible meta boxes are present).

There's more options if we implement it in JavaScript, iterating children to verify the presence of a visible meta box. This becomes more difficult to implement well, and as such even if intended for support, should probably be done in a follow-up pull request.

@youknowriad youknowriad merged commit 06c4dc3 into master Dec 9, 2018
@youknowriad youknowriad deleted the fix/hide-disabled-meta-boxes-using-css branch December 9, 2018 16:17
@elliotcondon
Copy link

elliotcondon commented Dec 9, 2018

@noisysocks Thanks for opening this ticket.

I think that, going forward, third parties should use these APIs instead of manually $.show()ing and $.hide()ing meta boxes.

Definitely. I'm good with this.

The empty container is correctly not shown when meta boxes are hidden using removeEditorPanel(), toggleEditorPanel(), or remove_meta_box().

This is good to know. One question: If we hide a metabox via remove_meta_box(), can we add the same metabox later via a similar add_meta_box() function?

pinarol added a commit that referenced this pull request Dec 11, 2018
* RichText: fix onSetup doc (#12607)

* Update broken links (#12660)

* Update broken links

There was 2 broken links. I changed same way with Data Module Reference page.

* Update docs/designers-developers/developers/block-api/README.md

Co-Authored-By: cagdasdag <cagdasdag81@gmail.com>

* Update docs/designers-developers/developers/block-api/README.md

Co-Authored-By: cagdasdag <cagdasdag81@gmail.com>

* Docs: Update Glossary (#12479)

* Update glossar with missing terms

* Convert glossary to use dl/dt/dd dtags. Fixes #9976

* Fix TinyMCE link

* remove spacing around tags

* Add template definition, with link

* Updates per review

* Update docs/designers-developers/glossary.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Add documentation for `safeDecodeURI()` and `filterURLForDisplay()` (#12570)

* Add documentation for `safeDecodeURI()` and `filterURLForDisplay()`

* Whitespace

* Consistant Capit… i mean capitalization

* Oxford comma

Co-Authored-By: georgeh <george@hotelling.net>

* Update theme-support.md (#12661)

* Fix: Undoing Image Selection in Image Block results in Broken Image (#12567)

* Optimize isViewportMatch (#12542)

* Cache createBlock call in isUnmodifiedDefaultBlock (#12521)

* Cache createBlock call in isUnmodifiedDefaultBlock

* Invalidate cache when default block name changes

* Merge ifs

* Font Size Picker: Use a menuitemradio role and better labels. (#12372)

* Use a menuitemradio role and better labels.

* Restore Button and remove MenuItem import.

* Use template literals.

* Set document title for preview interstitial (#12466)

* Fix e2e tests after the WordPress 5.0 upgrade (#12715)

* Fix e2e tests after the WordPress 5.0 upgrade

* Remove the php unit tests testing the WP5.0 core instead of the plugin

* Meta Boxes: Don't hide disabled meta boxes by modifying DOM (#12628)

Hiding disabled meta boxes by setting `element.style.display = 'none'`
interferes with plugins like ACF which rely on being able to show and
hide meta boxes using `$.hide()` and `$.show()`.

Hiding the meta box using a new `.edit-post-meta-boxes-area .is-hidden`
class ensures that we don't interfere with third party code.

* Add a word-wrap style to the facebook embed preview screen  (#11890)

* Add a word-break style to the facebook embed preview screen to prevent the long embed url from breaking the block boundary

* Fix typo and missing space in scss comment

* Adding @aldavigdis to the contributors list (#12686)

* Make media & text block placeholder translatable (#12706)

* Fix: Problems on Media & Text block resizing; Load wp-block-library styles before wp-edit-blocks. (#12619)

* Get wordcount type from translation (#12586)

* get wordcount type from translation

* Add description to explain the options for wordcount type

* Added mylsef into contributors.md. :)

* Only render InserterWithShortcuts on hover (#12510)

* Fix issue where default appender has icons overlaying the text (#12536)

* Fix issue where default appender has icons overlaying the text

This fixes #11425.

It adds padding to the right of the default block appender to fit 3 icons.

* chore: Tweak spelling

* Classic Block: set correct focus back after blur (#12415)

* Classic Block: set correct focus back after blur

* Add e2e test

* reset bookmark on mousedown and touchstart

* e2e: Look for aria-label="Add Media" rather than "Insert Media"

* RichText: only replace range and nodes if different (#12547)

* RichText: only set range if different

* Check rangeCount

* Also compare nodes

* Add e2e test

* Simplify

* RichText: Document isRangeEqual

* Testing: RichText: Assure subscriber removal

* Unsubscribe in page.evaluate

* Mark temporary eslint-config package as private (#12734)

* When a post is saved, check for tinymce and save any editors. (#12568)

* When a post is saved, check for tinymce and save any editors.

* Importing tinymce and using tinyMCE vs the object stored in window.tinymce.

* Updated version number and changelog.

* no longer importing tinymce since we use the tinyMCE global. tinyMCE.triggerSave works now. checking if tinyMCE exists before making the call just in case.

* Using typeof to check for tinyMCE and fixed issues brought up in travis run.

* using window.tinyMCE again to avoid warning RE undefined var

* Restore the package.json version.

* Add e2e tests for the custom wp_editor metaboxes

* Rename functions, removing gutenberg_ prefix (#12326)

* Rename functions, removing gutenberg_ and prefixing with wp_

* Remove wp_ prefix to match core

* Remove function check per review

* Annotations: Apply annotation className as string (#12741)

* RichText: Ensure instance is selected before setting back selection (#12737)

* Fix for #11663 (#12728)

* Fixed Deleting an HTML Anchor attribute leaves an empty HTML id attribute

* Fixed Deleting an HTML Anchor attribute leaves an empty

* Update plugin version to 4.7.0-rc.1 (#12752)

* Add an error state to the image block to allow upload errors to display (#10224)

* Try: JS Console warning for when in Quirks Mode (#12575)

* Try: JS Console warning for when in Quirks Mode

This PR detects whether the browser is in Quirks Mode. Quirks Mode is a rendering method used when the doctype definition is missing or incorrectly placed in the HTML source, causing the browser to have difficulty detecting the type of document it is to render.

This is usually caused by a PHP error, or even just a style tag that is output incorrectly on the page. See discussion in #12455 and #11378.

The usual result is Gutenberg rendering incorrectly, notably with metaboxes overlapping content.

The purpose of this PR is to help developers debug the issue and fix it at the root. As such, it adds a console warning, props @nickcernis for the text:

```
[Warning] Your browser is using Quirks Mode. This can cause rendering issues such as blocks overlaying meta boxes in the editor. Quirks Mode can be triggered by PHP errors or HTML code appearing before the opening <!DOCTYPE html>. Try checking the raw page source or your site's PHP error log and resolving errors there, removing any HTML before the doctype, or disabling plugins.
```

It also augments the documentation to add a note about this.

* Move warning to index.js

* Remove try/catch.

* Tweak: Remove redundant [warning] in warn call

* Organizing screenshot assets for the block tutorial inside the designers-developers directory in the repo (#12745)

* Rename backwards compatiblity to backward compatibility (#12751)

* Rename backwards compatiblity to backward compatibility

* Remove package-lock from commit

* Update CONTRIBUTING.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Update CONTRIBUTING.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Whitespace in manifest

* Update node-sass to 4.11.0 to support Node.js 11 (#12541)

## Description
Fixes #12539 by updating node-sass to support Node.js 11.

## How has this been tested?
Running `npm install` on macOS 10.14 with Node.js 11.2 without problems.

## Types of changes
Minor dependency bump to support Node.js 11.

## Checklist:
- [x] My code is tested.
- [x] My code follows the WordPress code style. <!-- Check code: `npm run lint`, Guidelines: https://make.wordpress.org/core/handbook/best-practices/coding-standards/javascript/ -->
- [x] My code follows the accessibility standards. <!-- Guidelines: https://make.wordpress.org/core/handbook/best-practices/coding-standards/accessibility-coding-standards/ -->
- [x] My code has proper inline documentation. <!-- Guidelines: https://make.wordpress.org/core/handbook/best-practices/inline-documentation-standards/javascript/ -->
Tug added a commit that referenced this pull request Dec 13, 2018
* RichText: fix onSetup doc (#12607)

* Update broken links (#12660)

* Update broken links

There was 2 broken links. I changed same way with Data Module Reference page.

* Update docs/designers-developers/developers/block-api/README.md

Co-Authored-By: cagdasdag <cagdasdag81@gmail.com>

* Update docs/designers-developers/developers/block-api/README.md

Co-Authored-By: cagdasdag <cagdasdag81@gmail.com>

* Docs: Update Glossary (#12479)

* Update glossar with missing terms

* Convert glossary to use dl/dt/dd dtags. Fixes #9976

* Fix TinyMCE link

* remove spacing around tags

* Add template definition, with link

* Updates per review

* Update docs/designers-developers/glossary.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Add documentation for `safeDecodeURI()` and `filterURLForDisplay()` (#12570)

* Add documentation for `safeDecodeURI()` and `filterURLForDisplay()`

* Whitespace

* Consistant Capit… i mean capitalization

* Oxford comma

Co-Authored-By: georgeh <george@hotelling.net>

* Update theme-support.md (#12661)

* Fix: Undoing Image Selection in Image Block results in Broken Image (#12567)

* Optimize isViewportMatch (#12542)

* Cache createBlock call in isUnmodifiedDefaultBlock (#12521)

* Cache createBlock call in isUnmodifiedDefaultBlock

* Invalidate cache when default block name changes

* Merge ifs

* Font Size Picker: Use a menuitemradio role and better labels. (#12372)

* Use a menuitemradio role and better labels.

* Restore Button and remove MenuItem import.

* Use template literals.

* Set document title for preview interstitial (#12466)

* Fix e2e tests after the WordPress 5.0 upgrade (#12715)

* Fix e2e tests after the WordPress 5.0 upgrade

* Remove the php unit tests testing the WP5.0 core instead of the plugin

* Meta Boxes: Don't hide disabled meta boxes by modifying DOM (#12628)

Hiding disabled meta boxes by setting `element.style.display = 'none'`
interferes with plugins like ACF which rely on being able to show and
hide meta boxes using `$.hide()` and `$.show()`.

Hiding the meta box using a new `.edit-post-meta-boxes-area .is-hidden`
class ensures that we don't interfere with third party code.

* Add a word-wrap style to the facebook embed preview screen  (#11890)

* Add a word-break style to the facebook embed preview screen to prevent the long embed url from breaking the block boundary

* Fix typo and missing space in scss comment

* Adding @aldavigdis to the contributors list (#12686)

* Make media & text block placeholder translatable (#12706)

* Fix: Problems on Media & Text block resizing; Load wp-block-library styles before wp-edit-blocks. (#12619)

* Get wordcount type from translation (#12586)

* get wordcount type from translation

* Add description to explain the options for wordcount type

* Added mylsef into contributors.md. :)

* Only render InserterWithShortcuts on hover (#12510)

* Fix issue where default appender has icons overlaying the text (#12536)

* Fix issue where default appender has icons overlaying the text

This fixes #11425.

It adds padding to the right of the default block appender to fit 3 icons.

* chore: Tweak spelling

* Classic Block: set correct focus back after blur (#12415)

* Classic Block: set correct focus back after blur

* Add e2e test

* reset bookmark on mousedown and touchstart

* e2e: Look for aria-label="Add Media" rather than "Insert Media"

* RichText: only replace range and nodes if different (#12547)

* RichText: only set range if different

* Check rangeCount

* Also compare nodes

* Add e2e test

* Simplify

* RichText: Document isRangeEqual

* Testing: RichText: Assure subscriber removal

* Unsubscribe in page.evaluate

* Mark temporary eslint-config package as private (#12734)

* When a post is saved, check for tinymce and save any editors. (#12568)

* When a post is saved, check for tinymce and save any editors.

* Importing tinymce and using tinyMCE vs the object stored in window.tinymce.

* Updated version number and changelog.

* no longer importing tinymce since we use the tinyMCE global. tinyMCE.triggerSave works now. checking if tinyMCE exists before making the call just in case.

* Using typeof to check for tinyMCE and fixed issues brought up in travis run.

* using window.tinyMCE again to avoid warning RE undefined var

* Restore the package.json version.

* Add e2e tests for the custom wp_editor metaboxes

* Rename functions, removing gutenberg_ prefix (#12326)

* Rename functions, removing gutenberg_ and prefixing with wp_

* Remove wp_ prefix to match core

* Remove function check per review

* Annotations: Apply annotation className as string (#12741)

* RichText: Ensure instance is selected before setting back selection (#12737)

* Fix for #11663 (#12728)

* Fixed Deleting an HTML Anchor attribute leaves an empty HTML id attribute

* Fixed Deleting an HTML Anchor attribute leaves an empty

* Update plugin version to 4.7.0-rc.1 (#12752)

* Add an error state to the image block to allow upload errors to display (#10224)

* Try: JS Console warning for when in Quirks Mode (#12575)

* Try: JS Console warning for when in Quirks Mode

This PR detects whether the browser is in Quirks Mode. Quirks Mode is a rendering method used when the doctype definition is missing or incorrectly placed in the HTML source, causing the browser to have difficulty detecting the type of document it is to render.

This is usually caused by a PHP error, or even just a style tag that is output incorrectly on the page. See discussion in #12455 and #11378.

The usual result is Gutenberg rendering incorrectly, notably with metaboxes overlapping content.

The purpose of this PR is to help developers debug the issue and fix it at the root. As such, it adds a console warning, props @nickcernis for the text:

```
[Warning] Your browser is using Quirks Mode. This can cause rendering issues such as blocks overlaying meta boxes in the editor. Quirks Mode can be triggered by PHP errors or HTML code appearing before the opening <!DOCTYPE html>. Try checking the raw page source or your site's PHP error log and resolving errors there, removing any HTML before the doctype, or disabling plugins.
```

It also augments the documentation to add a note about this.

* Move warning to index.js

* Remove try/catch.

* Tweak: Remove redundant [warning] in warn call

* Organizing screenshot assets for the block tutorial inside the designers-developers directory in the repo (#12745)

* Rename backwards compatiblity to backward compatibility (#12751)

* Rename backwards compatiblity to backward compatibility

* Remove package-lock from commit

* Update CONTRIBUTING.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Update CONTRIBUTING.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Whitespace in manifest

* Update node-sass to 4.11.0 to support Node.js 11 (#12541)

## Description
Fixes #12539 by updating node-sass to support Node.js 11.

## How has this been tested?
Running `npm install` on macOS 10.14 with Node.js 11.2 without problems.

## Types of changes
Minor dependency bump to support Node.js 11.

## Checklist:
- [x] My code is tested.
- [x] My code follows the WordPress code style. <!-- Check code: `npm run lint`, Guidelines: https://make.wordpress.org/core/handbook/best-practices/coding-standards/javascript/ -->
- [x] My code follows the accessibility standards. <!-- Guidelines: https://make.wordpress.org/core/handbook/best-practices/coding-standards/accessibility-coding-standards/ -->
- [x] My code has proper inline documentation. <!-- Guidelines: https://make.wordpress.org/core/handbook/best-practices/inline-documentation-standards/javascript/ -->

* Add attributes to ServerSideRender readme (#12793)

* Add attributes to ServerSideRender readme

Adds a code example demonstrating how to define attributes when registering a block that will use attributes in a ServerSideRender component.

* Add whitespace and inline code markup to ServerSideRender readme

Implements requested changes from code review.

* Scripts: Add check-engines script to the package (#12721)

* Scripts: Add check-engines script to the package

* Update packages/scripts/CHANGELOG.md

Co-Authored-By: gziolo <grzegorz@gziolo.pl>

* Update packages/scripts/README.md

Co-Authored-By: gziolo <grzegorz@gziolo.pl>

* Update minimal node version to 10.x

Co-Authored-By: gziolo <grzegorz@gziolo.pl>

* Move devDependencies to root package.json file (#12720)

* Chore: Remove unused npm dependencies from the root package.json file

* Move devDependencies to root package.json file

* Fix php notice from the recent comments block (#12812)

* RichText: Fix React warning shown when unmounting a currently selected RichText. (#12817)

* Packages: Reimplement ESLint config as plugin (#12763)

* Packages: Move eslint-config to eslint-plugin

(Fails pre-commit, but in effort to ensure history preservation)

* eslint-plugin: Add npmrc to avoid package-lock.json

* Framework: Update path references for eslint-config to -plugin

* eslint-plugin: Reimplement ESLint config as plugin

* eslint-plugin: Unmark as private

* eslint-plugin: Undocument custom ruleset

* 4.7 (#12819)

* Bump plugin version to 4.7.0

* chore(release): publish

 - @wordpress/annotations@1.0.4
 - @wordpress/api-fetch@2.2.6
 - @wordpress/block-library@2.2.10
 - @wordpress/block-serialization-default-parser@2.0.2
 - @wordpress/block-serialization-spec-parser@2.0.2
 - @wordpress/blocks@6.0.4
 - @wordpress/components@7.0.4
 - @wordpress/core-data@2.0.15
 - @wordpress/data@4.1.0
 - @wordpress/date@3.0.1
 - @wordpress/edit-post@3.1.5
 - @wordpress/editor@9.0.5
 - @wordpress/eslint-plugin@1.0.0
 - @wordpress/format-library@1.2.8
 - @wordpress/html-entities@2.0.4
 - @wordpress/list-reusable-blocks@1.1.17
 - @wordpress/notices@1.1.1
 - @wordpress/nux@3.0.5
 - @wordpress/rich-text@3.0.3
 - @wordpress/url@2.3.2
 - @wordpress/viewport@2.0.13

* Update changelogs after 4.7 package releases

* Add back package-lock.json
@noisysocks
Copy link
Member Author

This is good to know. One question: If we hide a metabox via remove_meta_box(), can we add the same metabox later via a similar add_meta_box() function?

I think that that's supported, but I'd test it to make sure 🙂

youknowriad pushed a commit that referenced this pull request Dec 19, 2018
* Make a simple version of DefaultBlockAppender for mobile (#12434)

* Make a simple version of DefaultBlockAppender for mobile

* Use the same padding used for other blocks in DefaultBlockAppender

* Update copy, auto focus and bind keypress

* Do not bind key events

* Change style of placeholder

* Stop using classname-to-style autotransform in react native (#12552)

* [RNMobile] Fix crash editing More blocks (#12620)

* Use onChange instead of onChangeText in PlainText updates.

* Convert the More block to Component, and re-use the same logic of the web when the field is empty.

* Remove unsed variable, and format code

* Move `);` to a new line

* Fix SVG styles for mobile (#12608)

* Fix SVG styles for mobile

* Simplify condition since className is always a string

* Check if Enter.key is the last inserted character, and add a new default block after the current More block. (#12639)

Note: This is detected after the fact, and the newline could be visible on the block for a very small time. This is OK for the alpha, we will revisit the logic later.
See wordpress-mobile/gutenberg-mobile#324

* Call blur when deselecting RichText or PlainText component. (#12765)

This is required to hide the keyboard when the focus moves to a non textual block.

* Merge master into mobile (#12796)

* RichText: fix onSetup doc (#12607)

* Update broken links (#12660)

* Update broken links

There was 2 broken links. I changed same way with Data Module Reference page.

* Update docs/designers-developers/developers/block-api/README.md

Co-Authored-By: cagdasdag <cagdasdag81@gmail.com>

* Update docs/designers-developers/developers/block-api/README.md

Co-Authored-By: cagdasdag <cagdasdag81@gmail.com>

* Docs: Update Glossary (#12479)

* Update glossar with missing terms

* Convert glossary to use dl/dt/dd dtags. Fixes #9976

* Fix TinyMCE link

* remove spacing around tags

* Add template definition, with link

* Updates per review

* Update docs/designers-developers/glossary.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Add documentation for `safeDecodeURI()` and `filterURLForDisplay()` (#12570)

* Add documentation for `safeDecodeURI()` and `filterURLForDisplay()`

* Whitespace

* Consistant Capit… i mean capitalization

* Oxford comma

Co-Authored-By: georgeh <george@hotelling.net>

* Update theme-support.md (#12661)

* Fix: Undoing Image Selection in Image Block results in Broken Image (#12567)

* Optimize isViewportMatch (#12542)

* Cache createBlock call in isUnmodifiedDefaultBlock (#12521)

* Cache createBlock call in isUnmodifiedDefaultBlock

* Invalidate cache when default block name changes

* Merge ifs

* Font Size Picker: Use a menuitemradio role and better labels. (#12372)

* Use a menuitemradio role and better labels.

* Restore Button and remove MenuItem import.

* Use template literals.

* Set document title for preview interstitial (#12466)

* Fix e2e tests after the WordPress 5.0 upgrade (#12715)

* Fix e2e tests after the WordPress 5.0 upgrade

* Remove the php unit tests testing the WP5.0 core instead of the plugin

* Meta Boxes: Don't hide disabled meta boxes by modifying DOM (#12628)

Hiding disabled meta boxes by setting `element.style.display = 'none'`
interferes with plugins like ACF which rely on being able to show and
hide meta boxes using `$.hide()` and `$.show()`.

Hiding the meta box using a new `.edit-post-meta-boxes-area .is-hidden`
class ensures that we don't interfere with third party code.

* Add a word-wrap style to the facebook embed preview screen  (#11890)

* Add a word-break style to the facebook embed preview screen to prevent the long embed url from breaking the block boundary

* Fix typo and missing space in scss comment

* Adding @aldavigdis to the contributors list (#12686)

* Make media & text block placeholder translatable (#12706)

* Fix: Problems on Media & Text block resizing; Load wp-block-library styles before wp-edit-blocks. (#12619)

* Get wordcount type from translation (#12586)

* get wordcount type from translation

* Add description to explain the options for wordcount type

* Added mylsef into contributors.md. :)

* Only render InserterWithShortcuts on hover (#12510)

* Fix issue where default appender has icons overlaying the text (#12536)

* Fix issue where default appender has icons overlaying the text

This fixes #11425.

It adds padding to the right of the default block appender to fit 3 icons.

* chore: Tweak spelling

* Classic Block: set correct focus back after blur (#12415)

* Classic Block: set correct focus back after blur

* Add e2e test

* reset bookmark on mousedown and touchstart

* e2e: Look for aria-label="Add Media" rather than "Insert Media"

* RichText: only replace range and nodes if different (#12547)

* RichText: only set range if different

* Check rangeCount

* Also compare nodes

* Add e2e test

* Simplify

* RichText: Document isRangeEqual

* Testing: RichText: Assure subscriber removal

* Unsubscribe in page.evaluate

* Mark temporary eslint-config package as private (#12734)

* When a post is saved, check for tinymce and save any editors. (#12568)

* When a post is saved, check for tinymce and save any editors.

* Importing tinymce and using tinyMCE vs the object stored in window.tinymce.

* Updated version number and changelog.

* no longer importing tinymce since we use the tinyMCE global. tinyMCE.triggerSave works now. checking if tinyMCE exists before making the call just in case.

* Using typeof to check for tinyMCE and fixed issues brought up in travis run.

* using window.tinyMCE again to avoid warning RE undefined var

* Restore the package.json version.

* Add e2e tests for the custom wp_editor metaboxes

* Rename functions, removing gutenberg_ prefix (#12326)

* Rename functions, removing gutenberg_ and prefixing with wp_

* Remove wp_ prefix to match core

* Remove function check per review

* Annotations: Apply annotation className as string (#12741)

* RichText: Ensure instance is selected before setting back selection (#12737)

* Fix for #11663 (#12728)

* Fixed Deleting an HTML Anchor attribute leaves an empty HTML id attribute

* Fixed Deleting an HTML Anchor attribute leaves an empty

* Update plugin version to 4.7.0-rc.1 (#12752)

* Add an error state to the image block to allow upload errors to display (#10224)

* Try: JS Console warning for when in Quirks Mode (#12575)

* Try: JS Console warning for when in Quirks Mode

This PR detects whether the browser is in Quirks Mode. Quirks Mode is a rendering method used when the doctype definition is missing or incorrectly placed in the HTML source, causing the browser to have difficulty detecting the type of document it is to render.

This is usually caused by a PHP error, or even just a style tag that is output incorrectly on the page. See discussion in #12455 and #11378.

The usual result is Gutenberg rendering incorrectly, notably with metaboxes overlapping content.

The purpose of this PR is to help developers debug the issue and fix it at the root. As such, it adds a console warning, props @nickcernis for the text:

```
[Warning] Your browser is using Quirks Mode. This can cause rendering issues such as blocks overlaying meta boxes in the editor. Quirks Mode can be triggered by PHP errors or HTML code appearing before the opening <!DOCTYPE html>. Try checking the raw page source or your site's PHP error log and resolving errors there, removing any HTML before the doctype, or disabling plugins.
```

It also augments the documentation to add a note about this.

* Move warning to index.js

* Remove try/catch.

* Tweak: Remove redundant [warning] in warn call

* Organizing screenshot assets for the block tutorial inside the designers-developers directory in the repo (#12745)

* Rename backwards compatiblity to backward compatibility (#12751)

* Rename backwards compatiblity to backward compatibility

* Remove package-lock from commit

* Update CONTRIBUTING.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Update CONTRIBUTING.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Whitespace in manifest

* Update node-sass to 4.11.0 to support Node.js 11 (#12541)

## Description
Fixes #12539 by updating node-sass to support Node.js 11.

## How has this been tested?
Running `npm install` on macOS 10.14 with Node.js 11.2 without problems.

## Types of changes
Minor dependency bump to support Node.js 11.

## Checklist:
- [x] My code is tested.
- [x] My code follows the WordPress code style. <!-- Check code: `npm run lint`, Guidelines: https://make.wordpress.org/core/handbook/best-practices/coding-standards/javascript/ -->
- [x] My code follows the accessibility standards. <!-- Guidelines: https://make.wordpress.org/core/handbook/best-practices/coding-standards/accessibility-coding-standards/ -->
- [x] My code has proper inline documentation. <!-- Guidelines: https://make.wordpress.org/core/handbook/best-practices/inline-documentation-standards/javascript/ -->

* Update native more block styling (#12767)

* Update styling of more block

* Fix CI test

* Update spaces

* Convert indentation to tabs

* Fix lint issues

* Remove key attribute

* Set fixed width to the text input

* Make sure to properly compare empty text and undefined text variable before resetting the eventCount field. (#12815)

* Use the default "Read More" text on init only, giving the ability to user to empty the field and re-start with empty text (#12821)

* Merge 'origin/master' into 'origin/mobile' (#12836)

* RichText: fix onSetup doc (#12607)

* Update broken links (#12660)

* Update broken links

There was 2 broken links. I changed same way with Data Module Reference page.

* Update docs/designers-developers/developers/block-api/README.md

Co-Authored-By: cagdasdag <cagdasdag81@gmail.com>

* Update docs/designers-developers/developers/block-api/README.md

Co-Authored-By: cagdasdag <cagdasdag81@gmail.com>

* Docs: Update Glossary (#12479)

* Update glossar with missing terms

* Convert glossary to use dl/dt/dd dtags. Fixes #9976

* Fix TinyMCE link

* remove spacing around tags

* Add template definition, with link

* Updates per review

* Update docs/designers-developers/glossary.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Add documentation for `safeDecodeURI()` and `filterURLForDisplay()` (#12570)

* Add documentation for `safeDecodeURI()` and `filterURLForDisplay()`

* Whitespace

* Consistant Capit… i mean capitalization

* Oxford comma

Co-Authored-By: georgeh <george@hotelling.net>

* Update theme-support.md (#12661)

* Fix: Undoing Image Selection in Image Block results in Broken Image (#12567)

* Optimize isViewportMatch (#12542)

* Cache createBlock call in isUnmodifiedDefaultBlock (#12521)

* Cache createBlock call in isUnmodifiedDefaultBlock

* Invalidate cache when default block name changes

* Merge ifs

* Font Size Picker: Use a menuitemradio role and better labels. (#12372)

* Use a menuitemradio role and better labels.

* Restore Button and remove MenuItem import.

* Use template literals.

* Set document title for preview interstitial (#12466)

* Fix e2e tests after the WordPress 5.0 upgrade (#12715)

* Fix e2e tests after the WordPress 5.0 upgrade

* Remove the php unit tests testing the WP5.0 core instead of the plugin

* Meta Boxes: Don't hide disabled meta boxes by modifying DOM (#12628)

Hiding disabled meta boxes by setting `element.style.display = 'none'`
interferes with plugins like ACF which rely on being able to show and
hide meta boxes using `$.hide()` and `$.show()`.

Hiding the meta box using a new `.edit-post-meta-boxes-area .is-hidden`
class ensures that we don't interfere with third party code.

* Add a word-wrap style to the facebook embed preview screen  (#11890)

* Add a word-break style to the facebook embed preview screen to prevent the long embed url from breaking the block boundary

* Fix typo and missing space in scss comment

* Adding @aldavigdis to the contributors list (#12686)

* Make media & text block placeholder translatable (#12706)

* Fix: Problems on Media & Text block resizing; Load wp-block-library styles before wp-edit-blocks. (#12619)

* Get wordcount type from translation (#12586)

* get wordcount type from translation

* Add description to explain the options for wordcount type

* Added mylsef into contributors.md. :)

* Only render InserterWithShortcuts on hover (#12510)

* Fix issue where default appender has icons overlaying the text (#12536)

* Fix issue where default appender has icons overlaying the text

This fixes #11425.

It adds padding to the right of the default block appender to fit 3 icons.

* chore: Tweak spelling

* Classic Block: set correct focus back after blur (#12415)

* Classic Block: set correct focus back after blur

* Add e2e test

* reset bookmark on mousedown and touchstart

* e2e: Look for aria-label="Add Media" rather than "Insert Media"

* RichText: only replace range and nodes if different (#12547)

* RichText: only set range if different

* Check rangeCount

* Also compare nodes

* Add e2e test

* Simplify

* RichText: Document isRangeEqual

* Testing: RichText: Assure subscriber removal

* Unsubscribe in page.evaluate

* Mark temporary eslint-config package as private (#12734)

* When a post is saved, check for tinymce and save any editors. (#12568)

* When a post is saved, check for tinymce and save any editors.

* Importing tinymce and using tinyMCE vs the object stored in window.tinymce.

* Updated version number and changelog.

* no longer importing tinymce since we use the tinyMCE global. tinyMCE.triggerSave works now. checking if tinyMCE exists before making the call just in case.

* Using typeof to check for tinyMCE and fixed issues brought up in travis run.

* using window.tinyMCE again to avoid warning RE undefined var

* Restore the package.json version.

* Add e2e tests for the custom wp_editor metaboxes

* Rename functions, removing gutenberg_ prefix (#12326)

* Rename functions, removing gutenberg_ and prefixing with wp_

* Remove wp_ prefix to match core

* Remove function check per review

* Annotations: Apply annotation className as string (#12741)

* RichText: Ensure instance is selected before setting back selection (#12737)

* Fix for #11663 (#12728)

* Fixed Deleting an HTML Anchor attribute leaves an empty HTML id attribute

* Fixed Deleting an HTML Anchor attribute leaves an empty

* Update plugin version to 4.7.0-rc.1 (#12752)

* Add an error state to the image block to allow upload errors to display (#10224)

* Try: JS Console warning for when in Quirks Mode (#12575)

* Try: JS Console warning for when in Quirks Mode

This PR detects whether the browser is in Quirks Mode. Quirks Mode is a rendering method used when the doctype definition is missing or incorrectly placed in the HTML source, causing the browser to have difficulty detecting the type of document it is to render.

This is usually caused by a PHP error, or even just a style tag that is output incorrectly on the page. See discussion in #12455 and #11378.

The usual result is Gutenberg rendering incorrectly, notably with metaboxes overlapping content.

The purpose of this PR is to help developers debug the issue and fix it at the root. As such, it adds a console warning, props @nickcernis for the text:

```
[Warning] Your browser is using Quirks Mode. This can cause rendering issues such as blocks overlaying meta boxes in the editor. Quirks Mode can be triggered by PHP errors or HTML code appearing before the opening <!DOCTYPE html>. Try checking the raw page source or your site's PHP error log and resolving errors there, removing any HTML before the doctype, or disabling plugins.
```

It also augments the documentation to add a note about this.

* Move warning to index.js

* Remove try/catch.

* Tweak: Remove redundant [warning] in warn call

* Organizing screenshot assets for the block tutorial inside the designers-developers directory in the repo (#12745)

* Rename backwards compatiblity to backward compatibility (#12751)

* Rename backwards compatiblity to backward compatibility

* Remove package-lock from commit

* Update CONTRIBUTING.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Update CONTRIBUTING.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Whitespace in manifest

* Update node-sass to 4.11.0 to support Node.js 11 (#12541)

## Description
Fixes #12539 by updating node-sass to support Node.js 11.

## How has this been tested?
Running `npm install` on macOS 10.14 with Node.js 11.2 without problems.

## Types of changes
Minor dependency bump to support Node.js 11.

## Checklist:
- [x] My code is tested.
- [x] My code follows the WordPress code style. <!-- Check code: `npm run lint`, Guidelines: https://make.wordpress.org/core/handbook/best-practices/coding-standards/javascript/ -->
- [x] My code follows the accessibility standards. <!-- Guidelines: https://make.wordpress.org/core/handbook/best-practices/coding-standards/accessibility-coding-standards/ -->
- [x] My code has proper inline documentation. <!-- Guidelines: https://make.wordpress.org/core/handbook/best-practices/inline-documentation-standards/javascript/ -->

* Add attributes to ServerSideRender readme (#12793)

* Add attributes to ServerSideRender readme

Adds a code example demonstrating how to define attributes when registering a block that will use attributes in a ServerSideRender component.

* Add whitespace and inline code markup to ServerSideRender readme

Implements requested changes from code review.

* Scripts: Add check-engines script to the package (#12721)

* Scripts: Add check-engines script to the package

* Update packages/scripts/CHANGELOG.md

Co-Authored-By: gziolo <grzegorz@gziolo.pl>

* Update packages/scripts/README.md

Co-Authored-By: gziolo <grzegorz@gziolo.pl>

* Update minimal node version to 10.x

Co-Authored-By: gziolo <grzegorz@gziolo.pl>

* Move devDependencies to root package.json file (#12720)

* Chore: Remove unused npm dependencies from the root package.json file

* Move devDependencies to root package.json file

* Fix php notice from the recent comments block (#12812)

* RichText: Fix React warning shown when unmounting a currently selected RichText. (#12817)

* Packages: Reimplement ESLint config as plugin (#12763)

* Packages: Move eslint-config to eslint-plugin

(Fails pre-commit, but in effort to ensure history preservation)

* eslint-plugin: Add npmrc to avoid package-lock.json

* Framework: Update path references for eslint-config to -plugin

* eslint-plugin: Reimplement ESLint config as plugin

* eslint-plugin: Unmark as private

* eslint-plugin: Undocument custom ruleset

* 4.7 (#12819)

* Bump plugin version to 4.7.0

* chore(release): publish

 - @wordpress/annotations@1.0.4
 - @wordpress/api-fetch@2.2.6
 - @wordpress/block-library@2.2.10
 - @wordpress/block-serialization-default-parser@2.0.2
 - @wordpress/block-serialization-spec-parser@2.0.2
 - @wordpress/blocks@6.0.4
 - @wordpress/components@7.0.4
 - @wordpress/core-data@2.0.15
 - @wordpress/data@4.1.0
 - @wordpress/date@3.0.1
 - @wordpress/edit-post@3.1.5
 - @wordpress/editor@9.0.5
 - @wordpress/eslint-plugin@1.0.0
 - @wordpress/format-library@1.2.8
 - @wordpress/html-entities@2.0.4
 - @wordpress/list-reusable-blocks@1.1.17
 - @wordpress/notices@1.1.1
 - @wordpress/nux@3.0.5
 - @wordpress/rich-text@3.0.3
 - @wordpress/url@2.3.2
 - @wordpress/viewport@2.0.13

* Update changelogs after 4.7 package releases

* Add back package-lock.json

* Remove the call that does `blur` the undelying native component when deselecting RichText or PlainText. (#12886)

* [rnmobile]: Send blockType prop to RNAztecView (#12869)

* Revert "Remove the call that does `blur` the undelying native component (#12946)

* Revert "Remove the call that does `blur` the undelying native component when deselecting RichText or PlainText. (#12886)"

This reverts commit d9fe45e.

* Blur only if it is iOS

* Fix imports

* Fix lint errors in the mobile branch (#12990)

* Revert changes to lib/load.php

* Fix lint errors in More

* Fix lint error for PlainText

* Fix lint error in RichText
youknowriad pushed a commit that referenced this pull request Jan 9, 2019
* Make a simple version of DefaultBlockAppender for mobile (#12434)

* Make a simple version of DefaultBlockAppender for mobile

* Use the same padding used for other blocks in DefaultBlockAppender

* Update copy, auto focus and bind keypress

* Do not bind key events

* Change style of placeholder

* Stop using classname-to-style autotransform in react native (#12552)

* [RNMobile] Fix crash editing More blocks (#12620)

* Use onChange instead of onChangeText in PlainText updates.

* Convert the More block to Component, and re-use the same logic of the web when the field is empty.

* Remove unsed variable, and format code

* Move `);` to a new line

* Fix SVG styles for mobile (#12608)

* Fix SVG styles for mobile

* Simplify condition since className is always a string

* Check if Enter.key is the last inserted character, and add a new default block after the current More block. (#12639)

Note: This is detected after the fact, and the newline could be visible on the block for a very small time. This is OK for the alpha, we will revisit the logic later.
See wordpress-mobile/gutenberg-mobile#324

* Call blur when deselecting RichText or PlainText component. (#12765)

This is required to hide the keyboard when the focus moves to a non textual block.

* Merge master into mobile (#12796)

* RichText: fix onSetup doc (#12607)

* Update broken links (#12660)

* Update broken links

There was 2 broken links. I changed same way with Data Module Reference page.

* Update docs/designers-developers/developers/block-api/README.md

Co-Authored-By: cagdasdag <cagdasdag81@gmail.com>

* Update docs/designers-developers/developers/block-api/README.md

Co-Authored-By: cagdasdag <cagdasdag81@gmail.com>

* Docs: Update Glossary (#12479)

* Update glossar with missing terms

* Convert glossary to use dl/dt/dd dtags. Fixes #9976

* Fix TinyMCE link

* remove spacing around tags

* Add template definition, with link

* Updates per review

* Update docs/designers-developers/glossary.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Add documentation for `safeDecodeURI()` and `filterURLForDisplay()` (#12570)

* Add documentation for `safeDecodeURI()` and `filterURLForDisplay()`

* Whitespace

* Consistant Capit… i mean capitalization

* Oxford comma

Co-Authored-By: georgeh <george@hotelling.net>

* Update theme-support.md (#12661)

* Fix: Undoing Image Selection in Image Block results in Broken Image (#12567)

* Optimize isViewportMatch (#12542)

* Cache createBlock call in isUnmodifiedDefaultBlock (#12521)

* Cache createBlock call in isUnmodifiedDefaultBlock

* Invalidate cache when default block name changes

* Merge ifs

* Font Size Picker: Use a menuitemradio role and better labels. (#12372)

* Use a menuitemradio role and better labels.

* Restore Button and remove MenuItem import.

* Use template literals.

* Set document title for preview interstitial (#12466)

* Fix e2e tests after the WordPress 5.0 upgrade (#12715)

* Fix e2e tests after the WordPress 5.0 upgrade

* Remove the php unit tests testing the WP5.0 core instead of the plugin

* Meta Boxes: Don't hide disabled meta boxes by modifying DOM (#12628)

Hiding disabled meta boxes by setting `element.style.display = 'none'`
interferes with plugins like ACF which rely on being able to show and
hide meta boxes using `$.hide()` and `$.show()`.

Hiding the meta box using a new `.edit-post-meta-boxes-area .is-hidden`
class ensures that we don't interfere with third party code.

* Add a word-wrap style to the facebook embed preview screen  (#11890)

* Add a word-break style to the facebook embed preview screen to prevent the long embed url from breaking the block boundary

* Fix typo and missing space in scss comment

* Adding @aldavigdis to the contributors list (#12686)

* Make media & text block placeholder translatable (#12706)

* Fix: Problems on Media & Text block resizing; Load wp-block-library styles before wp-edit-blocks. (#12619)

* Get wordcount type from translation (#12586)

* get wordcount type from translation

* Add description to explain the options for wordcount type

* Added mylsef into contributors.md. :)

* Only render InserterWithShortcuts on hover (#12510)

* Fix issue where default appender has icons overlaying the text (#12536)

* Fix issue where default appender has icons overlaying the text

This fixes #11425.

It adds padding to the right of the default block appender to fit 3 icons.

* chore: Tweak spelling

* Classic Block: set correct focus back after blur (#12415)

* Classic Block: set correct focus back after blur

* Add e2e test

* reset bookmark on mousedown and touchstart

* e2e: Look for aria-label="Add Media" rather than "Insert Media"

* RichText: only replace range and nodes if different (#12547)

* RichText: only set range if different

* Check rangeCount

* Also compare nodes

* Add e2e test

* Simplify

* RichText: Document isRangeEqual

* Testing: RichText: Assure subscriber removal

* Unsubscribe in page.evaluate

* Mark temporary eslint-config package as private (#12734)

* When a post is saved, check for tinymce and save any editors. (#12568)

* When a post is saved, check for tinymce and save any editors.

* Importing tinymce and using tinyMCE vs the object stored in window.tinymce.

* Updated version number and changelog.

* no longer importing tinymce since we use the tinyMCE global. tinyMCE.triggerSave works now. checking if tinyMCE exists before making the call just in case.

* Using typeof to check for tinyMCE and fixed issues brought up in travis run.

* using window.tinyMCE again to avoid warning RE undefined var

* Restore the package.json version.

* Add e2e tests for the custom wp_editor metaboxes

* Rename functions, removing gutenberg_ prefix (#12326)

* Rename functions, removing gutenberg_ and prefixing with wp_

* Remove wp_ prefix to match core

* Remove function check per review

* Annotations: Apply annotation className as string (#12741)

* RichText: Ensure instance is selected before setting back selection (#12737)

* Fix for #11663 (#12728)

* Fixed Deleting an HTML Anchor attribute leaves an empty HTML id attribute

* Fixed Deleting an HTML Anchor attribute leaves an empty

* Update plugin version to 4.7.0-rc.1 (#12752)

* Add an error state to the image block to allow upload errors to display (#10224)

* Try: JS Console warning for when in Quirks Mode (#12575)

* Try: JS Console warning for when in Quirks Mode

This PR detects whether the browser is in Quirks Mode. Quirks Mode is a rendering method used when the doctype definition is missing or incorrectly placed in the HTML source, causing the browser to have difficulty detecting the type of document it is to render.

This is usually caused by a PHP error, or even just a style tag that is output incorrectly on the page. See discussion in #12455 and #11378.

The usual result is Gutenberg rendering incorrectly, notably with metaboxes overlapping content.

The purpose of this PR is to help developers debug the issue and fix it at the root. As such, it adds a console warning, props @nickcernis for the text:

```
[Warning] Your browser is using Quirks Mode. This can cause rendering issues such as blocks overlaying meta boxes in the editor. Quirks Mode can be triggered by PHP errors or HTML code appearing before the opening <!DOCTYPE html>. Try checking the raw page source or your site's PHP error log and resolving errors there, removing any HTML before the doctype, or disabling plugins.
```

It also augments the documentation to add a note about this.

* Move warning to index.js

* Remove try/catch.

* Tweak: Remove redundant [warning] in warn call

* Organizing screenshot assets for the block tutorial inside the designers-developers directory in the repo (#12745)

* Rename backwards compatiblity to backward compatibility (#12751)

* Rename backwards compatiblity to backward compatibility

* Remove package-lock from commit

* Update CONTRIBUTING.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Update CONTRIBUTING.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Whitespace in manifest

* Update node-sass to 4.11.0 to support Node.js 11 (#12541)

## Description
Fixes #12539 by updating node-sass to support Node.js 11.

## How has this been tested?
Running `npm install` on macOS 10.14 with Node.js 11.2 without problems.

## Types of changes
Minor dependency bump to support Node.js 11.

## Checklist:
- [x] My code is tested.
- [x] My code follows the WordPress code style. <!-- Check code: `npm run lint`, Guidelines: https://make.wordpress.org/core/handbook/best-practices/coding-standards/javascript/ -->
- [x] My code follows the accessibility standards. <!-- Guidelines: https://make.wordpress.org/core/handbook/best-practices/coding-standards/accessibility-coding-standards/ -->
- [x] My code has proper inline documentation. <!-- Guidelines: https://make.wordpress.org/core/handbook/best-practices/inline-documentation-standards/javascript/ -->

* Update native more block styling (#12767)

* Update styling of more block

* Fix CI test

* Update spaces

* Convert indentation to tabs

* Fix lint issues

* Remove key attribute

* Set fixed width to the text input

* Make sure to properly compare empty text and undefined text variable before resetting the eventCount field. (#12815)

* Use the default "Read More" text on init only, giving the ability to user to empty the field and re-start with empty text (#12821)

* Merge 'origin/master' into 'origin/mobile' (#12836)

* RichText: fix onSetup doc (#12607)

* Update broken links (#12660)

* Update broken links

There was 2 broken links. I changed same way with Data Module Reference page.

* Update docs/designers-developers/developers/block-api/README.md

Co-Authored-By: cagdasdag <cagdasdag81@gmail.com>

* Update docs/designers-developers/developers/block-api/README.md

Co-Authored-By: cagdasdag <cagdasdag81@gmail.com>

* Docs: Update Glossary (#12479)

* Update glossar with missing terms

* Convert glossary to use dl/dt/dd dtags. Fixes #9976

* Fix TinyMCE link

* remove spacing around tags

* Add template definition, with link

* Updates per review

* Update docs/designers-developers/glossary.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Add documentation for `safeDecodeURI()` and `filterURLForDisplay()` (#12570)

* Add documentation for `safeDecodeURI()` and `filterURLForDisplay()`

* Whitespace

* Consistant Capit… i mean capitalization

* Oxford comma

Co-Authored-By: georgeh <george@hotelling.net>

* Update theme-support.md (#12661)

* Fix: Undoing Image Selection in Image Block results in Broken Image (#12567)

* Optimize isViewportMatch (#12542)

* Cache createBlock call in isUnmodifiedDefaultBlock (#12521)

* Cache createBlock call in isUnmodifiedDefaultBlock

* Invalidate cache when default block name changes

* Merge ifs

* Font Size Picker: Use a menuitemradio role and better labels. (#12372)

* Use a menuitemradio role and better labels.

* Restore Button and remove MenuItem import.

* Use template literals.

* Set document title for preview interstitial (#12466)

* Fix e2e tests after the WordPress 5.0 upgrade (#12715)

* Fix e2e tests after the WordPress 5.0 upgrade

* Remove the php unit tests testing the WP5.0 core instead of the plugin

* Meta Boxes: Don't hide disabled meta boxes by modifying DOM (#12628)

Hiding disabled meta boxes by setting `element.style.display = 'none'`
interferes with plugins like ACF which rely on being able to show and
hide meta boxes using `$.hide()` and `$.show()`.

Hiding the meta box using a new `.edit-post-meta-boxes-area .is-hidden`
class ensures that we don't interfere with third party code.

* Add a word-wrap style to the facebook embed preview screen  (#11890)

* Add a word-break style to the facebook embed preview screen to prevent the long embed url from breaking the block boundary

* Fix typo and missing space in scss comment

* Adding @aldavigdis to the contributors list (#12686)

* Make media & text block placeholder translatable (#12706)

* Fix: Problems on Media & Text block resizing; Load wp-block-library styles before wp-edit-blocks. (#12619)

* Get wordcount type from translation (#12586)

* get wordcount type from translation

* Add description to explain the options for wordcount type

* Added mylsef into contributors.md. :)

* Only render InserterWithShortcuts on hover (#12510)

* Fix issue where default appender has icons overlaying the text (#12536)

* Fix issue where default appender has icons overlaying the text

This fixes #11425.

It adds padding to the right of the default block appender to fit 3 icons.

* chore: Tweak spelling

* Classic Block: set correct focus back after blur (#12415)

* Classic Block: set correct focus back after blur

* Add e2e test

* reset bookmark on mousedown and touchstart

* e2e: Look for aria-label="Add Media" rather than "Insert Media"

* RichText: only replace range and nodes if different (#12547)

* RichText: only set range if different

* Check rangeCount

* Also compare nodes

* Add e2e test

* Simplify

* RichText: Document isRangeEqual

* Testing: RichText: Assure subscriber removal

* Unsubscribe in page.evaluate

* Mark temporary eslint-config package as private (#12734)

* When a post is saved, check for tinymce and save any editors. (#12568)

* When a post is saved, check for tinymce and save any editors.

* Importing tinymce and using tinyMCE vs the object stored in window.tinymce.

* Updated version number and changelog.

* no longer importing tinymce since we use the tinyMCE global. tinyMCE.triggerSave works now. checking if tinyMCE exists before making the call just in case.

* Using typeof to check for tinyMCE and fixed issues brought up in travis run.

* using window.tinyMCE again to avoid warning RE undefined var

* Restore the package.json version.

* Add e2e tests for the custom wp_editor metaboxes

* Rename functions, removing gutenberg_ prefix (#12326)

* Rename functions, removing gutenberg_ and prefixing with wp_

* Remove wp_ prefix to match core

* Remove function check per review

* Annotations: Apply annotation className as string (#12741)

* RichText: Ensure instance is selected before setting back selection (#12737)

* Fix for #11663 (#12728)

* Fixed Deleting an HTML Anchor attribute leaves an empty HTML id attribute

* Fixed Deleting an HTML Anchor attribute leaves an empty

* Update plugin version to 4.7.0-rc.1 (#12752)

* Add an error state to the image block to allow upload errors to display (#10224)

* Try: JS Console warning for when in Quirks Mode (#12575)

* Try: JS Console warning for when in Quirks Mode

This PR detects whether the browser is in Quirks Mode. Quirks Mode is a rendering method used when the doctype definition is missing or incorrectly placed in the HTML source, causing the browser to have difficulty detecting the type of document it is to render.

This is usually caused by a PHP error, or even just a style tag that is output incorrectly on the page. See discussion in #12455 and #11378.

The usual result is Gutenberg rendering incorrectly, notably with metaboxes overlapping content.

The purpose of this PR is to help developers debug the issue and fix it at the root. As such, it adds a console warning, props @nickcernis for the text:

```
[Warning] Your browser is using Quirks Mode. This can cause rendering issues such as blocks overlaying meta boxes in the editor. Quirks Mode can be triggered by PHP errors or HTML code appearing before the opening <!DOCTYPE html>. Try checking the raw page source or your site's PHP error log and resolving errors there, removing any HTML before the doctype, or disabling plugins.
```

It also augments the documentation to add a note about this.

* Move warning to index.js

* Remove try/catch.

* Tweak: Remove redundant [warning] in warn call

* Organizing screenshot assets for the block tutorial inside the designers-developers directory in the repo (#12745)

* Rename backwards compatiblity to backward compatibility (#12751)

* Rename backwards compatiblity to backward compatibility

* Remove package-lock from commit

* Update CONTRIBUTING.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Update CONTRIBUTING.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Whitespace in manifest

* Update node-sass to 4.11.0 to support Node.js 11 (#12541)

## Description
Fixes #12539 by updating node-sass to support Node.js 11.

## How has this been tested?
Running `npm install` on macOS 10.14 with Node.js 11.2 without problems.

## Types of changes
Minor dependency bump to support Node.js 11.

## Checklist:
- [x] My code is tested.
- [x] My code follows the WordPress code style. <!-- Check code: `npm run lint`, Guidelines: https://make.wordpress.org/core/handbook/best-practices/coding-standards/javascript/ -->
- [x] My code follows the accessibility standards. <!-- Guidelines: https://make.wordpress.org/core/handbook/best-practices/coding-standards/accessibility-coding-standards/ -->
- [x] My code has proper inline documentation. <!-- Guidelines: https://make.wordpress.org/core/handbook/best-practices/inline-documentation-standards/javascript/ -->

* Add attributes to ServerSideRender readme (#12793)

* Add attributes to ServerSideRender readme

Adds a code example demonstrating how to define attributes when registering a block that will use attributes in a ServerSideRender component.

* Add whitespace and inline code markup to ServerSideRender readme

Implements requested changes from code review.

* Scripts: Add check-engines script to the package (#12721)

* Scripts: Add check-engines script to the package

* Update packages/scripts/CHANGELOG.md

Co-Authored-By: gziolo <grzegorz@gziolo.pl>

* Update packages/scripts/README.md

Co-Authored-By: gziolo <grzegorz@gziolo.pl>

* Update minimal node version to 10.x

Co-Authored-By: gziolo <grzegorz@gziolo.pl>

* Move devDependencies to root package.json file (#12720)

* Chore: Remove unused npm dependencies from the root package.json file

* Move devDependencies to root package.json file

* Fix php notice from the recent comments block (#12812)

* RichText: Fix React warning shown when unmounting a currently selected RichText. (#12817)

* Packages: Reimplement ESLint config as plugin (#12763)

* Packages: Move eslint-config to eslint-plugin

(Fails pre-commit, but in effort to ensure history preservation)

* eslint-plugin: Add npmrc to avoid package-lock.json

* Framework: Update path references for eslint-config to -plugin

* eslint-plugin: Reimplement ESLint config as plugin

* eslint-plugin: Unmark as private

* eslint-plugin: Undocument custom ruleset

* 4.7 (#12819)

* Bump plugin version to 4.7.0

* chore(release): publish

 - @wordpress/annotations@1.0.4
 - @wordpress/api-fetch@2.2.6
 - @wordpress/block-library@2.2.10
 - @wordpress/block-serialization-default-parser@2.0.2
 - @wordpress/block-serialization-spec-parser@2.0.2
 - @wordpress/blocks@6.0.4
 - @wordpress/components@7.0.4
 - @wordpress/core-data@2.0.15
 - @wordpress/data@4.1.0
 - @wordpress/date@3.0.1
 - @wordpress/edit-post@3.1.5
 - @wordpress/editor@9.0.5
 - @wordpress/eslint-plugin@1.0.0
 - @wordpress/format-library@1.2.8
 - @wordpress/html-entities@2.0.4
 - @wordpress/list-reusable-blocks@1.1.17
 - @wordpress/notices@1.1.1
 - @wordpress/nux@3.0.5
 - @wordpress/rich-text@3.0.3
 - @wordpress/url@2.3.2
 - @wordpress/viewport@2.0.13

* Update changelogs after 4.7 package releases

* Add back package-lock.json

* Remove the call that does `blur` the undelying native component when deselecting RichText or PlainText. (#12886)

* [rnmobile]: Send blockType prop to RNAztecView (#12869)

* Revert "Remove the call that does `blur` the undelying native component (#12946)

* Revert "Remove the call that does `blur` the undelying native component when deselecting RichText or PlainText. (#12886)"

This reverts commit d9fe45e.

* Blur only if it is iOS

* Fix imports

* Fix lint errors in the mobile branch (#12990)

* Revert changes to lib/load.php

* Fix lint errors in More

* Fix lint error for PlainText

* Fix lint error in RichText
youknowriad pushed a commit that referenced this pull request Jan 9, 2019
* Make a simple version of DefaultBlockAppender for mobile (#12434)

* Make a simple version of DefaultBlockAppender for mobile

* Use the same padding used for other blocks in DefaultBlockAppender

* Update copy, auto focus and bind keypress

* Do not bind key events

* Change style of placeholder

* Stop using classname-to-style autotransform in react native (#12552)

* [RNMobile] Fix crash editing More blocks (#12620)

* Use onChange instead of onChangeText in PlainText updates.

* Convert the More block to Component, and re-use the same logic of the web when the field is empty.

* Remove unsed variable, and format code

* Move `);` to a new line

* Fix SVG styles for mobile (#12608)

* Fix SVG styles for mobile

* Simplify condition since className is always a string

* Check if Enter.key is the last inserted character, and add a new default block after the current More block. (#12639)

Note: This is detected after the fact, and the newline could be visible on the block for a very small time. This is OK for the alpha, we will revisit the logic later.
See wordpress-mobile/gutenberg-mobile#324

* Call blur when deselecting RichText or PlainText component. (#12765)

This is required to hide the keyboard when the focus moves to a non textual block.

* Merge master into mobile (#12796)

* RichText: fix onSetup doc (#12607)

* Update broken links (#12660)

* Update broken links

There was 2 broken links. I changed same way with Data Module Reference page.

* Update docs/designers-developers/developers/block-api/README.md

Co-Authored-By: cagdasdag <cagdasdag81@gmail.com>

* Update docs/designers-developers/developers/block-api/README.md

Co-Authored-By: cagdasdag <cagdasdag81@gmail.com>

* Docs: Update Glossary (#12479)

* Update glossar with missing terms

* Convert glossary to use dl/dt/dd dtags. Fixes #9976

* Fix TinyMCE link

* remove spacing around tags

* Add template definition, with link

* Updates per review

* Update docs/designers-developers/glossary.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Add documentation for `safeDecodeURI()` and `filterURLForDisplay()` (#12570)

* Add documentation for `safeDecodeURI()` and `filterURLForDisplay()`

* Whitespace

* Consistant Capit… i mean capitalization

* Oxford comma

Co-Authored-By: georgeh <george@hotelling.net>

* Update theme-support.md (#12661)

* Fix: Undoing Image Selection in Image Block results in Broken Image (#12567)

* Optimize isViewportMatch (#12542)

* Cache createBlock call in isUnmodifiedDefaultBlock (#12521)

* Cache createBlock call in isUnmodifiedDefaultBlock

* Invalidate cache when default block name changes

* Merge ifs

* Font Size Picker: Use a menuitemradio role and better labels. (#12372)

* Use a menuitemradio role and better labels.

* Restore Button and remove MenuItem import.

* Use template literals.

* Set document title for preview interstitial (#12466)

* Fix e2e tests after the WordPress 5.0 upgrade (#12715)

* Fix e2e tests after the WordPress 5.0 upgrade

* Remove the php unit tests testing the WP5.0 core instead of the plugin

* Meta Boxes: Don't hide disabled meta boxes by modifying DOM (#12628)

Hiding disabled meta boxes by setting `element.style.display = 'none'`
interferes with plugins like ACF which rely on being able to show and
hide meta boxes using `$.hide()` and `$.show()`.

Hiding the meta box using a new `.edit-post-meta-boxes-area .is-hidden`
class ensures that we don't interfere with third party code.

* Add a word-wrap style to the facebook embed preview screen  (#11890)

* Add a word-break style to the facebook embed preview screen to prevent the long embed url from breaking the block boundary

* Fix typo and missing space in scss comment

* Adding @aldavigdis to the contributors list (#12686)

* Make media & text block placeholder translatable (#12706)

* Fix: Problems on Media & Text block resizing; Load wp-block-library styles before wp-edit-blocks. (#12619)

* Get wordcount type from translation (#12586)

* get wordcount type from translation

* Add description to explain the options for wordcount type

* Added mylsef into contributors.md. :)

* Only render InserterWithShortcuts on hover (#12510)

* Fix issue where default appender has icons overlaying the text (#12536)

* Fix issue where default appender has icons overlaying the text

This fixes #11425.

It adds padding to the right of the default block appender to fit 3 icons.

* chore: Tweak spelling

* Classic Block: set correct focus back after blur (#12415)

* Classic Block: set correct focus back after blur

* Add e2e test

* reset bookmark on mousedown and touchstart

* e2e: Look for aria-label="Add Media" rather than "Insert Media"

* RichText: only replace range and nodes if different (#12547)

* RichText: only set range if different

* Check rangeCount

* Also compare nodes

* Add e2e test

* Simplify

* RichText: Document isRangeEqual

* Testing: RichText: Assure subscriber removal

* Unsubscribe in page.evaluate

* Mark temporary eslint-config package as private (#12734)

* When a post is saved, check for tinymce and save any editors. (#12568)

* When a post is saved, check for tinymce and save any editors.

* Importing tinymce and using tinyMCE vs the object stored in window.tinymce.

* Updated version number and changelog.

* no longer importing tinymce since we use the tinyMCE global. tinyMCE.triggerSave works now. checking if tinyMCE exists before making the call just in case.

* Using typeof to check for tinyMCE and fixed issues brought up in travis run.

* using window.tinyMCE again to avoid warning RE undefined var

* Restore the package.json version.

* Add e2e tests for the custom wp_editor metaboxes

* Rename functions, removing gutenberg_ prefix (#12326)

* Rename functions, removing gutenberg_ and prefixing with wp_

* Remove wp_ prefix to match core

* Remove function check per review

* Annotations: Apply annotation className as string (#12741)

* RichText: Ensure instance is selected before setting back selection (#12737)

* Fix for #11663 (#12728)

* Fixed Deleting an HTML Anchor attribute leaves an empty HTML id attribute

* Fixed Deleting an HTML Anchor attribute leaves an empty

* Update plugin version to 4.7.0-rc.1 (#12752)

* Add an error state to the image block to allow upload errors to display (#10224)

* Try: JS Console warning for when in Quirks Mode (#12575)

* Try: JS Console warning for when in Quirks Mode

This PR detects whether the browser is in Quirks Mode. Quirks Mode is a rendering method used when the doctype definition is missing or incorrectly placed in the HTML source, causing the browser to have difficulty detecting the type of document it is to render.

This is usually caused by a PHP error, or even just a style tag that is output incorrectly on the page. See discussion in #12455 and #11378.

The usual result is Gutenberg rendering incorrectly, notably with metaboxes overlapping content.

The purpose of this PR is to help developers debug the issue and fix it at the root. As such, it adds a console warning, props @nickcernis for the text:

```
[Warning] Your browser is using Quirks Mode. This can cause rendering issues such as blocks overlaying meta boxes in the editor. Quirks Mode can be triggered by PHP errors or HTML code appearing before the opening <!DOCTYPE html>. Try checking the raw page source or your site's PHP error log and resolving errors there, removing any HTML before the doctype, or disabling plugins.
```

It also augments the documentation to add a note about this.

* Move warning to index.js

* Remove try/catch.

* Tweak: Remove redundant [warning] in warn call

* Organizing screenshot assets for the block tutorial inside the designers-developers directory in the repo (#12745)

* Rename backwards compatiblity to backward compatibility (#12751)

* Rename backwards compatiblity to backward compatibility

* Remove package-lock from commit

* Update CONTRIBUTING.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Update CONTRIBUTING.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Whitespace in manifest

* Update node-sass to 4.11.0 to support Node.js 11 (#12541)

## Description
Fixes #12539 by updating node-sass to support Node.js 11.

## How has this been tested?
Running `npm install` on macOS 10.14 with Node.js 11.2 without problems.

## Types of changes
Minor dependency bump to support Node.js 11.

## Checklist:
- [x] My code is tested.
- [x] My code follows the WordPress code style. <!-- Check code: `npm run lint`, Guidelines: https://make.wordpress.org/core/handbook/best-practices/coding-standards/javascript/ -->
- [x] My code follows the accessibility standards. <!-- Guidelines: https://make.wordpress.org/core/handbook/best-practices/coding-standards/accessibility-coding-standards/ -->
- [x] My code has proper inline documentation. <!-- Guidelines: https://make.wordpress.org/core/handbook/best-practices/inline-documentation-standards/javascript/ -->

* Update native more block styling (#12767)

* Update styling of more block

* Fix CI test

* Update spaces

* Convert indentation to tabs

* Fix lint issues

* Remove key attribute

* Set fixed width to the text input

* Make sure to properly compare empty text and undefined text variable before resetting the eventCount field. (#12815)

* Use the default "Read More" text on init only, giving the ability to user to empty the field and re-start with empty text (#12821)

* Merge 'origin/master' into 'origin/mobile' (#12836)

* RichText: fix onSetup doc (#12607)

* Update broken links (#12660)

* Update broken links

There was 2 broken links. I changed same way with Data Module Reference page.

* Update docs/designers-developers/developers/block-api/README.md

Co-Authored-By: cagdasdag <cagdasdag81@gmail.com>

* Update docs/designers-developers/developers/block-api/README.md

Co-Authored-By: cagdasdag <cagdasdag81@gmail.com>

* Docs: Update Glossary (#12479)

* Update glossar with missing terms

* Convert glossary to use dl/dt/dd dtags. Fixes #9976

* Fix TinyMCE link

* remove spacing around tags

* Add template definition, with link

* Updates per review

* Update docs/designers-developers/glossary.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Add documentation for `safeDecodeURI()` and `filterURLForDisplay()` (#12570)

* Add documentation for `safeDecodeURI()` and `filterURLForDisplay()`

* Whitespace

* Consistant Capit… i mean capitalization

* Oxford comma

Co-Authored-By: georgeh <george@hotelling.net>

* Update theme-support.md (#12661)

* Fix: Undoing Image Selection in Image Block results in Broken Image (#12567)

* Optimize isViewportMatch (#12542)

* Cache createBlock call in isUnmodifiedDefaultBlock (#12521)

* Cache createBlock call in isUnmodifiedDefaultBlock

* Invalidate cache when default block name changes

* Merge ifs

* Font Size Picker: Use a menuitemradio role and better labels. (#12372)

* Use a menuitemradio role and better labels.

* Restore Button and remove MenuItem import.

* Use template literals.

* Set document title for preview interstitial (#12466)

* Fix e2e tests after the WordPress 5.0 upgrade (#12715)

* Fix e2e tests after the WordPress 5.0 upgrade

* Remove the php unit tests testing the WP5.0 core instead of the plugin

* Meta Boxes: Don't hide disabled meta boxes by modifying DOM (#12628)

Hiding disabled meta boxes by setting `element.style.display = 'none'`
interferes with plugins like ACF which rely on being able to show and
hide meta boxes using `$.hide()` and `$.show()`.

Hiding the meta box using a new `.edit-post-meta-boxes-area .is-hidden`
class ensures that we don't interfere with third party code.

* Add a word-wrap style to the facebook embed preview screen  (#11890)

* Add a word-break style to the facebook embed preview screen to prevent the long embed url from breaking the block boundary

* Fix typo and missing space in scss comment

* Adding @aldavigdis to the contributors list (#12686)

* Make media & text block placeholder translatable (#12706)

* Fix: Problems on Media & Text block resizing; Load wp-block-library styles before wp-edit-blocks. (#12619)

* Get wordcount type from translation (#12586)

* get wordcount type from translation

* Add description to explain the options for wordcount type

* Added mylsef into contributors.md. :)

* Only render InserterWithShortcuts on hover (#12510)

* Fix issue where default appender has icons overlaying the text (#12536)

* Fix issue where default appender has icons overlaying the text

This fixes #11425.

It adds padding to the right of the default block appender to fit 3 icons.

* chore: Tweak spelling

* Classic Block: set correct focus back after blur (#12415)

* Classic Block: set correct focus back after blur

* Add e2e test

* reset bookmark on mousedown and touchstart

* e2e: Look for aria-label="Add Media" rather than "Insert Media"

* RichText: only replace range and nodes if different (#12547)

* RichText: only set range if different

* Check rangeCount

* Also compare nodes

* Add e2e test

* Simplify

* RichText: Document isRangeEqual

* Testing: RichText: Assure subscriber removal

* Unsubscribe in page.evaluate

* Mark temporary eslint-config package as private (#12734)

* When a post is saved, check for tinymce and save any editors. (#12568)

* When a post is saved, check for tinymce and save any editors.

* Importing tinymce and using tinyMCE vs the object stored in window.tinymce.

* Updated version number and changelog.

* no longer importing tinymce since we use the tinyMCE global. tinyMCE.triggerSave works now. checking if tinyMCE exists before making the call just in case.

* Using typeof to check for tinyMCE and fixed issues brought up in travis run.

* using window.tinyMCE again to avoid warning RE undefined var

* Restore the package.json version.

* Add e2e tests for the custom wp_editor metaboxes

* Rename functions, removing gutenberg_ prefix (#12326)

* Rename functions, removing gutenberg_ and prefixing with wp_

* Remove wp_ prefix to match core

* Remove function check per review

* Annotations: Apply annotation className as string (#12741)

* RichText: Ensure instance is selected before setting back selection (#12737)

* Fix for #11663 (#12728)

* Fixed Deleting an HTML Anchor attribute leaves an empty HTML id attribute

* Fixed Deleting an HTML Anchor attribute leaves an empty

* Update plugin version to 4.7.0-rc.1 (#12752)

* Add an error state to the image block to allow upload errors to display (#10224)

* Try: JS Console warning for when in Quirks Mode (#12575)

* Try: JS Console warning for when in Quirks Mode

This PR detects whether the browser is in Quirks Mode. Quirks Mode is a rendering method used when the doctype definition is missing or incorrectly placed in the HTML source, causing the browser to have difficulty detecting the type of document it is to render.

This is usually caused by a PHP error, or even just a style tag that is output incorrectly on the page. See discussion in #12455 and #11378.

The usual result is Gutenberg rendering incorrectly, notably with metaboxes overlapping content.

The purpose of this PR is to help developers debug the issue and fix it at the root. As such, it adds a console warning, props @nickcernis for the text:

```
[Warning] Your browser is using Quirks Mode. This can cause rendering issues such as blocks overlaying meta boxes in the editor. Quirks Mode can be triggered by PHP errors or HTML code appearing before the opening <!DOCTYPE html>. Try checking the raw page source or your site's PHP error log and resolving errors there, removing any HTML before the doctype, or disabling plugins.
```

It also augments the documentation to add a note about this.

* Move warning to index.js

* Remove try/catch.

* Tweak: Remove redundant [warning] in warn call

* Organizing screenshot assets for the block tutorial inside the designers-developers directory in the repo (#12745)

* Rename backwards compatiblity to backward compatibility (#12751)

* Rename backwards compatiblity to backward compatibility

* Remove package-lock from commit

* Update CONTRIBUTING.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Update CONTRIBUTING.md

Co-Authored-By: mkaz <marcus@mkaz.com>

* Whitespace in manifest

* Update node-sass to 4.11.0 to support Node.js 11 (#12541)

## Description
Fixes #12539 by updating node-sass to support Node.js 11.

## How has this been tested?
Running `npm install` on macOS 10.14 with Node.js 11.2 without problems.

## Types of changes
Minor dependency bump to support Node.js 11.

## Checklist:
- [x] My code is tested.
- [x] My code follows the WordPress code style. <!-- Check code: `npm run lint`, Guidelines: https://make.wordpress.org/core/handbook/best-practices/coding-standards/javascript/ -->
- [x] My code follows the accessibility standards. <!-- Guidelines: https://make.wordpress.org/core/handbook/best-practices/coding-standards/accessibility-coding-standards/ -->
- [x] My code has proper inline documentation. <!-- Guidelines: https://make.wordpress.org/core/handbook/best-practices/inline-documentation-standards/javascript/ -->

* Add attributes to ServerSideRender readme (#12793)

* Add attributes to ServerSideRender readme

Adds a code example demonstrating how to define attributes when registering a block that will use attributes in a ServerSideRender component.

* Add whitespace and inline code markup to ServerSideRender readme

Implements requested changes from code review.

* Scripts: Add check-engines script to the package (#12721)

* Scripts: Add check-engines script to the package

* Update packages/scripts/CHANGELOG.md

Co-Authored-By: gziolo <grzegorz@gziolo.pl>

* Update packages/scripts/README.md

Co-Authored-By: gziolo <grzegorz@gziolo.pl>

* Update minimal node version to 10.x

Co-Authored-By: gziolo <grzegorz@gziolo.pl>

* Move devDependencies to root package.json file (#12720)

* Chore: Remove unused npm dependencies from the root package.json file

* Move devDependencies to root package.json file

* Fix php notice from the recent comments block (#12812)

* RichText: Fix React warning shown when unmounting a currently selected RichText. (#12817)

* Packages: Reimplement ESLint config as plugin (#12763)

* Packages: Move eslint-config to eslint-plugin

(Fails pre-commit, but in effort to ensure history preservation)

* eslint-plugin: Add npmrc to avoid package-lock.json

* Framework: Update path references for eslint-config to -plugin

* eslint-plugin: Reimplement ESLint config as plugin

* eslint-plugin: Unmark as private

* eslint-plugin: Undocument custom ruleset

* 4.7 (#12819)

* Bump plugin version to 4.7.0

* chore(release): publish

 - @wordpress/annotations@1.0.4
 - @wordpress/api-fetch@2.2.6
 - @wordpress/block-library@2.2.10
 - @wordpress/block-serialization-default-parser@2.0.2
 - @wordpress/block-serialization-spec-parser@2.0.2
 - @wordpress/blocks@6.0.4
 - @wordpress/components@7.0.4
 - @wordpress/core-data@2.0.15
 - @wordpress/data@4.1.0
 - @wordpress/date@3.0.1
 - @wordpress/edit-post@3.1.5
 - @wordpress/editor@9.0.5
 - @wordpress/eslint-plugin@1.0.0
 - @wordpress/format-library@1.2.8
 - @wordpress/html-entities@2.0.4
 - @wordpress/list-reusable-blocks@1.1.17
 - @wordpress/notices@1.1.1
 - @wordpress/nux@3.0.5
 - @wordpress/rich-text@3.0.3
 - @wordpress/url@2.3.2
 - @wordpress/viewport@2.0.13

* Update changelogs after 4.7 package releases

* Add back package-lock.json

* Remove the call that does `blur` the undelying native component when deselecting RichText or PlainText. (#12886)

* [rnmobile]: Send blockType prop to RNAztecView (#12869)

* Revert "Remove the call that does `blur` the undelying native component (#12946)

* Revert "Remove the call that does `blur` the undelying native component when deselecting RichText or PlainText. (#12886)"

This reverts commit d9fe45e.

* Blur only if it is iOS

* Fix imports

* Fix lint errors in the mobile branch (#12990)

* Revert changes to lib/load.php

* Fix lint errors in More

* Fix lint error for PlainText

* Fix lint error in RichText
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Meta Boxes A draggable box shown on the post editing screen [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants