Remove non-color related styling from Modern color scheme

[johnbillion]

The Modern color scheme was introduced in #50504. In addition to its colors, it adds styling that increases the bottom padding on admin menu items.

This was pointed out in 32:ticket:50504.

No explanation for the padding was included in the change, so unless there is a good reason for it then it should be removed. Colour schemes should only affect colours otherwise we risk diverging from a consistent and predictable layout, especially if core CSS rules subsequently change.

cc @Joen - do you have any information about this increased admin menu padding?

[Joen]

Hi!

I believe that my have been an error on my part. It looks like a remainder of a failed separate exploration. Thanks for the ticket, should be safe to remove. Would you like a PR?

[johnbillion]

Great, thanks for the info Joen. Yes please!

This PR fixes a padding value present in the modern scheme, which shouldn't be there.

Fixes https://core.trac.wordpress.org/ticket/51127

Before:

<img width="200" alt="Screenshot 2020-08-25 at 09 39 22" src="​https://user-images.githubusercontent.com/1204802/91146756-9ca9d180-e6b7-11ea-9975-06f8eb71f210.png">

After:

<img width="179" alt="Screenshot 2020-08-25 at 09 42 52" src="​https://user-images.githubusercontent.com/1204802/91146763-9f0c2b80-e6b7-11ea-92bd-950879f5a479.png">

This PR fixes a padding value present in the modern scheme, which shouldn't be there.

Fixes https://core.trac.wordpress.org/ticket/51127

Before:

<img width="200" alt="Screenshot 2020-08-25 at 09 39 22" src="​https://user-images.githubusercontent.com/1204802/91146756-9ca9d180-e6b7-11ea-9975-06f8eb71f210.png">

After:

<img width="179" alt="Screenshot 2020-08-25 at 09 42 52" src="​https://user-images.githubusercontent.com/1204802/91146763-9f0c2b80-e6b7-11ea-92bd-950879f5a479.png">

[Joen]

☝️ Patch here: ​https://github.com/WordPress/wordpress-develop/pull/497

[Joen]

I messed up the keywords!

[johnbillion]

Looks good. I see no reason for this not to go into 5.5.1. Any objections?

[Joen]

No objections!

I'm unable to merge this branch, but please do let me know if you need my help with anything specific here!

[SergeyBiryukov]

In 48862:

Administration: Remove non-color related styling from Modern color scheme.

Color schemes should only affect colors, to avoid diverging from a consistent and predictable layout, especially if core CSS rules subsequently change.

Follow-up to [48277], [48286], [48345].

Props Joen, johnbillion, afercia, johnjamesjacoby.
Fixes #51127. See #50504.

[SergeyBiryukov]

In 48863:

Administration: Remove non-color related styling from Modern color scheme.

Color schemes should only affect colors, to avoid diverging from a consistent and predictable layout, especially if core CSS rules subsequently change.

Follow-up to [48277], [48286], [48345].

Props Joen, johnbillion, afercia, johnjamesjacoby.
Merges [48862] to the 5.5 branch.
Fixes #51127. See #50504.

Merged in https://core.trac.wordpress.org/changeset/48862