From 08d6560282dcd46710d7ad2db4ab8a4bb969e116 Mon Sep 17 00:00:00 2001 From: Amaury Levé Date: Fri, 22 Jun 2018 14:41:52 +0200 Subject: Rule S4655: '!important' should not be used on 'keyframes' (#61) --- .../org/sonar/l10n/css/rules/css/S4655.html | 23 ++++++++++++++++++++++ .../org/sonar/l10n/css/rules/css/S4655.json | 16 +++++++++++++++ .../l10n/css/rules/css/Sonar_way_profile.json | 1 + 3 files changed, 40 insertions(+) create mode 100644 sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/S4655.html create mode 100644 sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/S4655.json (limited to 'sonar-css-plugin/src/main/resources') diff --git a/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/S4655.html b/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/S4655.html new file mode 100644 index 0000000..e0653cf --- /dev/null +++ b/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/S4655.html @@ -0,0 +1,23 @@ +

!important within keyframes declarations is completely ignored in some browsers and therefore it should not be used to be consistent +among all browsers.

+

Noncompliant Code Example

+
+@keyframes kf {
+  from { margin-top: 50px; }
+  50%  { margin-top: 150px !important; } /* Noncompliant; ignored */
+  to   { margin-top: 100px; }
+}
+
+

Compliant Solution

+
+@keyframes kf {
+  from { margin-top: 50px; }
+  50%  { margin-top: 150px; }
+  to   { margin-top: 100px; }
+}
+
+

See

+ + diff --git a/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/S4655.json b/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/S4655.json new file mode 100644 index 0000000..75a203f --- /dev/null +++ b/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/S4655.json @@ -0,0 +1,16 @@ +{ + "title": "\"!important\" should not be used on \"keyframes\"", + "type": "BUG", + "status": "ready", + "remediation": { + "func": "Constant\/Issue", + "constantCost": "1min" + }, + "tags": [ + + ], + "defaultSeverity": "Major", + "ruleSpecification": "RSPEC-4655", + "sqKey": "S4655", + "scope": "Main" +} diff --git a/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/Sonar_way_profile.json b/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/Sonar_way_profile.json index 613e33f..8f0de75 100644 --- a/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/Sonar_way_profile.json +++ b/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/Sonar_way_profile.json @@ -4,6 +4,7 @@ "S4647", "S4652", "S4653", + "S4655", "S4663", "S4667" ] -- cgit v1.2.3