diff --git a/app/node_views/results.html b/app/node_views/results.html index 301788a..226dde7 100644 --- a/app/node_views/results.html +++ b/app/node_views/results.html @@ -504,6 +504,20 @@ +
+
Redundant tags selectors
+
+ {{phantomasResults.metrics.cssRedundantChildNodesSelectors}} + +
+
+
+ +

Some tags included inside other tags are obvious. For example, when "ul li" is specified in a rule, "ul" can be removed because the "li" element is always inside a "ul". Same thing for "tr td", "select option", ...

+

Lowering compexity in CSS selectors can make the page load a little faster.

+
+
+
diff --git a/app/public/scripts/resultsCtrl.js b/app/public/scripts/resultsCtrl.js index 6c2088f..23be2f7 100644 --- a/app/public/scripts/resultsCtrl.js +++ b/app/public/scripts/resultsCtrl.js @@ -263,7 +263,7 @@ app.controller('ResultsCtrl', function ($scope) { var note = 'A'; var score = $scope.phantomasResults.metrics.cssRules + $scope.phantomasResults.metrics.cssComplexSelectors * 5 + - $scope.phantomasResults.metrics.cssComplexSelectorsByAttribute * 15; + $scope.phantomasResults.metrics.cssComplexSelectorsByAttribute * 10; if (score > 500) { note = 'B'; } @@ -298,12 +298,13 @@ app.controller('ResultsCtrl', function ($scope) { $scope.phantomasResults.metrics.cssOldIEFixes * 10 + $scope.phantomasResults.metrics.cssOldPropertyPrefixes + $scope.phantomasResults.metrics.cssUniversalSelectors * 5 + - $scope.phantomasResults.metrics.cssRedundantBodySelectors + - ($scope.phantomasResults.metrics.cssImports || 0) * 50; + $scope.phantomasResults.metrics.cssRedundantBodySelectors * 0.5 + + $scope.phantomasResults.metrics.cssRedundantChildNodesSelectors * 0.5 + + $scope.phantomasResults.metrics.cssImports * 50; if (score > 50) { note = 'B'; } - if (score > 10) { + if (score > 100) { note = 'C'; } if (score > 200) {