Apply the seven style guide to the status report

Delete empty classes and row color is OK.

The error and warning table rows has got background colors. It is set in system.theme.css, and also used by status/warning/error messages. See the screenshot. And when hover, it's get the light blue bg.
table.system-status-report tr.ok:not(:hover) {background-color: transparent;}, used to hide that color, but table row can use the table hover color, as excpected the styleguide.

Please help to decide, which style is better.

OK, i made a patch, where i moved styles to the seven stylesheet, from core.
Delete empty classes.

Set transparent background on error/warning/ok rows, but in a sorthen way.

The patch no longer applies as styles.css has been moved into the CSS folder since the patch was made. Not sure if this means a reroll or the patch being manually adjusted as files have moved around in.

Thank you for noticing! Updated.
Removed .hide-text class, because it not used anymore on status report page.

Status update.

One more change: remove border from left and right side of the table.

Can you upload a patch file? Your files have .txt extenstions and then I will review it :)

Thanks Emma!

Overview of work.

Changes to Seven
Matched the Status Report to the Seven style guide (spacing and borders), the /admin/structure type layout (spacing and borders) and the Messages styling (icon spacing)

Changes to system theme CSS:
Removed the OK green styling because of the decision at the end of the mother issue #665790: Redesign the status report page.

Screenshot of Seven

As I removed something from system theme css here are screenshots of Bartik and Stark for good measure.

Screenshot of Stark

Screenshot of Bartik

Looks good to me!

+++ b/core/themes/seven/css/components/tables.css
@@ -1,4 +1,4 @@
+f/**

What is that "f" thing there? :)

Erm.... I will remove the f and an extra line space I just noticed somewhere else.

New patch with mistakes corrected and an interdiff.

Oooh, lovely. Yes please.

HEAD was broken. back to rtbc.

Testbot was broken last night, back to RTBC

Sure, sure, everything is broken, it's not the patch... ;)

So it turns out there are two places the table row colours are declared and the old colours are stronger selectors than the Seven Styleguide colours.

I have removed the old colours with the over specific CSS selectors, the rest of the patch is still the same.

Screenshots show table rows for error and warning, using styles shared with messages (that have the Seven Styleguide colours in place).

In the following screenshot, you can see that the hover effect is not working neither on warning nor error rows since the background color stays yellow and read once they are hovered.

That's intentionally, see comment #33

I am not sure I understand what is the issue. Can you explain it please?

Hmm, tested the latest patch with simplytest.me and I see that the hover effect *does* get applied to warning messages. I think this needs work to exclude errors and messages from the hover effect.

I added the .info class to existing tr:hover and tr:focus since the hover effect has to be applied only to the rows with that class.

It seems like we are missing some changes from the latest patch

I have created a new patch based off of #31 with added selectors to keep the error and warning rows red and yellow when you hover over them.

Here's a quick screenshot...

This issue is an unfrozen change (css) as per https://www.drupal.org/core/beta-changes and it's benefits outweigh any disruption. Committed c58599c and pushed to 8.0.x. Thanks!