From 122bcd5f9abb5ddc38709263de0e5a719b252314 Mon Sep 17 00:00:00 2001 From: James Koster Date: Fri, 5 Aug 2016 15:47:54 +0100 Subject: [PATCH] Remove unnecessary `!important`s in css and explain ones that are necessary. closes #11415 --- assets/css/woocommerce-smallscreen.scss | 4 ++-- assets/css/woocommerce.scss | 16 ++++++++-------- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/assets/css/woocommerce-smallscreen.scss b/assets/css/woocommerce-smallscreen.scss index 0309d868849..9c7cf8cd6f3 100644 --- a/assets/css/woocommerce-smallscreen.scss +++ b/assets/css/woocommerce-smallscreen.scss @@ -41,7 +41,7 @@ text-align: right !important; // Important to overwrite order status inline styling &.order-actions { - text-align: left !important; + text-align: left !important; // This must always align left on handheld } &:before { @@ -109,7 +109,7 @@ &:nth-child(2n) { float: right; - clear: none !important; + clear: none !important; // This should never clear. } } } diff --git a/assets/css/woocommerce.scss b/assets/css/woocommerce.scss index 296f391165b..53daff01c87 100644 --- a/assets/css/woocommerce.scss +++ b/assets/css/woocommerce.scss @@ -69,13 +69,13 @@ p.demo_store { text-align: center; line-height: 1; border-radius: 100%; - color: red !important; + color: red !important; // Required for default theme compatibility text-decoration: none; font-weight: 700; border: 0; &:hover { - color: #fff !important; + color: #fff !important; // Required for default theme compatibility background: red; } } @@ -1496,13 +1496,13 @@ p.demo_store { .woocommerce-message, .woocommerce-error, .woocommerce-info { - padding: 1em 2em 1em 3.5em !important; - margin: 0 0 2em !important; + padding: 1em 2em 1em 3.5em; + margin: 0 0 2em; position: relative; background-color: lighten($secondary,5%); color: $secondarytext; border-top: 3px solid $primary; - list-style: none outside !important; + list-style: none outside; @include clearfix(); width: auto; word-wrap: break-word; @@ -1521,9 +1521,9 @@ p.demo_store { } li { - list-style: none outside !important; - padding-left: 0 !important; - margin-left: 0 !important; + list-style: none outside !important; // Required for default theme compatibility + padding-left: 0 !important; // Required for default theme compatibility + margin-left: 0 !important; // Required for default theme compatibility } }