Update Press This' `normalize.css` 3rd party external library

[netweb]

Press This includes the ​Normalize library as part of press-this.css

Press This currently uses v3.0.0 of Normalize and the ​most recent version at time of writing is v4.2.0 which now includes support for Microsoft's Edge browser and various updates and tweaks for Chrome, Firefox, and Safari

Prior to Press This v2 being merged into core the original source file scss/_utilities/_normalize.scss included the Normalize library's comments and license, these were ​removed in preparation for the merge into core.

There have been 3 changes to Normalize library section of press-this.css since the library was introduced:

• r31566
• r37740
• r38152

The above changes should be tested with the latest Normalize (4.2) and merged into core with the original source comments and license.

Also adding a reference to the library in the "External Libraries" credits at wp-admin/credits.php

[kraftbj]

Thanks for catching this. I concur.

[netweb]

Also adding a reference to the library in the "External Libraries" credits at wp-admin/credits.php

We should add a credit yes? I've created ​#meta1932 with a patch for this.

In 37596.1.diff​ I've added section header comments per the ​CSS Coding Standards.

[jorbin]

What about moving normalize into its own file and then having press-this.css include it? Seems like that will make it easier to keep this up to date going forward.

[desrosj]

Replying to jorbin:

What about moving normalize into its own file and then having press-this.css include it? Seems like that will make it easier to keep this up to date going forward.

Do you see making Normalize available through the wp_enqueue_style()? If so, we can just add Normalize as a stylesheet dependency to press-this.css, and then theme/plugin authors could load it if desired.

[desrosj]

Added Normalize as a separate CSS file. Press This loads it via dependency. Themes can now enqueue Normalize to prevent loading it twice.

[ocean90]

Are there any issues with the current version which is used by Press This? I'm wondering if Press This requires all bundled normalizations or if we could use a trimmed version instead.

[jorbin]

@ocean90 I would really rather we didn't edit third party libraries without good cause. This is ~200 lines of css outside of comments and whitespace.

[ocean90]

@jorbin Sorry, I wasn't clear enough. I'm not talking about editing the library. I was thinking about extending our existing selectors with reset rules, if necessary, and remove Normalize as a dependency.
I just removed the Normalize rules from press-this.css and it doesn't look that bad.

[azaozz]

Agree with @ocean90. Not sure we need yet another dependency that is used only in one place for one thing. CSS normalization used to be a "big thing" long ago when IE6 was around. There was even "global" normalization for all of the admin CSS. That was removed years ago, don't see a compelling reason to add back something similar.

Would be good to hear what @michaelarestad (who made that CSS) thinks about updating the normalization. As far as I see it's not needed. Seems it was used as a base/good place to start from as the Press This CSS is separate from the rest of wp-admin CSS.

[netweb]

FYI: Normalize has been updated to v5, so if this is to go in, needs-refresh.

​https://github.com/necolas/normalize.css/blob/master/CHANGELOG.md#500-october-3-2016

[desrosj]

Updated patch including Normalize 5.0.0, in case we do decide to move forward with this method.

[desrosj]

Press This without Normalize dependency

[desrosj]

No normalize dependency, mobile.

[desrosj]

I updated the patch to include the 5.0.0 version of Normalize. I also attached some screenshots to show issues when not loading Normalize as a dependency. There are not that many. After first glance the following needs work:

1. Publish buttons
2. Mobile tag icon top right used for opening the slide out menu
3. Margin around the outside of the page.
4. Media items are larger in the import area above the editor.

I do not have the ability to test some older browsers, but for current versions the changes needed to eliminate the dependency are minimal.

[ocean90]

See comment:12.

[kraftbj]

Will continue the conversation at ​https://github.com/WordPress/press-this/issues/8 with PT being moved into a plugin and out of Core.