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

RichText: collapse toolbar #14233

Merged
merged 11 commits into from Mar 8, 2019

Conversation

@ellatrix
Copy link
Member

commented Mar 5, 2019

Description

This PR solves two problems:

  • Currently, all rich text controls that are added by plugins are added to the block toolbar. This can get quickly out of hand, breaking the block toolbar. The solution would be to put overflowing buttons in a dropdown. I decided to cut this after Bold, Italic and Link, which are by far the most used buttons.
  • Moves the inline image button from the block inserter to the rich text drop down. The block inserter doesn't seem to be a good place for the inline image button. Lots of people have reported that they cannot find the button, and this makes sense as it doesn't really belong in the block inserter, but better with inline tools.

Fixes #13762.

Block toolbar:

screenshot 2019-03-05 at 10 29 17

The arrow down is the same as the one in the block switcher and alignment switcher, so it doesn't feel too much.

Rich text dropdown:

screenshot 2019-03-05 at 10 51 56

This is the same kind of menu as the alignment dropdown:

screenshot 2019-03-05 at 10 29 36

How has this been tested?

Screenshots

Types of changes

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.
@jasmussen

This comment has been minimized.

Copy link
Contributor

commented Mar 5, 2019

Wow, this is really nice! GIF:

dropdown

It's not the magical wonderous responsiveness that we've been dreaming of, where you scale the block toolbar by width and stuff happens magically inside. But it's a solid step in the right direction. Your argument about making the cutoff at the "strikethrough" is solid to me as well — and honestly I feel this paves the way for us to add text color and inline text highlight tools to core as well. And finally it doesn't block even better responsiveness tools to land in the future.

I pushed a small fix to balance the margins so the dropdown is centered when hovered:

screenshot 2019-03-05 at 11 16 36

Can we add a tooltip as well? Accessibility wise it seems good to me, but it might be nice to get a sanity check here as well.

Nice work!

@ellatrix

This comment has been minimized.

Copy link
Member Author

commented Mar 5, 2019

@jasmussen Thanks for the polish! I added a tooltip. :)

@ellatrix ellatrix requested review from nerrad and ntwb as code owners Mar 5, 2019

@gziolo

This comment has been minimized.

Copy link
Member

commented Mar 5, 2019

There are tests failing:

FAIL packages/e2e-tests/specs/plugins/format-api.test.js (10.174s)

It seems like they need to be updated to work with this newly introduced dropdown menu.

<Slot name="RichText.ToolbarControls" />
<Slot name="RichText.ToolbarControls">
{ ( fills ) => fills.length &&
<DropdownMenu

This comment has been minimized.

Copy link
@youknowriad

youknowriad Mar 5, 2019

Contributor

In terms of design, any button added to the block toolbar should ideally be wrapped in <Toolbar>. There are many examples of this in the code base and it gives the right aria-roles, design...

This comment has been minimized.

Copy link
@gziolo

gziolo Mar 8, 2019

Member

I think we are using:

{ ( fills ) => ! isEmpty( fills ) && (

in other places. I don't remember why. Ideally, we should be using fills.length.

This comment has been minimized.

Copy link
@gziolo

gziolo Mar 8, 2019

Member

Okey, I double checked our code and it's safe to use fills.length. We should default to it moving on 👍

Intrernally we use map from lodash which ensures it returns an array:
https://github.com/WordPress/gutenberg/blob/master/packages/components/src/slot-fill/slot.js#L66

screen shot 2019-03-08 at 11 14 51

@ellatrix

This comment has been minimized.

Copy link
Member Author

commented Mar 5, 2019

@youknowriad Could you point me to such an example? The ones I see have the menu role, and the items are menu items. This dropdown is just like the alignment dropdown.

@youknowriad

This comment has been minimized.

Copy link
Contributor

commented Mar 5, 2019

The block alignment and the alignment toolbars both use the Toolbar component, The block switcher also uses a Toolbar component with a button inside it.

IIRC just wrapping any button in a Toolbar component gives it the right design for the block toolbar. The Toolbar component also has an isCollapsed flag, you could force to true. Is this not enough for the use-case here?

@ellatrix

This comment has been minimized.

Copy link
Member Author

commented Mar 5, 2019

@youknowriad I still don't understand. It has the right design? I've attached some screenshots above. The button is also wrapped in a toolbar:

screenshot 2019-03-05 at 17 07 51

@ellatrix

This comment has been minimized.

Copy link
Member Author

commented Mar 5, 2019

<Toolbar>
{ controls.map( ( format ) =>
<Slot name={ `RichText.ToolbarControls.${ format }` } key={ format } />
) }
<Slot name="RichText.ToolbarControls">
{ ( fills ) => fills.length &&
<DropdownMenu

@youknowriad

This comment has been minimized.

Copy link
Contributor

commented Mar 5, 2019

mmm Right, I missed that it was already inside the formatting toolbar. I guess what bothers me is the width of the button as it's not similar to the other ones.

@ellatrix

This comment has been minimized.

Copy link
Member Author

commented Mar 5, 2019

I like that it's smaller. It's also in line with the icons for the block switcher and alignment switcher.

@iamthomasbishop

This comment has been minimized.

Copy link

commented Mar 6, 2019

Thanks for the work on this thus far! I love that we're thinking about how we can accommodate smaller screens, and I like the ideas y'all are exploring. I have a one tiny bit of drive-by feedback with regards to mobile.

Beyond the personal/visual preference of the dropdown icon feeling too small (which I agree with @youknowriad on), it appears that the button doesn't provide adequate tap area. Unless my eyes are playing tricks on me, it looks like the current buttons are 36x36, so for now I'd at least match that. Then in the future I would consider going with ≥44pt (via iOS HIG) or better yet ≥48dp (via Google Material).

Obviously that concern isn't limited to this specific button, and it's another separate project, but maybe something to consider/keep in mind moving forward. :)

@ellatrix

This comment has been minimized.

Copy link
Member Author

commented Mar 7, 2019

Okay, sounds good, I can make the button bigger.

@jasmussen

This comment has been minimized.

Copy link
Contributor

commented Mar 7, 2019

Nice, can you then remove the right margin I added to the drop-down icon, or is that still necessary to balance things?

@ellatrix

This comment has been minimized.

Copy link
Member Author

commented Mar 7, 2019

@jasmussen I replaced it in 88bfc3e.

@jasmussen

This comment has been minimized.

Copy link
Contributor

commented Mar 7, 2019

🤘

@iamthomasbishop

This comment has been minimized.

Copy link

commented Mar 7, 2019

Looks great, thanks @ellatrix!

@youknowriad
Copy link
Contributor

left a comment

I like this, I like that we remove the inline items from the inserter, it was confusing for a lot of people there (based on feedback), that said, it's not a trivial change in terms of UI. So I'd appreciate an approval from designers @jasmussen @mapk

@jasmussen
Copy link
Contributor

left a comment

Yep, this is solid. Couldn't break it on any breakpoint, or with fullscreen or top toolbar or anything. It solves a problem and takes steps in the right direction. It's also a good place for the Inline Image button.

It's entirely possible additional enhancements can be made in the future, but this is a solid PR.

@ellatrix

This comment has been minimized.

Copy link
Member Author

commented Mar 8, 2019

Thanks for the reviews!

@ellatrix ellatrix merged commit 9e7a851 into master Mar 8, 2019

@ellatrix ellatrix deleted the try/rich-text-collapse-toolbar branch Mar 8, 2019

@@ -22,7 +22,6 @@ export {
default as RichText,
RichTextShortcut,
RichTextToolbarButton,
RichTextInserterItem,

This comment has been minimized.

Copy link
@youknowriad

youknowriad Mar 8, 2019

Contributor

Is this a breaking change? Should we stub it at least to avoid breakage or find a way to move it automatically to the toolbar?

This comment has been minimized.

Copy link
@gziolo

gziolo Mar 8, 2019

Member

Yeah, mapping to RichTextToolbarButton should do the trick.

Tug added a commit that referenced this pull request Mar 14, 2019

Get the last master changes into the mobile develop branch (#14375)
* Add ESNext syntax to meta block tutorial (#13954)

* Add ESNext syntax to meta block tutorial

* Applied WordPress code styles to the examples

* Apply suggestions from code review

Co-Authored-By: mkaz <[email protected]>

* Editor: RichText: Check for presence of inputType (#13986)

* Bump plugin version to 5.1.1 (#13990)

* Added a snippet for observing the browser when running e2e tests (#13993)

* Added a snippet for observing the browser when running e2e tests

* Extract reusable part of Webpack config and put in @wordpress/scripts (#13814)

* New build-config package with webpack config.

Pull the Gutenberg webpack config into a package so it can be re-used for
block/extension development.

* Require new build-config package.

* Dynamically handle WP externals with a function.

Use code from WP Calypso for handling WP externals so we don't have to have the
actual list of packages accessible in our webpack configuration.

* Use webpack config from build-config package.

* Require build-config package.

* Adjust file refs for WP packages.

* Move main gutenberg entry definition and webpack copy plugin out of build-config.

* Add react-dev-utils for formatting webpack compiler messages.

* Implement build script using webpack config from build-config.

* Adjust output path so build goes to working directory.

* Update package name to webpack-config

* Apply more tweaks to the way webpack config package is structured

* Update the way externals are handled

* Add default values for entry and output

* Move shared webpack config under @wordpress/scripts package

* Improve the way how loaders are handled

* Replace GUTENBERG with WP in webpack config env variables

Co-Authored-By: gziolo <[email protected]>

* Bring back feature flag to webpack config accidentally removed during merge

* Add missing dev dependencies for the packages used in webpack config

* Fix the list of excluded folders for babel-loader

* Use globals instead of imports in tutorials (#13995)

* Use globals instead of imports

* Update docs/designers-developers/developers/tutorials/metabox/meta-block-3-add.md

Co-Authored-By: nosolosw <[email protected]>

* Update docs/designers-developers/developers/tutorials/metabox/meta-block-3-add.md

Co-Authored-By: nosolosw <[email protected]>

* URL input popover visual cleanup (#13973)

* Use chevron instead of ellipsis in url input field options.

* Mimic toolbar icon styles for the icons in the URL popover.

* Add a left divider before the URL settings toggle

* Even up the spacing in the settings panel.

* Add periods to code comments.

* Update snapshot

* Reinstate "underline" in default formats list (#14008)

* Upgrade React to 16.8.2: Welcome React Hooks (#13992)

* Upgrade React to 16.8.2

* Update package-lock.json file

* Expose React Hooks

* Revert "Upgrade React to 16.8.2: Welcome React Hooks (#13992)" (#14017)

This reverts commit 6f88bec.

* Cleanup URL Popover stylesheet. (#14015)

As noted in #13973, this stylesheet uses a relatively non-standard SCSS method of nesting some classnames. For instance:

`.editor-url-popover { &__settings-toggle { ... } }`.

... instead of:

`.editor-url-popover__settings-toggle { ... }`

This is different from the conventions used elsewhere in Gutenberg, and is a bit more difficult to follow for that reason.

This commit un-nests those styles, and should have no effect on the compiled CSS.

* Add a generic block editor module (#13088)

* RichText: warn when using inline container (#13921)

* RichText: warn when using inline container

* Add env check

* Update documentation

* Try updating the WP database (#14048)

* Merge native mobile release v1.0 to master (#14061)

* Bump plugin version to 5.1.0-rc.1

* RichText: only ignore input types that insert HTML (#13914)

* RichText: only ignore input types that insert HTML

* Mark RichTextInputEvent as unstable

* Use Set

* Bump plugin version to 5.1.0

* Deprecate RichTextInputEvent on mobile too (#13975)

* The undelying RichText component implementation has changed the parameters returned onChange, and we forgot to update the PostTitle component (#13967)

* Fixes wrong state comparison (#13987)

Upload media progress bar is missing while media is uploading new

* Re-add rootTagsToEliminate prop (#14006)

* [Mobile]Update PostTitle to apply borders when it is focused (#13970)

* Trigger onFocusStatusChange from PostTitle

* Fix lint issue

* Update post title shadow mechanism

Also open inner ref so that focus state can be updated when focus is made programmatically

* Update props

* Update onRef as ref

* Update title padding&margin

* Mobile: Rename ref to innerRef on PostTitle (#14024)

* Fixes a red screen in mobile. (#14011)

* Change background color on image placeholder block (#14033)

* Changed upload media icon color

* Changed media placeholder background color

* Fix post title native syntax (#14041)

* Fix unexpected token in native code

* Dummy commit to trigger Travis

* Include the rnmobile release branch to Travis builds

* Mobile: Links UI using BottomSheet component (#13972)

* Mobile: Replaced Links UI with bottom-sheet component

* Mobile links UI: Removed commented code.

* Mobile: Fix lint issues

* Mobile Links UI: Remove autofocus on Android.
This hides an issue where the modal sometimes will be under the keyboard on Android.

* Fixes pasting links. (#14038)

* Update post title vertical paddings (#14040)

* Add try/catch fallback to plain text for pasteHandler (#14044)

* Fix link interface. (#14052)

* [Mobile]Fix title padding on Android (#14057)

* Remove title vertical paddings for Android

* Revert "Remove title vertical paddings for Android"

This reverts commit 09f0d35.

* Import padding variables

* Revert wrong format image color (#14058)

* Stop building the mobile release branch on Travis (#14060)

* Avoid mutating imported default config in webpack config (#14039)

* Make Babel import JSX pragma plugin aware of `wp.element.createElement` (#13809)

* Test: Add test which verifies wheter JSX pragma detects WP global

* Update packages/babel-plugin-import-jsx-pragma/test/index.js

Co-Authored-By: gziolo <[email protected]>

* Skip import when the scope variable is already defined

* Add failing tests for inner scope variable defined verification

* Add import statement when there is any undefined scope variable

* Docs: Add details about changes introduced to Babel plugin

* Upgrade Jest to version 24 (breaking changes) (#13922)

* Upgrade Jest to version 24 (breaking changes)

* Update changelog files to list all dependencies upgraded

* Downgrade puppeteer to the previous version 1.6.1

* Try to fix failing e2e tests setup

* Added clarification in the changelog

* Testing: Remove expect-puppeteer import reference

* Address issues raised during code review

* Codeowners changes for @chrisvanpatten (#14062)

* Codeowners changes for @chrisvanpatten

* Remove obsolete ghost  placeholder for no longer onowned styles

* Add repository.directory fields (fixes #13946) (#14059)

* Improve default Babel preset to include JSX pragma (#13540)

* Include custome JSX pragma support in Babel preset

* Stop using Babel tranpiliation for two packages
babel-plugin-import-jsx-pragma and postcss-themes

* Add engines field to node based packages

* Use the block editor store instead of the editor one (#13105)

* Docs: release.md: miscellaneous fixes (#14083)

* Docs: release.md: misc. fixes

* Disable block navigation and document outline items in text mode (#14081)

* Use aria-disabled instead of disabled.

* Fix unrecognized prop warning.

* Disable Document Outline items in text mode.

* Improve buttons alignment.

* Pass isTextModeEnabled as prop from parent.

* Fix selector in document outline. (#14094)

* Fix selector in document outline.

* Import getBlocks from its new location.

* Plugin: Require includes for deprecated `use_block_editor_for_` functions (#14096)

* Plugin: Fix 5.1.0 deprecated functions to correct plugin version

* Plugin: Require includes for deprecated `use_block_editor_for_` functions

* Plugin: Update server blocks script to use core equivalent function (#14097)

`gutenberg_prepare_blocks_for_js` was deprecated in Gutenberg 5.0

* Correct visual error in the quote block icon (#14091)

Fixes #13659.

The current quote block icon appears to have some over-simplified edges. This replaces it with a crisper version.

* Try: Add a subtle animation to the is-active indicator for sidebar tabs (#13956)

* Add subtle animation to the is-active indicator for sidebar tabs

* Re-instate the empty border, to prevent browser defaults from kicking in.

* Remove extra 1px of empty space

* Focus state cleanup for tab buttons

Allow them to inherit the box shadow (used for the active tab)

Use the usual standard dotted outline instead of a solid one.

* Add a pseudoclass to ensure the active border appears in Windows High Contrast Mode.

* Fix typo.

This should match the value from `/packages/edit-post/src/components/sidebar/settings-header/style.scss`

* Plugin: Remove vendor script registration (#13573)

* chore: Fix: FormToggle docs documents props the component does not uses (#14099)

* Babel Plugin Import JSX Pragma: Remove import visitor (#14106)

Props to @aduth for coding this optimization.

* Add: Block specific toolbar button sample to the format api tutorial (#14113)

## Description
This PR updates the format API tutorial (toolbar button section) to include a sample of a button that only renders on a certain block.
Answers a question posted on #14104.
Closes: #14104

## How has this been tested?
I pasted the sample code on the browser console and verified it works as expected.

* Use the editor settings to pass a mediaUpload handler (#14115)

* Use the editor settings to pass a mediaUpload handler

* Update media block snapshots

* Block Editor: Consider RECEIVE_BLOCKS as non-persistent change (#14108)

* Block Editor: Consider RECEIVE_BLOCKS as non-persistent change

* Block Editor: Compare last action in reducer enhancer only if non-ignored

* Plugin: Remove PHP functions slated for removal in 5.2 (#14090)

* Plugin: Remove PHP functions slated for removal in 5.2

* Documentation: Update FAQ to avoid reference to deprecated function

* Paste: ignore Google Docs UID tag (#14138)

* Fix  typos in copy-guide.md, readme.md and scripts.md. (#14089)

* Separate the block editor shortcuts from the post editor shortcuts (#14116)

* Fix: FocalPointPicker renders unlabelled input fields (#14152)

* Testing: Remove unnecessary Enzyme React 16 workarounds (#14156)

* Testing: Bump `enzyme-adapter-react-16` to 1.10.0

* Testing: Avoid unforwarded Button mock

No longer necessary with native support for forwardRef in Enzyme

* Testing: Un-skip BlockControls snapshot test

* Plugin: Preserve inline scripts in Gutenberg override (#13581)

* Plugin: Preserve inline scripts in Gutenberg override

* Plugin: Restore storageKey assignment for persistence migration

* Refactor to remove usage of post related effects in packages/editor. (#13716)

This pull is the first step in moving away from the lingering usage of effects in various data stores among packages. This pull specifically deals with post related effects in the @wordpress/editor package (`core/editor` store).

* Add array-callback-return rule; Fix current code breaking the rule. (#14154)

## Description
Adds a rule to make sure Array functions that iterate on the array and should return something contain the return statement, otherwise, a forEach should probably be used instead.
A case like this was fixed at #13953.

In PR #13953 @aduth suggested the implementation of a lint rule to catch these cases. While trying to implement the rule and researching the best ways to do it, I noticed a rule like that already existed in the community and this PR is enabling it.

We are also changing the code to respect the new rule no observable changes should be expected.

## How has this been tested?
Observe the tests pass.
Do some smoke testing, adding blocks, uploading files, and verify everything still works as before.

* Update text displayed when an embed can't be previewed (#13715)

* Update text displayed when an embed can't be previewed

* Add translator note for update to embedded content np preview message

* Fix: Image that is uploaded to an existing gallery does not appear in the edit gallery view (#12435)

* RichText: fix wordwise selection on Windows (#14184)

* RichText: fix wordwise selection on Windows

* Fix crtl ctrl typo.

* Add docs

* Chore: Update: Code Quality:  Remove some editor store references from block-editor (#14161)

We missed to update the editor references from selectPreviousBlock, and selectNextBlock.

## Tests

I added multiple blocks, I verified that when I remove a block the previous block still gets selected.

* Framework: Update package-lock.json for new Enzyme adapter (#14192)

* Notices: Remove inaccurate createNotice sole argument feature (#14177)

* New package to auto-generate public API documentation (#13329)

* Update nosolosw notifs (#14196)

* Make IconButton able to be referenced. (#14163)

* Make IconButton able to get referenced.

* Components: Forward IconButton ref as stateless function

* Components: Restore IconButton aria-pressed prop pass-through

* Paste: preserve empty table cells (#14137)

* Update Codeowners, add mkaz to docgen (#14198)

* Plugin: Remove wp-editor-font stylesheet override (#14176)

* Bring i18n functionality up to date (#12559)

* Plugin: Add Text Domain to plugin headers

* Bring i18n functionality up to date

* Plugin: Set Gutenberg script translations as default domain

Filter loading behavior to load from plugin translation files

* Testing: Update gutenberg_override_script per localization changes

* Plugin: Provide second argument for set script translations

Technically optional, only after WP5.1+, which is newer than the current minimum version supported by Gutenberg

* Fix: deleting the last block triggers a focus loss. (#14189)

## Description
This PR fixes a problem: if the post contains one block and it is removed the focus is lost.

This is a regression that happened when removeBlocks action was refactored to be a generator. We had an effect that inserts the default block during remove blocks action when certain conditions are met, this effect stopped working.
This PR removes the effect and makes sure everything is handled by the removeBlocks action creator.

End to end test available at #14191.

## How has this been tested?
I created a new post.
I wrote something in a paragraph I removed the paragraph using the remove button in the side menu and I verified the default block was added.

* Clarify the block editor settings from the post editor settings (#14082)

* Add repository.directory linting rule (fixes #13947) (#14200)

* Add repository.directory rule to npm-package-json-lint-config (fix #13947)

* Fix lock file (props @aduth)

* Fix rule, update changelogs

* Add PR reference

* Add directory field to packages/docgen

* Apply changes suggested by @ntwb

* Update CHANGELOG.md

* Plugin: Remove deprecated `_wpLoadGutenbergEditor`, `gutenberg` theme supports (#14144)

* Plugin: Remove deprecated `gutenberg` theme supports
* Plugin: Remove deprecated `_wpLoadGutenbergEditor`

* Bump plugin version to 5.2.0-rc.1 (#14210)

* Packages: Add missing expect-pupeteer dependency to e2e-tests package (#14212)

* Lowercase Block Editor to block editor as per the core spelling Best Practices (#14205)

## Description
I've updated all references to Block Editor to be lowercase as they aren't proper nouns. This conforms to the core spelling Best Practices;
“block editor” or “block-based editor” | “Block Editor” or “Gutenberg” | When referring to the new editor.
Reference - https://make.wordpress.org/core/handbook/best-practices/spelling/

Also ties back to the original discussion around the capitalization convention here;
#12856

Previously opened #14203 to handle the Classic Editor changes.

## How has this been tested?
Only tested that it didn't break anything.

## Types of changes
Mostly comment changes, there was one string change and a couple translator comment changes.

## 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/ -->
- [x] I've included developer documentation if appropriate. <!-- Handbook: https://wordpress.org/gutenberg/handbook/designers-developers/ -->

* Babel Preset Default: Avoid disabling regenerator option (#14130)

* Babel Preset Default: Remove redundant (defaulted) corejs option

* Scripts: Assign Babel runtime regenerator as externals in Webpack config

* Babel Preset Default: Avoid disabling regenerator option

* Testing: Await E2E promise interactions (#14219)

* Testing: Await E2E promise interactions

* Testing: Disable animations on every admin screen

* Testing: Refactor reusable block management to compare before/after entry count

* Lowercase classic editor when not in reference to the plugin. (#14203)

* Lowercase classic editor when not in reference to the plugin.

* Reverted heading to use capitalized as the heading choice throughout uses full capitalization style on headings so went back to make consistent as @torres126 recommended. Also updated the snap file to resolve the Travis CI error.

* Missed saving README.md where the revert on headding capitalization was in my editor.

* Fix: Calendar block: Always show current month for non post types on the editor (#13873)

* Fix allowed_block_types regression (#14229)

* Fix: Latest posts: Title is clickable across the full width of the row (#14109)

## Description
The title in latest posts block was clickable across the full width of the row, making it easy to click on the title by mistake.
This problem only affects the editor on the front end things worked as expected.

## How has this been tested?
I added the latest posts block.
I verified that the title is only clickable if the mouse is above the title and not above any part of its row.


## Screenshots <!-- if applicable -->
Before:
![feb-25-2019 21-49-15](https://user-images.githubusercontent.com/11271197/53371643-b3e3f380-3948-11e9-95f5-a8f235fabab2.gif)

After:
![feb-25-2019 21-53-30](https://user-images.githubusercontent.com/11271197/53371657-c2320f80-3948-11e9-8dfd-ab7ae4640955.gif)

* Blocks: Regenerate RSS, Search block fixtures (#14122)

* Component: date-time: Remove comparison to now when testing getMomentDate (#14230)

* RichText: Fix undo after pattern (#13917)

* Fix undo after pattern

* Update e2e test

* RichText: don't use DOM to add line padding (#13850)

* RichText: don't use DOM to add line padding

* Clean up

* Keep track of user inserted line breaks

* Mark isEditableTree param unstable

* Use a central script instead of one per package to generate docs (#14216)

* Update: Use escape key press instead of mouse movement to show block toolbar (#14247)

This PR updates our end to end tests to use escape key press instead of mouse movement to show the block toolbar. The PR follows a suggestion by @aduth in #14191 (comment)

## How has this been tested?
We only need to make sure end 2 end tests pass

* Switch the Travis badge from travis-ci.org to travis-ci.com. (#14250)

* Update the npm packages release process (#14136)

* Update plugin version to 5.2 (#14255)

* chore(release): publish

 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]

* Update the package changelogs after the npm release

* Add clickBlockToolbarButton end to end test util (#14254)

This PR adds an end 2 end test util that allows clicking in a block toolbar button and refactors existing code to use it.

## How has this been tested?
We only need to verify that the end 2 end tests pass.

* element: set up autogenerated API docs (#14269)

* Testing: Trash existing posts as admin user (#14244)

* escape-html: set up autogenerated API docs (#14268)

* html-entities: set up auto-generated API docs (#14267)

* keycodes: set up auto-generated API docs (#14265)

* Use currentColor for fill of placeholder icon—ensures icon contrasts with background color of block (#14257)

* a11y: set up auto-generated API docs (#14288)

* blob: set up auto-generated API docs (#14286)

* block-library: set up auto-generated API docs (#14282)

* compose: set up auto-generated API docs (#14278)

* dom: set up auto-generated API docs (#14273)

* i18n: set up auto-generated API docs (#14266)

* Clarify that we should rebase the branch when preparing the npm release branches (#14260)

* autop: set up auto-generated API docs (#14287)

* dom-ready: set up autogenerated API docs (#14272)

* block-editor: set up auto-generated API docs (#14285)

* Adding an e2e test verifying simple keyboard navigation through blocks (#13455)

* Adding an e2e test verifying simple keyboard navigation through blocks (Issue #12392) using bug resolved in #11773 as the basis for the steps

* Moving `navigateToContentEditorTop`, `tabThroughParagraphBlock`, `tabThroughBlockMoverControl` and `tabThroughBlockToolbar` to the parent scope. Using pressKeyWithModifier within navigateToContentEditorTop.

* Add nested blocks inside cover block (#13822)

* rich-text: set up autogenerated API docs (#14220)

* Add link to WordPress Support documentation (#14316)

* Remove users documentation (#14318)

* Fix the double dash issue (#14321)

The double dash was being converted to a single dash (&#8211;)

* docgen: Generate package docs in parallel (#14295)

* Add new actions for invalidating resolution caches (#14225)

* add new actions for invalidating resolution caches

* use path array for omit

* clarify logic

* CC-BY-3.0 is not GPLv2 compatible. (#14329)

* blocks: set up auto-generated API docs (#14279)

* deprecated: set up auto-generated API docs (#14275)

* priority-queue: set-up auto-generated API docs (#14262)

* shortcode: set up autogenerated API docs (#14218)

* viewport: set up autogenerated API docs in README (#14214)

* url: set up autogenerated API docs (#14217)

* redux-routine: set up autogenerated API docs (#14228)

* date: set up auto-generated API docs (#14276)

* block-serialization-default-parser: set up auto-generated API docs (#14280)

* RichText: collapse toolbar (#14233)

* RichText: collapse toolbar

* Move inline image

* Add balanced margin

* Add tooltip

* Update e2e test

* Update e2e test

* Remove RichTextInserterItem

* Ensure there are fills before rendering

* Make toggle button bigger

* Use clickBlockToolbarButton

* Fix e2e test

*  RichText: fix br padding in multiline and nesting (#14315)

* RichText: fix br padding in multiline and nesting

* Add e2e test

* Docs: Release: Suggest lighter SVN checkout (#14259)

* plugins: set up auto-generated API docs (#14263)

* Move the block components to the block editor module (#14112)

* wordcount: set up autogenerated API docs (#14213)

* Update packages used by eslint plugin to remove warnings (#14077)

* Update packages used by eslint plugin to remove warnings

* Manually modify package-lock.json

Props @gziolo.

* Specify React version in eslint-plugin-react settings

* Fix react/jsx-no-target lint errors

* Try Legacy widget block (#13511)

## Description
Implements: #4770

This PR is a **proof of concept** of a legacy widget block. A block that allows existing WordPress widgets to be added as Gutenberg blocks.


The design is similar to the one proposed by @melchoyce in #4770 (comment) (option 1). Although it seems option two was preferred, it would require to expose each widget as a block similar to what embeds do, and it would increase the technical complexity and make testing/debugging harder, so I preferred to use option 1 for now. I will gladly iterate on the UX after this proof of concept gets more stable.


## Some technical details

The available widgets are preloaded to Gutenberg similar to what happens with page templates.

### REST-API user story
I want to able to pass a widget identifier to an endpoint, pass the existing widget attributes and the changes the user is making if any, and receive from the rest API, the sanitized new widget attributes ready to save and an HTML form that allows the user to edit the widget in the new state.

###  REST-API endpoint
A very simple REST-API endpoint was implemented.  The endpoint receives the previous instance of a widget (previous attributes) the changed instance of a widget (changed attributes if any) and returns the new instance of the widget and the HTML form that allows editing this widget.
There are two ajax-admin endpoints save-widget and update-widget. It looks like each one has specificities that make using it here complex. The ajax admin code would probably require some changes to be used here. Our use case is straightforward from the backend perspective as the widget does not need to be saved anywhere, and the widget is not associated with any widget area. The most straightforward approach seemed to be using a very simple endpoint. If I missed something and adapting existing endpoints is simpler feel free to comment, and I will have a look.


### Block Architecture

The edit component of the block handles the start placeholder that allows selecting a widget, and the tab mechanism that allows switching between edit and preview.
The preview is done using the ServerSideRender component.
The edit is done using two components:
**WidgetEditHandler:** Is responsible for server communication using the endpoint we created, and for keeping the required local state for the edition. Renders an update button, when pressed we retrieve from the dom the changed fields (using a method provided by WidgetEditDomManager) issues a request to the server and updates the legacy widget instance attribute with the server answer.
**WidgetEditDomManager:** Component responsible for rendering the starting dom for a widget. After the first render React never rerenders the component again, the content rendered by this component is then managed by the scripts the widgets may implement. When a new instance of the form HTML is received we manually update the dom changing .widget-content (like the customizer and the widget screen) do. This component provides a method that returns an object with the widget changed attributes. When this component is mounted it triggers a widget-added event when a new update happens and the dom is changed by the component widget-update jQuery event is triggered.

On front end widget are rendered with a simple call to [the_widget](https://codex.wordpress.org/Function_Reference/the_widget).

 

## Screenshots <!-- if applicable -->

![jan-25-2019 19-37-44](https://user-images.githubusercontent.com/11271197/51768637-c3b5b100-20d8-11e9-941f-00adb4c7b0a1.gif)
![jan-25-2019 18-58-49](https://user-images.githubusercontent.com/11271197/51768645-cb755580-20d8-11e9-89e7-1aa9ba7256c3.gif)

## Known problems
- The block is not aware of any change inside the widget until the update button is pressed. This replicates the save button on the widgets screen. But it is annoying if we change something on the widget and go to the widget preview right away our changes are not reflected in the preview. Having an explicit update, makes testing and debugging easier, we may than explore other approaches e.g.: also save when preview happens, save on blur events, etc.

- The text widget that contains TinyMCE crashes and fails to init. It calls wp.editor.initialize to reference TinyMCE and on Gutenberg, wp.editor is our editor module. This problem may have happened with meta boxes if it was solved probably the same approach may be applied.

- The widget design may be affected by CSS that exists in Gutenberg, so the design of the widgets does not look the same. Ideally, Gutenberg CSS would not affect the widgets but as they are on the same page that's not the case.

- Some third-party widgets using don't initialize correctly. That happens because the dom of the editor is not equal to the dom of the customizer and/or the widget screen. Some JS widgets use click events on the widgets screen to initialize while on Gutenberg these events don't happen, some check if they are on the customizer page (body contains customizer classes) before handling widget-updated events. Normally adapting a widget that does not initialize correctly is a matter of changing a very simple condition on the plugin.

* Update changelog for esling-plugin (#14339)

* Add ability to transform [video] shortcodes to video block (#14042)

* CSS to remove the clipping of the toolbar in RTL languages (#14088)

* added the more tag via the old editor's image (#14173)

* Seimplify hierarchical term selector strings. (#13938)

* Add: End to end test to make sure when all  blocks get removed the default block appender gets inserter and selected (#14191)

* Fix the Deprecated Blocks link (#14355)

The *Deprecated Blocks link* was pointing to the wrong URL.

* Added testcases for isKeyboardEvent in keycodes module (#14073)

* Added testcases for isKeyboardEvent in keycodes module

* Fixed assertion checks and updated attachEventListeners function

* Added testcases for isKeyboardEvent in keycodes module

* Fixed assertion checks and updated attachEventListeners function

* Fix: Quote to heading transform (#14348)

* Latest Comments block: use align supports flag + code cleanup (#11411)

* ToggleControl allows setting custom classes (#13804)

* ToggleControl allows setting custom classes

Related Issue: #11349 
This pull request fixes the "ToggleControl does not allow setting custom classes" problem. This lets the user to add a custom classname on the element, just like in TextControl.

* Update README of toggle-control

Add property className and its description

* Update packages/components/src/toggle-control/README.md

Formatted style of README

Co-Authored-By: AmartyaU <[email protected]>

* Edited text of button to be actionable (#14347)

* Block library: Remove all test snapshots for blocks (#14349)

* Fix: Inserter impossible to collapse panels while searching. (#13884)

* Make taxonomies test relieable. (#14340)

* edit-post: set up autogenerated API docs (#14271)

* Plugin: Remove postinstall step (#14353)

* Plugin: Remove postinstall step

* Framework: Run build for JS unit tests

Assumed previously relied on postinstall

* Add check to the merge function in headings and paragraph blocks (#13981)

If trying to merge a blank heading or paragraph, if this check isn't performed,
the result is the string 'Null' appearing in the text.

* Update internationalization process with complete updated example. (#13909)

* Update internationalization process with complete updated example.

* Minor edits

* Update i18n package documentation

Updates with links to expanded section in Gutenberg Handbook

Fixes instructions to use wp-cli which is the recommended tool for
creating pot files, and po2json to convert the format.

* Apply suggestions from code review

Co-Authored-By: mkaz <[email protected]>

* Remove duplicate documentation, just link to Handbook

* Batch of changes from reviews props @swisspiddy

* :shakes-fist-at-whitespace:

* Updates to include full .pot and .po files per @nosolosw review

* Add JSON translation example

* Fix syntax to make gutenberg build in this branch

* E2E Test Utils: Add missing babel/runtime dependency (#14374)

* Build: Only prompt clean if unclean (#14352)

* Remove unused mobile stylesheets

* Port editor refactoring for mobile

* fix names (#14382)

* Add withRegistry HigherOrderComponent (#14370)

* Fix block validation error message (#13499)

* Rename variable to make code more clear

* Fix display of expected and actual block HTML

The variable were in the wrong order, making the message confusing. It
would show the actual HTML as the expected HTML and visa versa.

* Change wording to be more clear

Expected & Actual were confusing words to use in this content. This
change makes the message actually reflect what the values are.

* Regenerate docs

* Fix: Global inserter does not validates block insert restrictions (#14020)

* Teach build and start commands to use Webpack default if none is provided (#13877)

* Use a default webpack config if none is provided

* Extract webpack utils from build command

* Update start command

* Add docs

* Add Webpack documentation

* Add example of how to overwrite the default plugins

* Do not export hasWebpackConfig

as it is not used anywhere else.

* Always pass webpack CLI args to command

* Update README

* Remove section on extending the default webpack config file

* Simplify array passing

Co-Authored-By: nosolosw <[email protected]>

* Simplify pushing to array

Co-Authored-By: nosolosw <[email protected]>

* Fix externals docs

* Use webpack instead of Webpack

* Add Changelog entry

* RichText: Fix prepareEditableTree (#14284)

* Fix use of __experimentalCreatePrepareEditableTree without __experimentalCreateOnChangeEditableValue

* Add unit tests

* Add module entry point to notices package.json file (#14388)

* Plugin: Remove PHP functions slated for 5.3 removal (#14380)

* Make @wordpress/edit-post compatible with native

* Unify styles for UnsupportedBlock after sass build default imports is fixed on native

* Remove scss variables and colors import after fixing sass build in gb-mobile

* RichText: change value to have separate keys for line and object formats (#13948)

* Add objects and lineFormats

* Update RichText

* Fix image toolbar

* Update format placeholder

* lineFormat => lines

* concatPair => mergePair

* Update selectedFormat checks

* Add some extra info to create docs

* Move create docs inline

* Merge lines and objects

* Fix typos

* Add getActiveObject unit tests

* Update docs

* Rebase

* Adjust unstableToDom arguments

* Remove normaliseFormats from list functions

* Update native files

* Update native file

* Remove code formatting for mobile

* RichText: try alternative list shortcuts (to tab) (#14343)

* RichText: try alternative list shortcuts

* Try tooltips

* Change tooltips to use text

* Add inline comments

* Add e2e test

* Rebase

* Remove unused styles classes. (#14338)

* [Mobile]Fix placeholder position of block appender (#14386)

* Fix placeholder position of block appender

* Remove unused import

* Change import position

* Remove unused import in css file

* Remove extra container view

* Update travis.yml from master branch to fix CI issues

* [Mobile]Update caret position on insert link (#14317)

* Update selection on text change if needed

* Update caret position when editing the link

* Revert putting the caret at end when we transform the selected text into link

* Fix source map paths published to npm (#14409)

* Components: update Button readme to add design guidelines (#14194)

These changes add design documentation. Co-Authored-By: kjellr <[email protected]>

* Remove export from syntax still a proposal

* Update README.md
@azaozz

This comment has been minimized.

Copy link
Contributor

commented Mar 19, 2019

I'm not sure this is a good thing to do. It closely resembles the "Toolbar Toggle" button in the classic editor which was one of the most "questionable" functionality there. My (unofficial) stats were that about 90% of the people would open the second toolbar row there and leave it open ...forever (guessing the other 10% didn't know what that button does, and were afraid to test it) :)

...decided to cut this after Bold, Italic and Link, which are by far the most used buttons

Shouldn't the toolbar be roughly the same width as the block? Both visually and functionality wise that seems best. It would "fit" quite nicely visually, and at the same time won't obscure some of the (more useful) buttons.

Then, what about the alignment buttons. They are pretty much useless in "all text" blocks/areas like the paragraph block. I see in the screenshots that they are in another drop-down/popup. If that is implemented would we still need to hide the arguably more useful toolbar buttons added by plugins?

@@ -11,7 +18,16 @@ const FormatToolbar = ( { controls } ) => {
{ controls.map( ( format ) =>
<Slot name={ `RichText.ToolbarControls.${ format }` } key={ format } />
) }
<Slot name="RichText.ToolbarControls" />

This comment has been minimized.

Copy link
@technote-space

technote-space Mar 20, 2019

Contributor

I used this slot to add some toolbar controls.
There is no longer a way to add any controls at this position, isn't it?

This comment has been minimized.

Copy link
@ellatrix

ellatrix Mar 20, 2019

Author Member

You should use the RichText API to add controls, not the slot id directly.

This comment has been minimized.

Copy link
@technote-space

technote-space Mar 20, 2019

Contributor

Noted with thanks.

youknowriad added a commit that referenced this pull request Mar 20, 2019

RichText: collapse toolbar (#14233)
* RichText: collapse toolbar

* Move inline image

* Add balanced margin

* Add tooltip

* Update e2e test

* Update e2e test

* Remove RichTextInserterItem

* Ensure there are fills before rendering

* Make toggle button bigger

* Use clickBlockToolbarButton

* Fix e2e test

youknowriad added a commit that referenced this pull request Mar 20, 2019

RichText: collapse toolbar (#14233)
* RichText: collapse toolbar

* Move inline image

* Add balanced margin

* Add tooltip

* Update e2e test

* Update e2e test

* Remove RichTextInserterItem

* Ensure there are fills before rendering

* Make toggle button bigger

* Use clickBlockToolbarButton

* Fix e2e test

Tug added a commit that referenced this pull request Mar 21, 2019

Merge mobile development branch into master (v2) (#14503)
* Bump plugin version to 5.1.0-rc.1

* RichText: only ignore input types that insert HTML (#13914)

* RichText: only ignore input types that insert HTML

* Mark RichTextInputEvent as unstable

* Use Set

* Bump plugin version to 5.1.0

* Deprecate RichTextInputEvent on mobile too (#13975)

* The undelying RichText component implementation has changed the parameters returned onChange, and we forgot to update the PostTitle component (#13967)

* Fixes wrong state comparison (#13987)

Upload media progress bar is missing while media is uploading new

* Re-add rootTagsToEliminate prop (#14006)

* [Mobile]Update PostTitle to apply borders when it is focused (#13970)

* Trigger onFocusStatusChange from PostTitle

* Fix lint issue

* Update post title shadow mechanism

Also open inner ref so that focus state can be updated when focus is made programmatically

* Update props

* Update onRef as ref

* Update title padding&margin

* Mobile: Rename ref to innerRef on PostTitle (#14024)

* Fixes a red screen in mobile. (#14011)

* Change background color on image placeholder block (#14033)

* Changed upload media icon color

* Changed media placeholder background color

* Fix post title native syntax (#14041)

* Fix unexpected token in native code

* Dummy commit to trigger Travis

* Include the rnmobile release branch to Travis builds

* Mobile: Links UI using BottomSheet component (#13972)

* Mobile: Replaced Links UI with bottom-sheet component

* Mobile links UI: Removed commented code.

* Mobile: Fix lint issues

* Mobile Links UI: Remove autofocus on Android.
This hides an issue where the modal sometimes will be under the keyboard on Android.

* Fixes pasting links. (#14038)

* Update post title vertical paddings (#14040)

* Add try/catch fallback to plain text for pasteHandler (#14044)

* Fix link interface. (#14052)

* [Mobile]Fix title padding on Android (#14057)

* Remove title vertical paddings for Android

* Revert "Remove title vertical paddings for Android"

This reverts commit 09f0d35.

* Import padding variables

* Revert wrong format image color (#14058)

* Stop building the mobile release branch on Travis (#14060)

* [Native mobile] Bring release v1.0.1 back to "mobile develop" (#14075)

* Don't allow placeholder to moves once we tap on it (#14066)

* Use platform based SCSS vars for block minHeight (#14070)

* Rnmobile/refactor rich text sizing code (#14164)

* Make richText height changes contained to the rich text block.

* Remove commented out code.

* Remove aztec height from state.

* Allow minHeight to be optional.

* Remove minHeight from postTitle.

* Remove minHeight on heading block.

* Mobile: Fix pasting in header (#14118)

* Fixes pasting in heading blocks.

* Fixed an issue with a missing constant.

* Removes some code that was added back as part of a merge conflict.

* Re-adds some code removed by mistake.

* Mobile: Fix links ui on landscape iOS v2 (#14240)

* RNMobile: Adding original versin of keyboard-avoiding-view (to be modified)

* Modified keyboard-avoiding-view to work well with bottom-sheets.

* Mobile: Avoid adding empty link to text. (#14270)

* Mobile Links UI: Avoid creating link with empty url field

* Mobile Links UI: Using URL Keyboard type for url field

* Fix lint issues

* Avoid to reset html to empty string if block is heading and platform is android (#14301)

* Avoid to reset html to empty string if block is heading on android platform

* Send empty tag flag from heading block to RichText

* Set minHeight based on fontSize or style. (#14344)

* Set minHeight based on fontSize or style.

* Only use the styles to set the min-height.

* Remove unused styles classes. (#14338)

* [Mobile]Fix placeholder position of block appender (#14386)

* Fix placeholder position of block appender

* Remove unused import

* Change import position

* Remove unused import in css file

* Remove extra container view

* Update travis.yml from master branch to fix CI issues

* [Mobile]Update caret position on insert link (#14317)

* Update selection on text change if needed

* Update caret position when editing the link

* Revert putting the caret at end when we transform the selected text into link

* Get the last master changes into the mobile develop branch (#14375)

* Add ESNext syntax to meta block tutorial (#13954)

* Add ESNext syntax to meta block tutorial

* Applied WordPress code styles to the examples

* Apply suggestions from code review

Co-Authored-By: mkaz <[email protected]>

* Editor: RichText: Check for presence of inputType (#13986)

* Bump plugin version to 5.1.1 (#13990)

* Added a snippet for observing the browser when running e2e tests (#13993)

* Added a snippet for observing the browser when running e2e tests

* Extract reusable part of Webpack config and put in @wordpress/scripts (#13814)

* New build-config package with webpack config.

Pull the Gutenberg webpack config into a package so it can be re-used for
block/extension development.

* Require new build-config package.

* Dynamically handle WP externals with a function.

Use code from WP Calypso for handling WP externals so we don't have to have the
actual list of packages accessible in our webpack configuration.

* Use webpack config from build-config package.

* Require build-config package.

* Adjust file refs for WP packages.

* Move main gutenberg entry definition and webpack copy plugin out of build-config.

* Add react-dev-utils for formatting webpack compiler messages.

* Implement build script using webpack config from build-config.

* Adjust output path so build goes to working directory.

* Update package name to webpack-config

* Apply more tweaks to the way webpack config package is structured

* Update the way externals are handled

* Add default values for entry and output

* Move shared webpack config under @wordpress/scripts package

* Improve the way how loaders are handled

* Replace GUTENBERG with WP in webpack config env variables

Co-Authored-By: gziolo <[email protected]>

* Bring back feature flag to webpack config accidentally removed during merge

* Add missing dev dependencies for the packages used in webpack config

* Fix the list of excluded folders for babel-loader

* Use globals instead of imports in tutorials (#13995)

* Use globals instead of imports

* Update docs/designers-developers/developers/tutorials/metabox/meta-block-3-add.md

Co-Authored-By: nosolosw <[email protected]>

* Update docs/designers-developers/developers/tutorials/metabox/meta-block-3-add.md

Co-Authored-By: nosolosw <[email protected]>

* URL input popover visual cleanup (#13973)

* Use chevron instead of ellipsis in url input field options.

* Mimic toolbar icon styles for the icons in the URL popover.

* Add a left divider before the URL settings toggle

* Even up the spacing in the settings panel.

* Add periods to code comments.

* Update snapshot

* Reinstate "underline" in default formats list (#14008)

* Upgrade React to 16.8.2: Welcome React Hooks (#13992)

* Upgrade React to 16.8.2

* Update package-lock.json file

* Expose React Hooks

* Revert "Upgrade React to 16.8.2: Welcome React Hooks (#13992)" (#14017)

This reverts commit 6f88bec.

* Cleanup URL Popover stylesheet. (#14015)

As noted in #13973, this stylesheet uses a relatively non-standard SCSS method of nesting some classnames. For instance:

`.editor-url-popover { &__settings-toggle { ... } }`.

... instead of:

`.editor-url-popover__settings-toggle { ... }`

This is different from the conventions used elsewhere in Gutenberg, and is a bit more difficult to follow for that reason.

This commit un-nests those styles, and should have no effect on the compiled CSS.

* Add a generic block editor module (#13088)

* RichText: warn when using inline container (#13921)

* RichText: warn when using inline container

* Add env check

* Update documentation

* Try updating the WP database (#14048)

* Merge native mobile release v1.0 to master (#14061)

* Bump plugin version to 5.1.0-rc.1

* RichText: only ignore input types that insert HTML (#13914)

* RichText: only ignore input types that insert HTML

* Mark RichTextInputEvent as unstable

* Use Set

* Bump plugin version to 5.1.0

* Deprecate RichTextInputEvent on mobile too (#13975)

* The undelying RichText component implementation has changed the parameters returned onChange, and we forgot to update the PostTitle component (#13967)

* Fixes wrong state comparison (#13987)

Upload media progress bar is missing while media is uploading new

* Re-add rootTagsToEliminate prop (#14006)

* [Mobile]Update PostTitle to apply borders when it is focused (#13970)

* Trigger onFocusStatusChange from PostTitle

* Fix lint issue

* Update post title shadow mechanism

Also open inner ref so that focus state can be updated when focus is made programmatically

* Update props

* Update onRef as ref

* Update title padding&margin

* Mobile: Rename ref to innerRef on PostTitle (#14024)

* Fixes a red screen in mobile. (#14011)

* Change background color on image placeholder block (#14033)

* Changed upload media icon color

* Changed media placeholder background color

* Fix post title native syntax (#14041)

* Fix unexpected token in native code

* Dummy commit to trigger Travis

* Include the rnmobile release branch to Travis builds

* Mobile: Links UI using BottomSheet component (#13972)

* Mobile: Replaced Links UI with bottom-sheet component

* Mobile links UI: Removed commented code.

* Mobile: Fix lint issues

* Mobile Links UI: Remove autofocus on Android.
This hides an issue where the modal sometimes will be under the keyboard on Android.

* Fixes pasting links. (#14038)

* Update post title vertical paddings (#14040)

* Add try/catch fallback to plain text for pasteHandler (#14044)

* Fix link interface. (#14052)

* [Mobile]Fix title padding on Android (#14057)

* Remove title vertical paddings for Android

* Revert "Remove title vertical paddings for Android"

This reverts commit 09f0d35.

* Import padding variables

* Revert wrong format image color (#14058)

* Stop building the mobile release branch on Travis (#14060)

* Avoid mutating imported default config in webpack config (#14039)

* Make Babel import JSX pragma plugin aware of `wp.element.createElement` (#13809)

* Test: Add test which verifies wheter JSX pragma detects WP global

* Update packages/babel-plugin-import-jsx-pragma/test/index.js

Co-Authored-By: gziolo <[email protected]>

* Skip import when the scope variable is already defined

* Add failing tests for inner scope variable defined verification

* Add import statement when there is any undefined scope variable

* Docs: Add details about changes introduced to Babel plugin

* Upgrade Jest to version 24 (breaking changes) (#13922)

* Upgrade Jest to version 24 (breaking changes)

* Update changelog files to list all dependencies upgraded

* Downgrade puppeteer to the previous version 1.6.1

* Try to fix failing e2e tests setup

* Added clarification in the changelog

* Testing: Remove expect-puppeteer import reference

* Address issues raised during code review

* Codeowners changes for @chrisvanpatten (#14062)

* Codeowners changes for @chrisvanpatten

* Remove obsolete ghost  placeholder for no longer onowned styles

* Add repository.directory fields (fixes #13946) (#14059)

* Improve default Babel preset to include JSX pragma (#13540)

* Include custome JSX pragma support in Babel preset

* Stop using Babel tranpiliation for two packages
babel-plugin-import-jsx-pragma and postcss-themes

* Add engines field to node based packages

* Use the block editor store instead of the editor one (#13105)

* Docs: release.md: miscellaneous fixes (#14083)

* Docs: release.md: misc. fixes

* Disable block navigation and document outline items in text mode (#14081)

* Use aria-disabled instead of disabled.

* Fix unrecognized prop warning.

* Disable Document Outline items in text mode.

* Improve buttons alignment.

* Pass isTextModeEnabled as prop from parent.

* Fix selector in document outline. (#14094)

* Fix selector in document outline.

* Import getBlocks from its new location.

* Plugin: Require includes for deprecated `use_block_editor_for_` functions (#14096)

* Plugin: Fix 5.1.0 deprecated functions to correct plugin version

* Plugin: Require includes for deprecated `use_block_editor_for_` functions

* Plugin: Update server blocks script to use core equivalent function (#14097)

`gutenberg_prepare_blocks_for_js` was deprecated in Gutenberg 5.0

* Correct visual error in the quote block icon (#14091)

Fixes #13659.

The current quote block icon appears to have some over-simplified edges. This replaces it with a crisper version.

* Try: Add a subtle animation to the is-active indicator for sidebar tabs (#13956)

* Add subtle animation to the is-active indicator for sidebar tabs

* Re-instate the empty border, to prevent browser defaults from kicking in.

* Remove extra 1px of empty space

* Focus state cleanup for tab buttons

Allow them to inherit the box shadow (used for the active tab)

Use the usual standard dotted outline instead of a solid one.

* Add a pseudoclass to ensure the active border appears in Windows High Contrast Mode.

* Fix typo.

This should match the value from `/packages/edit-post/src/components/sidebar/settings-header/style.scss`

* Plugin: Remove vendor script registration (#13573)

* chore: Fix: FormToggle docs documents props the component does not uses (#14099)

* Babel Plugin Import JSX Pragma: Remove import visitor (#14106)

Props to @aduth for coding this optimization.

* Add: Block specific toolbar button sample to the format api tutorial (#14113)

## Description
This PR updates the format API tutorial (toolbar button section) to include a sample of a button that only renders on a certain block.
Answers a question posted on #14104.
Closes: #14104

## How has this been tested?
I pasted the sample code on the browser console and verified it works as expected.

* Use the editor settings to pass a mediaUpload handler (#14115)

* Use the editor settings to pass a mediaUpload handler

* Update media block snapshots

* Block Editor: Consider RECEIVE_BLOCKS as non-persistent change (#14108)

* Block Editor: Consider RECEIVE_BLOCKS as non-persistent change

* Block Editor: Compare last action in reducer enhancer only if non-ignored

* Plugin: Remove PHP functions slated for removal in 5.2 (#14090)

* Plugin: Remove PHP functions slated for removal in 5.2

* Documentation: Update FAQ to avoid reference to deprecated function

* Paste: ignore Google Docs UID tag (#14138)

* Fix  typos in copy-guide.md, readme.md and scripts.md. (#14089)

* Separate the block editor shortcuts from the post editor shortcuts (#14116)

* Fix: FocalPointPicker renders unlabelled input fields (#14152)

* Testing: Remove unnecessary Enzyme React 16 workarounds (#14156)

* Testing: Bump `enzyme-adapter-react-16` to 1.10.0

* Testing: Avoid unforwarded Button mock

No longer necessary with native support for forwardRef in Enzyme

* Testing: Un-skip BlockControls snapshot test

* Plugin: Preserve inline scripts in Gutenberg override (#13581)

* Plugin: Preserve inline scripts in Gutenberg override

* Plugin: Restore storageKey assignment for persistence migration

* Refactor to remove usage of post related effects in packages/editor. (#13716)

This pull is the first step in moving away from the lingering usage of effects in various data stores among packages. This pull specifically deals with post related effects in the @wordpress/editor package (`core/editor` store).

* Add array-callback-return rule; Fix current code breaking the rule. (#14154)

## Description
Adds a rule to make sure Array functions that iterate on the array and should return something contain the return statement, otherwise, a forEach should probably be used instead.
A case like this was fixed at #13953.

In PR #13953 @aduth suggested the implementation of a lint rule to catch these cases. While trying to implement the rule and researching the best ways to do it, I noticed a rule like that already existed in the community and this PR is enabling it.

We are also changing the code to respect the new rule no observable changes should be expected.

## How has this been tested?
Observe the tests pass.
Do some smoke testing, adding blocks, uploading files, and verify everything still works as before.

* Update text displayed when an embed can't be previewed (#13715)

* Update text displayed when an embed can't be previewed

* Add translator note for update to embedded content np preview message

* Fix: Image that is uploaded to an existing gallery does not appear in the edit gallery view (#12435)

* RichText: fix wordwise selection on Windows (#14184)

* RichText: fix wordwise selection on Windows

* Fix crtl ctrl typo.

* Add docs

* Chore: Update: Code Quality:  Remove some editor store references from block-editor (#14161)

We missed to update the editor references from selectPreviousBlock, and selectNextBlock.

## Tests

I added multiple blocks, I verified that when I remove a block the previous block still gets selected.

* Framework: Update package-lock.json for new Enzyme adapter (#14192)

* Notices: Remove inaccurate createNotice sole argument feature (#14177)

* New package to auto-generate public API documentation (#13329)

* Update nosolosw notifs (#14196)

* Make IconButton able to be referenced. (#14163)

* Make IconButton able to get referenced.

* Components: Forward IconButton ref as stateless function

* Components: Restore IconButton aria-pressed prop pass-through

* Paste: preserve empty table cells (#14137)

* Update Codeowners, add mkaz to docgen (#14198)

* Plugin: Remove wp-editor-font stylesheet override (#14176)

* Bring i18n functionality up to date (#12559)

* Plugin: Add Text Domain to plugin headers

* Bring i18n functionality up to date

* Plugin: Set Gutenberg script translations as default domain

Filter loading behavior to load from plugin translation files

* Testing: Update gutenberg_override_script per localization changes

* Plugin: Provide second argument for set script translations

Technically optional, only after WP5.1+, which is newer than the current minimum version supported by Gutenberg

* Fix: deleting the last block triggers a focus loss. (#14189)

## Description
This PR fixes a problem: if the post contains one block and it is removed the focus is lost.

This is a regression that happened when removeBlocks action was refactored to be a generator. We had an effect that inserts the default block during remove blocks action when certain conditions are met, this effect stopped working.
This PR removes the effect and makes sure everything is handled by the removeBlocks action creator.

End to end test available at #14191.

## How has this been tested?
I created a new post.
I wrote something in a paragraph I removed the paragraph using the remove button in the side menu and I verified the default block was added.

* Clarify the block editor settings from the post editor settings (#14082)

* Add repository.directory linting rule (fixes #13947) (#14200)

* Add repository.directory rule to npm-package-json-lint-config (fix #13947)

* Fix lock file (props @aduth)

* Fix rule, update changelogs

* Add PR reference

* Add directory field to packages/docgen

* Apply changes suggested by @ntwb

* Update CHANGELOG.md

* Plugin: Remove deprecated `_wpLoadGutenbergEditor`, `gutenberg` theme supports (#14144)

* Plugin: Remove deprecated `gutenberg` theme supports
* Plugin: Remove deprecated `_wpLoadGutenbergEditor`

* Bump plugin version to 5.2.0-rc.1 (#14210)

* Packages: Add missing expect-pupeteer dependency to e2e-tests package (#14212)

* Lowercase Block Editor to block editor as per the core spelling Best Practices (#14205)

## Description
I've updated all references to Block Editor to be lowercase as they aren't proper nouns. This conforms to the core spelling Best Practices;
“block editor” or “block-based editor” | “Block Editor” or “Gutenberg” | When referring to the new editor.
Reference - https://make.wordpress.org/core/handbook/best-practices/spelling/

Also ties back to the original discussion around the capitalization convention here;
#12856

Previously opened #14203 to handle the Classic Editor changes.

## How has this been tested?
Only tested that it didn't break anything.

## Types of changes
Mostly comment changes, there was one string change and a couple translator comment changes.

## 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/ -->
- [x] I've included developer documentation if appropriate. <!-- Handbook: https://wordpress.org/gutenberg/handbook/designers-developers/ -->

* Babel Preset Default: Avoid disabling regenerator option (#14130)

* Babel Preset Default: Remove redundant (defaulted) corejs option

* Scripts: Assign Babel runtime regenerator as externals in Webpack config

* Babel Preset Default: Avoid disabling regenerator option

* Testing: Await E2E promise interactions (#14219)

* Testing: Await E2E promise interactions

* Testing: Disable animations on every admin screen

* Testing: Refactor reusable block management to compare before/after entry count

* Lowercase classic editor when not in reference to the plugin. (#14203)

* Lowercase classic editor when not in reference to the plugin.

* Reverted heading to use capitalized as the heading choice throughout uses full capitalization style on headings so went back to make consistent as @torres126 recommended. Also updated the snap file to resolve the Travis CI error.

* Missed saving README.md where the revert on headding capitalization was in my editor.

* Fix: Calendar block: Always show current month for non post types on the editor (#13873)

* Fix allowed_block_types regression (#14229)

* Fix: Latest posts: Title is clickable across the full width of the row (#14109)

## Description
The title in latest posts block was clickable across the full width of the row, making it easy to click on the title by mistake.
This problem only affects the editor on the front end things worked as expected.

## How has this been tested?
I added the latest posts block.
I verified that the title is only clickable if the mouse is above the title and not above any part of its row.


## Screenshots <!-- if applicable -->
Before:
![feb-25-2019 21-49-15](https://user-images.githubusercontent.com/11271197/53371643-b3e3f380-3948-11e9-95f5-a8f235fabab2.gif)

After:
![feb-25-2019 21-53-30](https://user-images.githubusercontent.com/11271197/53371657-c2320f80-3948-11e9-8dfd-ab7ae4640955.gif)

* Blocks: Regenerate RSS, Search block fixtures (#14122)

* Component: date-time: Remove comparison to now when testing getMomentDate (#14230)

* RichText: Fix undo after pattern (#13917)

* Fix undo after pattern

* Update e2e test

* RichText: don't use DOM to add line padding (#13850)

* RichText: don't use DOM to add line padding

* Clean up

* Keep track of user inserted line breaks

* Mark isEditableTree param unstable

* Use a central script instead of one per package to generate docs (#14216)

* Update: Use escape key press instead of mouse movement to show block toolbar (#14247)

This PR updates our end to end tests to use escape key press instead of mouse movement to show the block toolbar. The PR follows a suggestion by @aduth in #14191 (comment)

## How has this been tested?
We only need to make sure end 2 end tests pass

* Switch the Travis badge from travis-ci.org to travis-ci.com. (#14250)

* Update the npm packages release process (#14136)

* Update plugin version to 5.2 (#14255)

* chore(release): publish

 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]

* Update the package changelogs after the npm release

* Add clickBlockToolbarButton end to end test util (#14254)

This PR adds an end 2 end test util that allows clicking in a block toolbar button and refactors existing code to use it.

## How has this been tested?
We only need to verify that the end 2 end tests pass.

* element: set up autogenerated API docs (#14269)

* Testing: Trash existing posts as admin user (#14244)

* escape-html: set up autogenerated API docs (#14268)

* html-entities: set up auto-generated API docs (#14267)

* keycodes: set up auto-generated API docs (#14265)

* Use currentColor for fill of placeholder icon—ensures icon contrasts with background color of block (#14257)

* a11y: set up auto-generated API docs (#14288)

* blob: set up auto-generated API docs (#14286)

* block-library: set up auto-generated API docs (#14282)

* compose: set up auto-generated API docs (#14278)

* dom: set up auto-generated API docs (#14273)

* i18n: set up auto-generated API docs (#14266)

* Clarify that we should rebase the branch when preparing the npm release branches (#14260)

* autop: set up auto-generated API docs (#14287)

* dom-ready: set up autogenerated API docs (#14272)

* block-editor: set up auto-generated API docs (#14285)

* Adding an e2e test verifying simple keyboard navigation through blocks (#13455)

* Adding an e2e test verifying simple keyboard navigation through blocks (Issue #12392) using bug resolved in #11773 as the basis for the steps

* Moving `navigateToContentEditorTop`, `tabThroughParagraphBlock`, `tabThroughBlockMoverControl` and `tabThroughBlockToolbar` to the parent scope. Using pressKeyWithModifier within navigateToContentEditorTop.

* Add nested blocks inside cover block (#13822)

* rich-text: set up autogenerated API docs (#14220)

* Add link to WordPress Support documentation (#14316)

* Remove users documentation (#14318)

* Fix the double dash issue (#14321)

The double dash was being converted to a single dash (&#8211;)

* docgen: Generate package docs in parallel (#14295)

* Add new actions for invalidating resolution caches (#14225)

* add new actions for invalidating resolution caches

* use path array for omit

* clarify logic

* CC-BY-3.0 is not GPLv2 compatible. (#14329)

* blocks: set up auto-generated API docs (#14279)

* deprecated: set up auto-generated API docs (#14275)

* priority-queue: set-up auto-generated API docs (#14262)

* shortcode: set up autogenerated API docs (#14218)

* viewport: set up autogenerated API docs in README (#14214)

* url: set up autogenerated API docs (#14217)

* redux-routine: set up autogenerated API docs (#14228)

* date: set up auto-generated API docs (#14276)

* block-serialization-default-parser: set up auto-generated API docs (#14280)

* RichText: collapse toolbar (#14233)

* RichText: collapse toolbar

* Move inline image

* Add balanced margin

* Add tooltip

* Update e2e test

* Update e2e test

* Remove RichTextInserterItem

* Ensure there are fills before rendering

* Make toggle button bigger

* Use clickBlockToolbarButton

* Fix e2e test

*  RichText: fix br padding in multiline and nesting (#14315)

* RichText: fix br padding in multiline and nesting

* Add e2e test

* Docs: Release: Suggest lighter SVN checkout (#14259)

* plugins: set up auto-generated API docs (#14263)

* Move the block components to the block editor module (#14112)

* wordcount: set up autogenerated API docs (#14213)

* Update packages used by eslint plugin to remove warnings (#14077)

* Update packages used by eslint plugin to remove warnings

* Manually modify package-lock.json

Props @gziolo.

* Specify React version in eslint-plugin-react settings

* Fix react/jsx-no-target lint errors

* Try Legacy widget block (#13511)

## Description
Implements: #4770

This PR is a **proof of concept** of a legacy widget block. A block that allows existing WordPress widgets to be added as Gutenberg blocks.


The design is similar to the one proposed by @melchoyce in #4770 (comment) (option 1). Although it seems option two was preferred, it would require to expose each widget as a block similar to what embeds do, and it would increase the technical complexity and make testing/debugging harder, so I preferred to use option 1 for now. I will gladly iterate on the UX after this proof of concept gets more stable.


## Some technical details

The available widgets are preloaded to Gutenberg similar to what happens with page templates.

### REST-API user story
I want to able to pass a widget identifier to an endpoint, pass the existing widget attributes and the changes the user is making if any, and receive from the rest API, the sanitized new widget attributes ready to save and an HTML form that allows the user to edit the widget in the new state.

###  REST-API endpoint
A very simple REST-API endpoint was implemented.  The endpoint receives the previous instance of a widget (previous attributes) the changed instance of a widget (changed attributes if any) and returns the new instance of the widget and the HTML form that allows editing this widget.
There are two ajax-admin endpoints save-widget and update-widget. It looks like each one has specificities that make using it here complex. The ajax admin code would probably require some changes to be used here. Our use case is straightforward from the backend perspective as the widget does not need to be saved anywhere, and the widget is not associated with any widget area. The most straightforward approach seemed to be using a very simple endpoint. If I missed something and adapting existing endpoints is simpler feel free to comment, and I will have a look.


### Block Architecture

The edit component of the block handles the start placeholder that allows selecting a widget, and the tab mechanism that allows switching between edit and preview.
The preview is done using the ServerSideRender component.
The edit is done using two components:
**WidgetEditHandler:** Is responsible for server communication using the endpoint we created, and for keeping the required local state for the edition. Renders an update button, when pressed we retrieve from the dom the changed fields (using a method provided by WidgetEditDomManager) issues a request to the server and updates the legacy widget instance attribute with the server answer.
**WidgetEditDomManager:** Component responsible for rendering the starting dom for a widget. After the first render React never rerenders the component again, the content rendered by this component is then managed by the scripts the widgets may implement. When a new instance of the form HTML is received we manually update the dom changing .widget-content (like the customizer and the widget screen) do. This component provides a method that returns an object with the widget changed attributes. When this component is mounted it triggers a widget-added event when a new update happens and the dom is changed by the component widget-update jQuery event is triggered.

On front end widget are rendered with a simple call to [the_widget](https://codex.wordpress.org/Function_Reference/the_widget).

 

## Screenshots <!-- if applicable -->

![jan-25-2019 19-37-44](https://user-images.githubusercontent.com/11271197/51768637-c3b5b100-20d8-11e9-941f-00adb4c7b0a1.gif)
![jan-25-2019 18-58-49](https://user-images.githubusercontent.com/11271197/51768645-cb755580-20d8-11e9-89e7-1aa9ba7256c3.gif)

## Known problems
- The block is not aware of any change inside the widget until the update button is pressed. This replicates the save button on the widgets screen. But it is annoying if we change something on the widget and go to the widget preview right away our changes are not reflected in the preview. Having an explicit update, makes testing and debugging easier, we may than explore other approaches e.g.: also save when preview happens, save on blur events, etc.

- The text widget that contains TinyMCE crashes and fails to init. It calls wp.editor.initialize to reference TinyMCE and on Gutenberg, wp.editor is our editor module. This problem may have happened with meta boxes if it was solved probably the same approach may be applied.

- The widget design may be affected by CSS that exists in Gutenberg, so the design of the widgets does not look the same. Ideally, Gutenberg CSS would not affect the widgets but as they are on the same page that's not the case.

- Some third-party widgets using don't initialize correctly. That happens because the dom of the editor is not equal to the dom of the customizer and/or the widget screen. Some JS widgets use click events on the widgets screen to initialize while on Gutenberg these events don't happen, some check if they are on the customizer page (body contains customizer classes) before handling widget-updated events. Normally adapting a widget that does not initialize correctly is a matter of changing a very simple condition on the plugin.

* Update changelog for esling-plugin (#14339)

* Add ability to transform [video] shortcodes to video block (#14042)

* CSS to remove the clipping of the toolbar in RTL languages (#14088)

* added the more tag via the old editor's image (#14173)

* Seimplify hierarchical term selector strings. (#13938)

* Add: End to end test to make sure when all  blocks get removed the default block appender gets inserter and selected (#14191)

* Fix the Deprecated Blocks link (#14355)

The *Deprecated Blocks link* was pointing to the wrong URL.

* Added testcases for isKeyboardEvent in keycodes module (#14073)

* Added testcases for isKeyboardEvent in keycodes module

* Fixed assertion checks and updated attachEventListeners function

* Added testcases for isKeyboardEvent in keycodes module

* Fixed assertion checks and updated attachEventListeners function

* Fix: Quote to heading transform (#14348)

* Latest Comments block: use align supports flag + code cleanup (#11411)

* ToggleControl allows setting custom classes (#13804)

* ToggleControl allows setting custom classes

Related Issue: #11349 
This pull request fixes the "ToggleControl does not allow setting custom classes" problem. This lets the user to add a custom classname on the element, just like in TextControl.

* Update README of toggle-control

Add property className and its description

* Update packages/components/src/toggle-control/README.md

Formatted style of README

Co-Authored-By: AmartyaU <[email protected]>

* Edited text of button to be actionable (#14347)

* Block library: Remove all test snapshots for blocks (#14349)

* Fix: Inserter impossible to collapse panels while searching. (#13884)

* Make taxonomies test relieable. (#14340)

* edit-post: set up autogenerated API docs (#14271)

* Plugin: Remove postinstall step (#14353)

* Plugin: Remove postinstall step

* Framework: Run build for JS unit tests

Assumed previously relied on postinstall

* Add check to the merge function in headings and paragraph blocks (#13981)

If trying to merge a blank heading or paragraph, if this check isn't performed,
the result is the string 'Null' appearing in the text.

* Update internationalization process with complete updated example. (#13909)

* Update internationalization process with complete updated example.

* Minor edits

* Update i18n package documentation

Updates with links to expanded section in Gutenberg Handbook

Fixes instructions to use wp-cli which is the recommended tool for
creating pot files, and po2json to convert the format.

* Apply suggestions from code review

Co-Authored-By: mkaz <[email protected]>

* Remove duplicate documentation, just link to Handbook

* Batch of changes from reviews props @swisspiddy

* :shakes-fist-at-whitespace:

* Updates to include full .pot and .po files per @nosolosw review

* Add JSON translation example

* Fix syntax to make gutenberg build in this branch

* E2E Test Utils: Add missing babel/runtime dependency (#14374)

* Build: Only prompt clean if unclean (#14352)

* Remove unused mobile stylesheets

* Port editor refactoring for mobile

* fix names (#14382)

* Add withRegistry HigherOrderComponent (#14370)

* Fix block validation error message (#13499)

* Rename variable to make code more clear

* Fix display of expected and actual block HTML

The variable were in the wrong order, making the message confusing. It
would show the actual HTML as the expected HTML and visa versa.

* Change wording to be more clear

Expected & Actual were confusing words to use in this content. This
change makes the message actually reflect what the values are.

* Regenerate docs

* Fix: Global inserter does not validates block insert restrictions (#14020)

* Teach build and start commands to use Webpack default if none is provided (#13877)

* Use a default webpack config if none is provided

* Extract webpack utils from build command

* Update start command

* Add docs

* Add Webpack documentation

* Add example of how to overwrite the default plugins

* Do not export hasWebpackConfig

as it is not used anywhere else.

* Always pass webpack CLI args to command

* Update README

* Remove section on extending the default webpack config file

* Simplify array passing

Co-Authored-By: nosolosw <[email protected]>

* Simplify pushing to array

Co-Authored-By: nosolosw <[email protected]>

* Fix externals docs

* Use webpack instead of Webpack

* Add Changelog entry

* RichText: Fix prepareEditableTree (#14284)

* Fix use of __experimentalCreatePrepareEditableTree without __experimentalCreateOnChangeEditableValue

* Add unit tests

* Add module entry point to notices package.json file (#14388)

* Plugin: Remove PHP functions slated for 5.3 removal (#14380)

* Make @wordpress/edit-post compatible with native

* Unify styles for UnsupportedBlock after sass build default imports is fixed on native

* Remove scss variables and colors import after fixing sass build in gb-mobile

* RichText: change value to have separate keys for line and object formats (#13948)

* Add objects and lineFormats

* Update RichText

* Fix image toolbar

* Update format placeholder

* lineFormat => lines

* concatPair => mergePair

* Update selectedFormat checks

* Add some extra info to create docs

* Move create docs inline

* Merge lines and objects

* Fix typos

* Add getActiveObject unit tests

* Update docs

* Rebase

* Adjust unstableToDom arguments

* Remove normaliseFormats from list functions

* Update native files

* Update native file

* Remove code formatting for mobile

* RichText: try alternative list shortcuts (to tab) (#14343)

* RichText: try alternative list shortcuts

* Try tooltips

* Change tooltips to use text

* Add inline comments

* Add e2e test

* Rebase

* Remove unused styles classes. (#14338)

* [Mobile]Fix placeholder position of block appender (#14386)

* Fix placeholder position of block appender

* Remove unused import

* Change import position

* Remove unused import in css file

* Remove extra container view

* Update travis.yml from master branch to fix CI issues

* [Mobile]Update caret position on insert link (#14317)

* Update selection on text change if needed

* Update caret position when editing the link

* Revert putting the caret at end when we transform the selected text into link

* Fix source map paths published to npm (#14409)

* Components: update Button readme to add design guidelines (#14194)

These changes add design documentation. Co-Authored-By: kjellr <[email protected]>

* Remove export from syntax still a proposal

* Update README.md

* [RNmobile] Bring Placeholder text back working on Heading blocks (#14404)

* Remove the temporary fix for heading issue on Android, and let's pass the `tagName` value down to the Native wrapper

* Do not pass the `tagName` prop to AztecView

* The heading block now uses a minHeight defined in variables.scss, in the outer mobile project. This is a fix for an ugly rendering issue we're seeing with the placeholder when starting wrinting text.

* Add missing file from the prev commit

* Remove import of variaboe.scss since it's not required an explict import anymore after latest changes to GB.

* Add disableEditingMenu prop to manage showing editing menu on iOS (#14435)

* Fix merge errors

* Update README.md

* Fix tab linting error in scss

* [React Native] request cancel image upload (#14391)

* added hook blocks.onRemoveBlockCheckUpload called in componentWillUnmount to make sure to cancel upload if block is deleted

* removed unused import

* moving bridge-specific code to Image component - if the filter exists and returns true, the upload cancellation will be requested

* using hook actions instead of filters

* Fix lint errors

* Investigate travis error

* Line modifications for packages/block-editor/src/index.js

* Fix circle dependency

* Update README.md

* Revert travis logging

mkevins added a commit to mkevins/gutenberg that referenced this pull request Mar 26, 2019

RichText: collapse toolbar (WordPress#14233)
* RichText: collapse toolbar

* Move inline image

* Add balanced margin

* Add tooltip

* Update e2e test

* Update e2e test

* Remove RichTextInserterItem

* Ensure there are fills before rendering

* Make toggle button bigger

* Use clickBlockToolbarButton

* Fix e2e test

mkevins added a commit to mkevins/gutenberg that referenced this pull request Mar 26, 2019

Merge mobile development branch into master (v2) (WordPress#14503)
* Bump plugin version to 5.1.0-rc.1

* RichText: only ignore input types that insert HTML (WordPress#13914)

* RichText: only ignore input types that insert HTML

* Mark RichTextInputEvent as unstable

* Use Set

* Bump plugin version to 5.1.0

* Deprecate RichTextInputEvent on mobile too (WordPress#13975)

* The undelying RichText component implementation has changed the parameters returned onChange, and we forgot to update the PostTitle component (WordPress#13967)

* Fixes wrong state comparison (WordPress#13987)

Upload media progress bar is missing while media is uploading new

* Re-add rootTagsToEliminate prop (WordPress#14006)

* [Mobile]Update PostTitle to apply borders when it is focused (WordPress#13970)

* Trigger onFocusStatusChange from PostTitle

* Fix lint issue

* Update post title shadow mechanism

Also open inner ref so that focus state can be updated when focus is made programmatically

* Update props

* Update onRef as ref

* Update title padding&margin

* Mobile: Rename ref to innerRef on PostTitle (WordPress#14024)

* Fixes a red screen in mobile. (WordPress#14011)

* Change background color on image placeholder block (WordPress#14033)

* Changed upload media icon color

* Changed media placeholder background color

* Fix post title native syntax (WordPress#14041)

* Fix unexpected token in native code

* Dummy commit to trigger Travis

* Include the rnmobile release branch to Travis builds

* Mobile: Links UI using BottomSheet component (WordPress#13972)

* Mobile: Replaced Links UI with bottom-sheet component

* Mobile links UI: Removed commented code.

* Mobile: Fix lint issues

* Mobile Links UI: Remove autofocus on Android.
This hides an issue where the modal sometimes will be under the keyboard on Android.

* Fixes pasting links. (WordPress#14038)

* Update post title vertical paddings (WordPress#14040)

* Add try/catch fallback to plain text for pasteHandler (WordPress#14044)

* Fix link interface. (WordPress#14052)

* [Mobile]Fix title padding on Android (WordPress#14057)

* Remove title vertical paddings for Android

* Revert "Remove title vertical paddings for Android"

This reverts commit 09f0d35.

* Import padding variables

* Revert wrong format image color (WordPress#14058)

* Stop building the mobile release branch on Travis (WordPress#14060)

* [Native mobile] Bring release v1.0.1 back to "mobile develop" (WordPress#14075)

* Don't allow placeholder to moves once we tap on it (WordPress#14066)

* Use platform based SCSS vars for block minHeight (WordPress#14070)

* Rnmobile/refactor rich text sizing code (WordPress#14164)

* Make richText height changes contained to the rich text block.

* Remove commented out code.

* Remove aztec height from state.

* Allow minHeight to be optional.

* Remove minHeight from postTitle.

* Remove minHeight on heading block.

* Mobile: Fix pasting in header (WordPress#14118)

* Fixes pasting in heading blocks.

* Fixed an issue with a missing constant.

* Removes some code that was added back as part of a merge conflict.

* Re-adds some code removed by mistake.

* Mobile: Fix links ui on landscape iOS v2 (WordPress#14240)

* RNMobile: Adding original versin of keyboard-avoiding-view (to be modified)

* Modified keyboard-avoiding-view to work well with bottom-sheets.

* Mobile: Avoid adding empty link to text. (WordPress#14270)

* Mobile Links UI: Avoid creating link with empty url field

* Mobile Links UI: Using URL Keyboard type for url field

* Fix lint issues

* Avoid to reset html to empty string if block is heading and platform is android (WordPress#14301)

* Avoid to reset html to empty string if block is heading on android platform

* Send empty tag flag from heading block to RichText

* Set minHeight based on fontSize or style. (WordPress#14344)

* Set minHeight based on fontSize or style.

* Only use the styles to set the min-height.

* Remove unused styles classes. (WordPress#14338)

* [Mobile]Fix placeholder position of block appender (WordPress#14386)

* Fix placeholder position of block appender

* Remove unused import

* Change import position

* Remove unused import in css file

* Remove extra container view

* Update travis.yml from master branch to fix CI issues

* [Mobile]Update caret position on insert link (WordPress#14317)

* Update selection on text change if needed

* Update caret position when editing the link

* Revert putting the caret at end when we transform the selected text into link

* Get the last master changes into the mobile develop branch (WordPress#14375)

* Add ESNext syntax to meta block tutorial (WordPress#13954)

* Add ESNext syntax to meta block tutorial

* Applied WordPress code styles to the examples

* Apply suggestions from code review

Co-Authored-By: mkaz <[email protected]>

* Editor: RichText: Check for presence of inputType (WordPress#13986)

* Bump plugin version to 5.1.1 (WordPress#13990)

* Added a snippet for observing the browser when running e2e tests (WordPress#13993)

* Added a snippet for observing the browser when running e2e tests

* Extract reusable part of Webpack config and put in @wordpress/scripts (WordPress#13814)

* New build-config package with webpack config.

Pull the Gutenberg webpack config into a package so it can be re-used for
block/extension development.

* Require new build-config package.

* Dynamically handle WP externals with a function.

Use code from WP Calypso for handling WP externals so we don't have to have the
actual list of packages accessible in our webpack configuration.

* Use webpack config from build-config package.

* Require build-config package.

* Adjust file refs for WP packages.

* Move main gutenberg entry definition and webpack copy plugin out of build-config.

* Add react-dev-utils for formatting webpack compiler messages.

* Implement build script using webpack config from build-config.

* Adjust output path so build goes to working directory.

* Update package name to webpack-config

* Apply more tweaks to the way webpack config package is structured

* Update the way externals are handled

* Add default values for entry and output

* Move shared webpack config under @wordpress/scripts package

* Improve the way how loaders are handled

* Replace GUTENBERG with WP in webpack config env variables

Co-Authored-By: gziolo <[email protected]>

* Bring back feature flag to webpack config accidentally removed during merge

* Add missing dev dependencies for the packages used in webpack config

* Fix the list of excluded folders for babel-loader

* Use globals instead of imports in tutorials (WordPress#13995)

* Use globals instead of imports

* Update docs/designers-developers/developers/tutorials/metabox/meta-block-3-add.md

Co-Authored-By: nosolosw <[email protected]>

* Update docs/designers-developers/developers/tutorials/metabox/meta-block-3-add.md

Co-Authored-By: nosolosw <[email protected]>

* URL input popover visual cleanup (WordPress#13973)

* Use chevron instead of ellipsis in url input field options.

* Mimic toolbar icon styles for the icons in the URL popover.

* Add a left divider before the URL settings toggle

* Even up the spacing in the settings panel.

* Add periods to code comments.

* Update snapshot

* Reinstate "underline" in default formats list (WordPress#14008)

* Upgrade React to 16.8.2: Welcome React Hooks (WordPress#13992)

* Upgrade React to 16.8.2

* Update package-lock.json file

* Expose React Hooks

* Revert "Upgrade React to 16.8.2: Welcome React Hooks (WordPress#13992)" (WordPress#14017)

This reverts commit 6f88bec.

* Cleanup URL Popover stylesheet. (WordPress#14015)

As noted in WordPress#13973, this stylesheet uses a relatively non-standard SCSS method of nesting some classnames. For instance:

`.editor-url-popover { &__settings-toggle { ... } }`.

... instead of:

`.editor-url-popover__settings-toggle { ... }`

This is different from the conventions used elsewhere in Gutenberg, and is a bit more difficult to follow for that reason.

This commit un-nests those styles, and should have no effect on the compiled CSS.

* Add a generic block editor module (WordPress#13088)

* RichText: warn when using inline container (WordPress#13921)

* RichText: warn when using inline container

* Add env check

* Update documentation

* Try updating the WP database (WordPress#14048)

* Merge native mobile release v1.0 to master (WordPress#14061)

* Bump plugin version to 5.1.0-rc.1

* RichText: only ignore input types that insert HTML (WordPress#13914)

* RichText: only ignore input types that insert HTML

* Mark RichTextInputEvent as unstable

* Use Set

* Bump plugin version to 5.1.0

* Deprecate RichTextInputEvent on mobile too (WordPress#13975)

* The undelying RichText component implementation has changed the parameters returned onChange, and we forgot to update the PostTitle component (WordPress#13967)

* Fixes wrong state comparison (WordPress#13987)

Upload media progress bar is missing while media is uploading new

* Re-add rootTagsToEliminate prop (WordPress#14006)

* [Mobile]Update PostTitle to apply borders when it is focused (WordPress#13970)

* Trigger onFocusStatusChange from PostTitle

* Fix lint issue

* Update post title shadow mechanism

Also open inner ref so that focus state can be updated when focus is made programmatically

* Update props

* Update onRef as ref

* Update title padding&margin

* Mobile: Rename ref to innerRef on PostTitle (WordPress#14024)

* Fixes a red screen in mobile. (WordPress#14011)

* Change background color on image placeholder block (WordPress#14033)

* Changed upload media icon color

* Changed media placeholder background color

* Fix post title native syntax (WordPress#14041)

* Fix unexpected token in native code

* Dummy commit to trigger Travis

* Include the rnmobile release branch to Travis builds

* Mobile: Links UI using BottomSheet component (WordPress#13972)

* Mobile: Replaced Links UI with bottom-sheet component

* Mobile links UI: Removed commented code.

* Mobile: Fix lint issues

* Mobile Links UI: Remove autofocus on Android.
This hides an issue where the modal sometimes will be under the keyboard on Android.

* Fixes pasting links. (WordPress#14038)

* Update post title vertical paddings (WordPress#14040)

* Add try/catch fallback to plain text for pasteHandler (WordPress#14044)

* Fix link interface. (WordPress#14052)

* [Mobile]Fix title padding on Android (WordPress#14057)

* Remove title vertical paddings for Android

* Revert "Remove title vertical paddings for Android"

This reverts commit 09f0d35.

* Import padding variables

* Revert wrong format image color (WordPress#14058)

* Stop building the mobile release branch on Travis (WordPress#14060)

* Avoid mutating imported default config in webpack config (WordPress#14039)

* Make Babel import JSX pragma plugin aware of `wp.element.createElement` (WordPress#13809)

* Test: Add test which verifies wheter JSX pragma detects WP global

* Update packages/babel-plugin-import-jsx-pragma/test/index.js

Co-Authored-By: gziolo <[email protected]>

* Skip import when the scope variable is already defined

* Add failing tests for inner scope variable defined verification

* Add import statement when there is any undefined scope variable

* Docs: Add details about changes introduced to Babel plugin

* Upgrade Jest to version 24 (breaking changes) (WordPress#13922)

* Upgrade Jest to version 24 (breaking changes)

* Update changelog files to list all dependencies upgraded

* Downgrade puppeteer to the previous version 1.6.1

* Try to fix failing e2e tests setup

* Added clarification in the changelog

* Testing: Remove expect-puppeteer import reference

* Address issues raised during code review

* Codeowners changes for @chrisvanpatten (WordPress#14062)

* Codeowners changes for @chrisvanpatten

* Remove obsolete ghost  placeholder for no longer onowned styles

* Add repository.directory fields (fixes WordPress#13946) (WordPress#14059)

* Improve default Babel preset to include JSX pragma (WordPress#13540)

* Include custome JSX pragma support in Babel preset

* Stop using Babel tranpiliation for two packages
babel-plugin-import-jsx-pragma and postcss-themes

* Add engines field to node based packages

* Use the block editor store instead of the editor one (WordPress#13105)

* Docs: release.md: miscellaneous fixes (WordPress#14083)

* Docs: release.md: misc. fixes

* Disable block navigation and document outline items in text mode (WordPress#14081)

* Use aria-disabled instead of disabled.

* Fix unrecognized prop warning.

* Disable Document Outline items in text mode.

* Improve buttons alignment.

* Pass isTextModeEnabled as prop from parent.

* Fix selector in document outline. (WordPress#14094)

* Fix selector in document outline.

* Import getBlocks from its new location.

* Plugin: Require includes for deprecated `use_block_editor_for_` functions (WordPress#14096)

* Plugin: Fix 5.1.0 deprecated functions to correct plugin version

* Plugin: Require includes for deprecated `use_block_editor_for_` functions

* Plugin: Update server blocks script to use core equivalent function (WordPress#14097)

`gutenberg_prepare_blocks_for_js` was deprecated in Gutenberg 5.0

* Correct visual error in the quote block icon (WordPress#14091)

Fixes WordPress#13659.

The current quote block icon appears to have some over-simplified edges. This replaces it with a crisper version.

* Try: Add a subtle animation to the is-active indicator for sidebar tabs (WordPress#13956)

* Add subtle animation to the is-active indicator for sidebar tabs

* Re-instate the empty border, to prevent browser defaults from kicking in.

* Remove extra 1px of empty space

* Focus state cleanup for tab buttons

Allow them to inherit the box shadow (used for the active tab)

Use the usual standard dotted outline instead of a solid one.

* Add a pseudoclass to ensure the active border appears in Windows High Contrast Mode.

* Fix typo.

This should match the value from `/packages/edit-post/src/components/sidebar/settings-header/style.scss`

* Plugin: Remove vendor script registration (WordPress#13573)

* chore: Fix: FormToggle docs documents props the component does not uses (WordPress#14099)

* Babel Plugin Import JSX Pragma: Remove import visitor (WordPress#14106)

Props to @aduth for coding this optimization.

* Add: Block specific toolbar button sample to the format api tutorial (WordPress#14113)

## Description
This PR updates the format API tutorial (toolbar button section) to include a sample of a button that only renders on a certain block.
Answers a question posted on WordPress#14104.
Closes: WordPress#14104

## How has this been tested?
I pasted the sample code on the browser console and verified it works as expected.

* Use the editor settings to pass a mediaUpload handler (WordPress#14115)

* Use the editor settings to pass a mediaUpload handler

* Update media block snapshots

* Block Editor: Consider RECEIVE_BLOCKS as non-persistent change (WordPress#14108)

* Block Editor: Consider RECEIVE_BLOCKS as non-persistent change

* Block Editor: Compare last action in reducer enhancer only if non-ignored

* Plugin: Remove PHP functions slated for removal in 5.2 (WordPress#14090)

* Plugin: Remove PHP functions slated for removal in 5.2

* Documentation: Update FAQ to avoid reference to deprecated function

* Paste: ignore Google Docs UID tag (WordPress#14138)

* Fix  typos in copy-guide.md, readme.md and scripts.md. (WordPress#14089)

* Separate the block editor shortcuts from the post editor shortcuts (WordPress#14116)

* Fix: FocalPointPicker renders unlabelled input fields (WordPress#14152)

* Testing: Remove unnecessary Enzyme React 16 workarounds (WordPress#14156)

* Testing: Bump `enzyme-adapter-react-16` to 1.10.0

* Testing: Avoid unforwarded Button mock

No longer necessary with native support for forwardRef in Enzyme

* Testing: Un-skip BlockControls snapshot test

* Plugin: Preserve inline scripts in Gutenberg override (WordPress#13581)

* Plugin: Preserve inline scripts in Gutenberg override

* Plugin: Restore storageKey assignment for persistence migration

* Refactor to remove usage of post related effects in packages/editor. (WordPress#13716)

This pull is the first step in moving away from the lingering usage of effects in various data stores among packages. This pull specifically deals with post related effects in the @wordpress/editor package (`core/editor` store).

* Add array-callback-return rule; Fix current code breaking the rule. (WordPress#14154)

## Description
Adds a rule to make sure Array functions that iterate on the array and should return something contain the return statement, otherwise, a forEach should probably be used instead.
A case like this was fixed at WordPress#13953.

In PR WordPress#13953 @aduth suggested the implementation of a lint rule to catch these cases. While trying to implement the rule and researching the best ways to do it, I noticed a rule like that already existed in the community and this PR is enabling it.

We are also changing the code to respect the new rule no observable changes should be expected.

## How has this been tested?
Observe the tests pass.
Do some smoke testing, adding blocks, uploading files, and verify everything still works as before.

* Update text displayed when an embed can't be previewed (WordPress#13715)

* Update text displayed when an embed can't be previewed

* Add translator note for update to embedded content np preview message

* Fix: Image that is uploaded to an existing gallery does not appear in the edit gallery view (WordPress#12435)

* RichText: fix wordwise selection on Windows (WordPress#14184)

* RichText: fix wordwise selection on Windows

* Fix crtl ctrl typo.

* Add docs

* Chore: Update: Code Quality:  Remove some editor store references from block-editor (WordPress#14161)

We missed to update the editor references from selectPreviousBlock, and selectNextBlock.

## Tests

I added multiple blocks, I verified that when I remove a block the previous block still gets selected.

* Framework: Update package-lock.json for new Enzyme adapter (WordPress#14192)

* Notices: Remove inaccurate createNotice sole argument feature (WordPress#14177)

* New package to auto-generate public API documentation (WordPress#13329)

* Update nosolosw notifs (WordPress#14196)

* Make IconButton able to be referenced. (WordPress#14163)

* Make IconButton able to get referenced.

* Components: Forward IconButton ref as stateless function

* Components: Restore IconButton aria-pressed prop pass-through

* Paste: preserve empty table cells (WordPress#14137)

* Update Codeowners, add mkaz to docgen (WordPress#14198)

* Plugin: Remove wp-editor-font stylesheet override (WordPress#14176)

* Bring i18n functionality up to date (WordPress#12559)

* Plugin: Add Text Domain to plugin headers

* Bring i18n functionality up to date

* Plugin: Set Gutenberg script translations as default domain

Filter loading behavior to load from plugin translation files

* Testing: Update gutenberg_override_script per localization changes

* Plugin: Provide second argument for set script translations

Technically optional, only after WP5.1+, which is newer than the current minimum version supported by Gutenberg

* Fix: deleting the last block triggers a focus loss. (WordPress#14189)

## Description
This PR fixes a problem: if the post contains one block and it is removed the focus is lost.

This is a regression that happened when removeBlocks action was refactored to be a generator. We had an effect that inserts the default block during remove blocks action when certain conditions are met, this effect stopped working.
This PR removes the effect and makes sure everything is handled by the removeBlocks action creator.

End to end test available at WordPress#14191.

## How has this been tested?
I created a new post.
I wrote something in a paragraph I removed the paragraph using the remove button in the side menu and I verified the default block was added.

* Clarify the block editor settings from the post editor settings (WordPress#14082)

* Add repository.directory linting rule (fixes WordPress#13947) (WordPress#14200)

* Add repository.directory rule to npm-package-json-lint-config (fix WordPress#13947)

* Fix lock file (props @aduth)

* Fix rule, update changelogs

* Add PR reference

* Add directory field to packages/docgen

* Apply changes suggested by @ntwb

* Update CHANGELOG.md

* Plugin: Remove deprecated `_wpLoadGutenbergEditor`, `gutenberg` theme supports (WordPress#14144)

* Plugin: Remove deprecated `gutenberg` theme supports
* Plugin: Remove deprecated `_wpLoadGutenbergEditor`

* Bump plugin version to 5.2.0-rc.1 (WordPress#14210)

* Packages: Add missing expect-pupeteer dependency to e2e-tests package (WordPress#14212)

* Lowercase Block Editor to block editor as per the core spelling Best Practices (WordPress#14205)

## Description
I've updated all references to Block Editor to be lowercase as they aren't proper nouns. This conforms to the core spelling Best Practices;
“block editor” or “block-based editor” | “Block Editor” or “Gutenberg” | When referring to the new editor.
Reference - https://make.wordpress.org/core/handbook/best-practices/spelling/

Also ties back to the original discussion around the capitalization convention here;
WordPress#12856

Previously opened WordPress#14203 to handle the Classic Editor changes.

## How has this been tested?
Only tested that it didn't break anything.

## Types of changes
Mostly comment changes, there was one string change and a couple translator comment changes.

## 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/ -->
- [x] I've included developer documentation if appropriate. <!-- Handbook: https://wordpress.org/gutenberg/handbook/designers-developers/ -->

* Babel Preset Default: Avoid disabling regenerator option (WordPress#14130)

* Babel Preset Default: Remove redundant (defaulted) corejs option

* Scripts: Assign Babel runtime regenerator as externals in Webpack config

* Babel Preset Default: Avoid disabling regenerator option

* Testing: Await E2E promise interactions (WordPress#14219)

* Testing: Await E2E promise interactions

* Testing: Disable animations on every admin screen

* Testing: Refactor reusable block management to compare before/after entry count

* Lowercase classic editor when not in reference to the plugin. (WordPress#14203)

* Lowercase classic editor when not in reference to the plugin.

* Reverted heading to use capitalized as the heading choice throughout uses full capitalization style on headings so went back to make consistent as @torres126 recommended. Also updated the snap file to resolve the Travis CI error.

* Missed saving README.md where the revert on headding capitalization was in my editor.

* Fix: Calendar block: Always show current month for non post types on the editor (WordPress#13873)

* Fix allowed_block_types regression (WordPress#14229)

* Fix: Latest posts: Title is clickable across the full width of the row (WordPress#14109)

## Description
The title in latest posts block was clickable across the full width of the row, making it easy to click on the title by mistake.
This problem only affects the editor on the front end things worked as expected.

## How has this been tested?
I added the latest posts block.
I verified that the title is only clickable if the mouse is above the title and not above any part of its row.


## Screenshots <!-- if applicable -->
Before:
![feb-25-2019 21-49-15](https://user-images.githubusercontent.com/11271197/53371643-b3e3f380-3948-11e9-95f5-a8f235fabab2.gif)

After:
![feb-25-2019 21-53-30](https://user-images.githubusercontent.com/11271197/53371657-c2320f80-3948-11e9-8dfd-ab7ae4640955.gif)

* Blocks: Regenerate RSS, Search block fixtures (WordPress#14122)

* Component: date-time: Remove comparison to now when testing getMomentDate (WordPress#14230)

* RichText: Fix undo after pattern (WordPress#13917)

* Fix undo after pattern

* Update e2e test

* RichText: don't use DOM to add line padding (WordPress#13850)

* RichText: don't use DOM to add line padding

* Clean up

* Keep track of user inserted line breaks

* Mark isEditableTree param unstable

* Use a central script instead of one per package to generate docs (WordPress#14216)

* Update: Use escape key press instead of mouse movement to show block toolbar (WordPress#14247)

This PR updates our end to end tests to use escape key press instead of mouse movement to show the block toolbar. The PR follows a suggestion by @aduth in WordPress#14191 (comment)

## How has this been tested?
We only need to make sure end 2 end tests pass

* Switch the Travis badge from travis-ci.org to travis-ci.com. (WordPress#14250)

* Update the npm packages release process (WordPress#14136)

* Update plugin version to 5.2 (WordPress#14255)

* chore(release): publish

 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]
 - @wordpress/[email protected]

* Update the package changelogs after the npm release

* Add clickBlockToolbarButton end to end test util (WordPress#14254)

This PR adds an end 2 end test util that allows clicking in a block toolbar button and refactors existing code to use it.

## How has this been tested?
We only need to verify that the end 2 end tests pass.

* element: set up autogenerated API docs (WordPress#14269)

* Testing: Trash existing posts as admin user (WordPress#14244)

* escape-html: set up autogenerated API docs (WordPress#14268)

* html-entities: set up auto-generated API docs (WordPress#14267)

* keycodes: set up auto-generated API docs (WordPress#14265)

* Use currentColor for fill of placeholder icon—ensures icon contrasts with background color of block (WordPress#14257)

* a11y: set up auto-generated API docs (WordPress#14288)

* blob: set up auto-generated API docs (WordPress#14286)

* block-library: set up auto-generated API docs (WordPress#14282)

* compose: set up auto-generated API docs (WordPress#14278)

* dom: set up auto-generated API docs (WordPress#14273)

* i18n: set up auto-generated API docs (WordPress#14266)

* Clarify that we should rebase the branch when preparing the npm release branches (WordPress#14260)

* autop: set up auto-generated API docs (WordPress#14287)

* dom-ready: set up autogenerated API docs (WordPress#14272)

* block-editor: set up auto-generated API docs (WordPress#14285)

* Adding an e2e test verifying simple keyboard navigation through blocks (WordPress#13455)

* Adding an e2e test verifying simple keyboard navigation through blocks (Issue WordPress#12392) using bug resolved in WordPress#11773 as the basis for the steps

* Moving `navigateToContentEditorTop`, `tabThroughParagraphBlock`, `tabThroughBlockMoverControl` and `tabThroughBlockToolbar` to the parent scope. Using pressKeyWithModifier within navigateToContentEditorTop.

* Add nested blocks inside cover block (WordPress#13822)

* rich-text: set up autogenerated API docs (WordPress#14220)

* Add link to WordPress Support documentation (WordPress#14316)

* Remove users documentation (WordPress#14318)

* Fix the double dash issue (WordPress#14321)

The double dash was being converted to a single dash (&WordPress#8211;)

* docgen: Generate package docs in parallel (WordPress#14295)

* Add new actions for invalidating resolution caches (WordPress#14225)

* add new actions for invalidating resolution caches

* use path array for omit

* clarify logic

* CC-BY-3.0 is not GPLv2 compatible. (WordPress#14329)

* blocks: set up auto-generated API docs (WordPress#14279)

* deprecated: set up auto-generated API docs (WordPress#14275)

* priority-queue: set-up auto-generated API docs (WordPress#14262)

* shortcode: set up autogenerated API docs (WordPress#14218)

* viewport: set up autogenerated API docs in README (WordPress#14214)

* url: set up autogenerated API docs (WordPress#14217)

* redux-routine: set up autogenerated API docs (WordPress#14228)

* date: set up auto-generated API docs (WordPress#14276)

* block-serialization-default-parser: set up auto-generated API docs (WordPress#14280)

* RichText: collapse toolbar (WordPress#14233)

* RichText: collapse toolbar

* Move inline image

* Add balanced margin

* Add tooltip

* Update e2e test

* Update e2e test

* Remove RichTextInserterItem

* Ensure there are fills before rendering

* Make toggle button bigger

* Use clickBlockToolbarButton

* Fix e2e test

*  RichText: fix br padding in multiline and nesting (WordPress#14315)

* RichText: fix br padding in multiline and nesting

* Add e2e test

* Docs: Release: Suggest lighter SVN checkout (WordPress#14259)

* plugins: set up auto-generated API docs (WordPress#14263)

* Move the block components to the block editor module (WordPress#14112)

* wordcount: set up autogenerated API docs (WordPress#14213)

* Update packages used by eslint plugin to remove warnings (WordPress#14077)

* Update packages used by eslint plugin to remove warnings

* Manually modify package-lock.json

Props @gziolo.

* Specify React version in eslint-plugin-react settings

* Fix react/jsx-no-target lint errors

* Try Legacy widget block (WordPress#13511)

## Description
Implements: WordPress#4770

This PR is a **proof of concept** of a legacy widget block. A block that allows existing WordPress widgets to be added as Gutenberg blocks.


The design is similar to the one proposed by @melchoyce in WordPress#4770 (comment) (option 1). Although it seems option two was preferred, it would require to expose each widget as a block similar to what embeds do, and it would increase the technical complexity and make testing/debugging harder, so I preferred to use option 1 for now. I will gladly iterate on the UX after this proof of concept gets more stable.


## Some technical details

The available widgets are preloaded to Gutenberg similar to what happens with page templates.

### REST-API user story
I want to able to pass a widget identifier to an endpoint, pass the existing widget attributes and the changes the user is making if any, and receive from the rest API, the sanitized new widget attributes ready to save and an HTML form that allows the user to edit the widget in the new state.

###  REST-API endpoint
A very simple REST-API endpoint was implemented.  The endpoint receives the previous instance of a widget (previous attributes) the changed instance of a widget (changed attributes if any) and returns the new instance of the widget and the HTML form that allows editing this widget.
There are two ajax-admin endpoints save-widget and update-widget. It looks like each one has specificities that make using it here complex. The ajax admin code would probably require some changes to be used here. Our use case is straightforward from the backend perspective as the widget does not need to be saved anywhere, and the widget is not associated with any widget area. The most straightforward approach seemed to be using a very simple endpoint. If I missed something and adapting existing endpoints is simpler feel free to comment, and I will have a look.


### Block Architecture

The edit component of the block handles the start placeholder that allows selecting a widget, and the tab mechanism that allows switching between edit and preview.
The preview is done using the ServerSideRender component.
The edit is done using two components:
**WidgetEditHandler:** Is responsible for server communication using the endpoint we created, and for keeping the required local state for the edition. Renders an update button, when pressed we retrieve from the dom the changed fields (using a method provided by WidgetEditDomManager) issues a request to the server and updates the legacy widget instance attribute with the server answer.
**WidgetEditDomManager:** Component responsible for rendering the starting dom for a widget. After the first render React never rerenders the component again, the content rendered by this component is then managed by the scripts the widgets may implement. When a new instance of the form HTML is received we manually update the dom changing .widget-content (like the customizer and the widget screen) do. This component provides a method that returns an object with the widget changed attributes. When this component is mounted it triggers a widget-added event when a new update happens and the dom is changed by the component widget-update jQuery event is triggered.

On front end widget are rendered with a simple call to [the_widget](https://codex.wordpress.org/Function_Reference/the_widget).

 

## Screenshots <!-- if applicable -->

![jan-25-2019 19-37-44](https://user-images.githubusercontent.com/11271197/51768637-c3b5b100-20d8-11e9-941f-00adb4c7b0a1.gif)
![jan-25-2019 18-58-49](https://user-images.githubusercontent.com/11271197/51768645-cb755580-20d8-11e9-89e7-1aa9ba7256c3.gif)

## Known problems
- The block is not aware of any change inside the widget until the update button is pressed. This replicates the save button on the widgets screen. But it is annoying if we change something on the widget and go to the widget preview right away our changes are not reflected in the preview. Having an explicit update, makes testing and debugging easier, we may than explore other approaches e.g.: also save when preview happens, save on blur events, etc.

- The text widget that contains TinyMCE crashes and fails to init. It calls wp.editor.initialize to reference TinyMCE and on Gutenberg, wp.editor is our editor module. This problem may have happened with meta boxes if it was solved probably the same approach may be applied.

- The widget design may be affected by CSS that exists in Gutenberg, so the design of the widgets does not look the same. Ideally, Gutenberg CSS would not affect the widgets but as they are on the same page that's not the case.

- Some third-party widgets using don't initialize correctly. That happens because the dom of the editor is not equal to the dom of the customizer and/or the widget screen. Some JS widgets use click events on the widgets screen to initialize while on Gutenberg these events don't happen, some check if they are on the customizer page (body contains customizer classes) before handling widget-updated events. Normally adapting a widget that does not initialize correctly is a matter of changing a very simple condition on the plugin.

* Update changelog for esling-plugin (WordPress#14339)

* Add ability to transform [video] shortcodes to video block (WordPress#14042)

* CSS to remove the clipping of the toolbar in RTL languages (WordPress#14088)

* added the more tag via the old editor's image (WordPress#14173)

* Seimplify hierarchical term selector strings. (WordPress#13938)

* Add: End to end test to make sure when all  blocks get removed the default block appender gets inserter and selected (WordPress#14191)

* Fix the Deprecated Blocks link (WordPress#14355)

The *Deprecated Blocks link* was pointing to the wrong URL.

* Added testcases for isKeyboardEvent in keycodes module (WordPress#14073)

* Added testcases for isKeyboardEvent in keycodes module

* Fixed assertion checks and updated attachEventListeners function

* Added testcases for isKeyboardEvent in keycodes module

* Fixed assertion checks and updated attachEventListeners function

* Fix: Quote to heading transform (WordPress#14348)

* Latest Comments block: use align supports flag + code cleanup (WordPress#11411)

* ToggleControl allows setting custom classes (WordPress#13804)

* ToggleControl allows setting custom classes

Related Issue: WordPress#11349 
This pull request fixes the "ToggleControl does not allow setting custom classes" problem. This lets the user to add a custom classname on the element, just like in TextControl.

* Update README of toggle-control

Add property className and its description

* Update packages/components/src/toggle-control/README.md

Formatted style of README

Co-Authored-By: AmartyaU <[email protected]>

* Edited text of button to be actionable (WordPress#14347)

* Block library: Remove all test snapshots for blocks (WordPress#14349)

* Fix: Inserter impossible to collapse panels while searching. (WordPress#13884)

* Make taxonomies test relieable. (WordPress#14340)

* edit-post: set up autogenerated API docs (WordPress#14271)

* Plugin: Remove postinstall step (WordPress#14353)

* Plugin: Remove postinstall step

* Framework: Run build for JS unit tests

Assumed previously relied on postinstall

* Add check to the merge function in headings and paragraph blocks (WordPress#13981)

If trying to merge a blank heading or paragraph, if this check isn't performed,
the result is the string 'Null' appearing in the text.

* Update internationalization process with complete updated example. (WordPress#13909)

* Update internationalization process with complete updated example.

* Minor edits

* Update i18n package documentation

Updates with links to expanded section in Gutenberg Handbook

Fixes instructions to use wp-cli which is the recommended tool for
creating pot files, and po2json to convert the format.

* Apply suggestions from code review

Co-Authored-By: mkaz <[email protected]>

* Remove duplicate documentation, just link to Handbook

* Batch of changes from reviews props @swisspiddy

* :shakes-fist-at-whitespace:

* Updates to include full .pot and .po files per @nosolosw review

* Add JSON translation example

* Fix syntax to make gutenberg build in this branch

* E2E Test Utils: Add missing babel/runtime dependency (WordPress#14374)

* Build: Only prompt clean if unclean (WordPress#14352)

* Remove unused mobile stylesheets

* Port editor refactoring for mobile

* fix names (WordPress#14382)

* Add withRegistry HigherOrderComponent (WordPress#14370)

* Fix block validation error message (WordPress#13499)

* Rename variable to make code more clear

* Fix display of expected and actual block HTML

The variable were in the wrong order, making the message confusing. It
would show the actual HTML as the expected HTML and visa versa.

* Change wording to be more clear

Expected & Actual were confusing words to use in this content. This
change makes the message actually reflect what the values are.

* Regenerate docs

* Fix: Global inserter does not validates block insert restrictions (WordPress#14020)

* Teach build and start commands to use Webpack default if none is provided (WordPress#13877)

* Use a default webpack config if none is provided

* Extract webpack utils from build command

* Update start command

* Add docs

* Add Webpack documentation

* Add example of how to overwrite the default plugins

* Do not export hasWebpackConfig

as it is not used anywhere else.

* Always pass webpack CLI args to command

* Update README

* Remove section on extending the default webpack config file

* Simplify array passing

Co-Authored-By: nosolosw <[email protected]>

* Simplify pushing to array

Co-Authored-By: nosolosw <[email protected]>

* Fix externals docs

* Use webpack instead of Webpack

* Add Changelog entry

* RichText: Fix prepareEditableTree (WordPress#14284)

* Fix use of __experimentalCreatePrepareEditableTree without __experimentalCreateOnChangeEditableValue

* Add unit tests

* Add module entry point to notices package.json file (WordPress#14388)

* Plugin: Remove PHP functions slated for 5.3 removal (WordPress#14380)

* Make @wordpress/edit-post compatible with native

* Unify styles for UnsupportedBlock after sass build default imports is fixed on native

* Remove scss variables and colors import after fixing sass build in gb-mobile

* RichText: change value to have separate keys for line and object formats (WordPress#13948)

* Add objects and lineFormats

* Update RichText

* Fix image toolbar

* Update format placeholder

* lineFormat => lines

* concatPair => mergePair

* Update selectedFormat checks

* Add some extra info to create docs

* Move create docs inline

* Merge lines and objects

* Fix typos

* Add getActiveObject unit tests

* Update docs

* Rebase

* Adjust unstableToDom arguments

* Remove normaliseFormats from list functions

* Update native files

* Update native file

* Remove code formatting for mobile

* RichText: try alternative list shortcuts (to tab) (WordPress#14343)

* RichText: try alternative list shortcuts

* Try tooltips

* Change tooltips to use text

* Add inline comments

* Add e2e test

* Rebase

* Remove unused styles classes. (WordPress#14338)

* [Mobile]Fix placeholder position of block appender (WordPress#14386)

* Fix placeholder position of block appender

* Remove unused import

* Change import position

* Remove unused import in css file

* Remove extra container view

* Update travis.yml from master branch to fix CI issues

* [Mobile]Update caret position on insert link (WordPress#14317)

* Update selection on text change if needed

* Update caret position when editing the link

* Revert putting the caret at end when we transform the selected text into link

* Fix source map paths published to npm (WordPress#14409)

* Components: update Button readme to add design guidelines (WordPress#14194)

These changes add design documentation. Co-Authored-By: kjellr <[email protected]>

* Remove export from syntax still a proposal

* Update README.md

* [RNmobile] Bring Placeholder text back working on Heading blocks (WordPress#14404)

* Remove the temporary fix for heading issue on Android, and let's pass the `tagName` value down to the Native wrapper

* Do not pass the `tagName` prop to AztecView

* The heading block now uses a minHeight defined in variables.scss, in the outer mobile project. This is a fix for an ugly rendering issue we're seeing with the placeholder when starting wrinting text.

* Add missing file from the prev commit

* Remove import of variaboe.scss since it's not required an explict import anymore after latest changes to GB.

* Add disableEditingMenu prop to manage showing editing menu on iOS (WordPress#14435)

* Fix merge errors

* Update README.md

* Fix tab linting error in scss

* [React Native] request cancel image upload (WordPress#14391)

* added hook blocks.onRemoveBlockCheckUpload called in componentWillUnmount to make sure to cancel upload if block is deleted

* removed unused import

* moving bridge-specific code to Image component - if the filter exists and returns true, the upload cancellation will be requested

* using hook actions instead of filters

* Fix lint errors

* Investigate travis error

* Line modifications for packages/block-editor/src/index.js

* Fix circle dependency

* Update README.md

* Revert travis logging
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.