Merge pull request #4 from amzrk2/dev

Merge branch "dev" into "master"

4 files changed, 36 insertions, 21 deletions

diff --git a/assets/sass/_list.scss b/assets/sass/_content.scss
index 30e94da..c92a3d4 100644
--- a/assets/sass/_list.scss
+++ b/assets/sass/_content.scss
@@ -11,6 +11,7 @@
     padding: 1.5em 0 1.5em 0;
 }
 
+// different padding of first & last child in archives page
 #post-archive {
     &:not(:first-child):not(:last-child) {
         padding: 0.5em 0 0.5em 0;
@@ -31,6 +32,7 @@
     border-top: $spliter;
 }
 
+// post title font & color
 #post-title {
     font-size: $font-size-l1;
 
@@ -43,6 +45,7 @@
     @include link-primary();
 }
 
+// post meta data style
 #post-meta {
     color: $color-mute;
     padding: 0.5em 0 1em 0;
@@ -75,29 +78,34 @@
     @include link-secondary();
 }
 
+// read more link style
 #post-exlink {
     margin: 0.5em 0 0 0;
 
     @include link-secondary();
 }
 
+// current tag indicator style in tags page
 #page-tag {
     padding: 1.5em 0 1.5em 0;
     font-size: $font-size-l3;
     border-bottom: $spliter;
 }
 
+// markdown article style
 article {
     padding-top: 1.5em;
     padding-bottom: 1.5em;
 }
 
+// in-post license style
 #license {
     @include link-primary();
 
     margin-bottom: 1.5em;
 }
 
+// comment loading indicator style
 #post-loading {
     border-top: $spliter;
     margin: 0 0 1.5em 0;
@@ -114,7 +122,7 @@ article {
     }
 }
 
-// not show while iframe not loaded
+// spliter between comment & post will not show while iframe not loaded
 #post-comment .utterances-frame {
     border-top: $spliter;
 }
diff --git a/assets/sass/_pagination.scss b/assets/sass/_pagination.scss
index 8e5d248..6c5200b 100644
--- a/assets/sass/_pagination.scss
+++ b/assets/sass/_pagination.scss
@@ -1,28 +1,26 @@
 #pagination {
-    font-size: $font-size-l3;
     text-align: center;
     // spliter between pagination and last post block
     border-top: $spliter;
     padding: 1em 0 1em 0;
 
-    ul li {
+    li {
         display: inline;
         @include link-primary();
 
-        // darker number when active
-        &.active a {
-            color: $color-primary-dark;
+        &.pag-item a {
+            vertical-align: middle;
         }
 
-        // remove prev page at first page, next page at last page
-        &.disabled {
-            display: none;
+        // darker number when active
+        &.pag-current a {
+            color: $color-secondary;
         }
+    }
 
-        // remove first page link and last page link
-        &:first-child,
-        &:last-child {
-            display: none;
-        }
+    svg {
+        height: 1em;
+        width: 1em;
+        vertical-align: middle;
     }
 }
\ No newline at end of file
diff --git a/assets/sass/_sidebar.scss b/assets/sass/_sidebar.scss
index 88a39d8..640494d 100644
--- a/assets/sass/_sidebar.scss
+++ b/assets/sass/_sidebar.scss
@@ -40,12 +40,16 @@
 }
 
 #container-progress {
+    cursor: pointer;
     position: fixed;
     width: 3em;
     right: 1em;
     bottom: 1em;
+    color: $color-primary;
 
-    @include link-primary();
+    &:hover {
+        color: $color-secondary;
+    }
 
     .progressbar-text {
         svg {
diff --git a/assets/sass/main.scss b/assets/sass/main.scss
index 13c1fdf..6356cb5 100644
--- a/assets/sass/main.scss
+++ b/assets/sass/main.scss
@@ -1,7 +1,10 @@
-// Cover
+//variables
+@import "_var";
+
+// cover primer variables
 @import "_primer.scss";
 
-// Only used part of @primer/css/core
+// include only used part of @primer/css/core
 // Include .octicon base styles
 @import "@primer/octicons/index.scss";
 // Global requirements
@@ -15,21 +18,23 @@
 // Utilities always go last so that they can override components
 @import "@primer/css/utilities/index.scss";
 
-@import "_var";
-
 @import "_header";
 
-@import "_list";
+// list page & single page content part style
+@import "_content";
 
+// sidebar & progress indicator style
 @import "_sidebar";
 
+// markdown style cover in summary & in-post content
 @import "_markdown";
 
 @import "_pagination";
 
 @import "_footer";
 
+// for 404 page
 #not-found {
     @include pdsans-title();
     padding-top: 1em;
-}
+}
\ No newline at end of file