Use upper camel case class name for component selector

terkel · Jun 14, 2015 · 19fb79e · 19fb79e
1 parent b522cb3
commit 19fb79e
Show file tree

Hide file tree

Showing 9 changed files with 17 additions and 23 deletions.
diff --git a/component/_archivesList.scss b/component/_archivesList.scss
@@ -1,4 +1,4 @@
-.archivesList[data-component] {
+.ArchivesList {
     @include vertical-rhythm(line-height, 5/6);
     padding: 0;
 

diff --git a/component/_articleSummary.scss b/component/_articleSummary.scss
@@ -1,4 +1,4 @@
-.articleSummary[data-component] {
+.ArticleSummary {
     @include vertical-rhythm(margin-bottom, 9/6);
     @include vertical-rhythm(margin-top, 9/6);
 

diff --git a/component/_homeSubsection.scss b/component/_homeSubsection.scss
@@ -1,6 +1,6 @@
-.homeSubsection[data-component] {
+.HomeSubsection {
 
-    & + .homeSubsection {
+    & + & {
         @include vertical-rhythm(margin-top, 9/6);
         @include vertical-rhythm(padding-top, 9/6);
         box-shadow: inset 0 1px $color-border;

diff --git a/component/_mainBody.scss b/component/_mainBody.scss
@@ -1,4 +1,4 @@
-.mainBody[data-component] {
+.MainBody {
     @include vertical-rhythm(margin-bottom, 3);
     @include vertical-rhythm(margin-top, 3);
 
@@ -22,10 +22,4 @@
             font-family: $font-family-sans, $font-family-gothic, sans-serif;
         }
     }
-
-    a {
-        // &:hover {
-        //     color: inherit;
-        // }
-    }
 }
diff --git a/component/_mainHeader.scss b/component/_mainHeader.scss
@@ -1,4 +1,4 @@
-.mainHeader[data-component] {
+.MainHeader {
     @include vertical-rhythm(padding-bottom, 1.5);
     @include vertical-rhythm(padding-top, 3);
     box-shadow: 0 1px $color-border;

diff --git a/component/_pageFooter.scss b/component/_pageFooter.scss
@@ -1,4 +1,4 @@
-[role="contentinfo"][data-component] {
+.PageFooter {
     @include vertical-rhythm(padding-bottom, 2);
     @include vertical-rhythm(padding-top, 1);
     box-shadow: inset 0 1px $color-border;

diff --git a/component/_pageHeader.scss b/component/_pageHeader.scss
@@ -1,4 +1,4 @@
-[role="banner"][data-component] {
+.PageHeader {
     @include vertical-rhythm(padding-top, 3/6);
     background-color: $color-background;
     box-shadow: 0 1px $color-border;

diff --git a/component/_primaryNav.scss b/component/_primaryNav.scss
@@ -1,4 +1,4 @@
-.primaryNav[data-component] {
+.PrimaryNav {
     @include vertical-rhythm(padding-bottom, 2/6);
     @include vertical-rhythm(padding-top, 2/6);
     align-items: center;

diff --git a/main.scss b/main.scss
@@ -14,14 +14,14 @@
 @import "base/base";
 
 // Component
-@import "component/pageHeader";
-@import "component/mainHeader";
-@import "component/mainBody";
-@import "component/primaryNav";
-@import "component/pageFooter";
-@import "component/articleSummary";
-@import "component/homeSubsection";
-@import "component/archivesList";
+@import "component/PageHeader";
+@import "component/MainHeader";
+@import "component/MainBody";
+@import "component/PrimaryNav";
+@import "component/PageFooter";
+@import "component/ArticleSummary";
+@import "component/HomeSubsection";
+@import "component/ArchivesList";
 
 // Font
 @import "font/merriweather";