Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3938)

Unified Diff: chrome/common/extensions/docs/static/sass/_article.scss

Issue 148663007: Updated DCC navigation Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/common/extensions/docs/static/sass/_article.scss
diff --git a/chrome/common/extensions/docs/static/sass/_article.scss b/chrome/common/extensions/docs/static/sass/_article.scss
index 830ebf730fb7909ff470b0c7203c6bea4a76d533..a3b441f50e2b0a12fe62c829eac261a4dc8a40a2 100644
--- a/chrome/common/extensions/docs/static/sass/_article.scss
+++ b/chrome/common/extensions/docs/static/sass/_article.scss
@@ -1,5 +1,100 @@
-$toc-width: 240px;
-$toc-margin-left: 195px;
+$article-width: 70%;
+$toc-width: 20%;
+
+// Site navigation on all pages
+.inline-site-toc {
+ line-height: 1.3em;
+
+ a,
+ a:link,
+ a:visited {
+ color: $gray-medium;
+ font-weight: normal;
+
+ &:hover,
+ &:focus {
+ color: $black;
+ }
+ }
+ li li a,
+ li li a:link,
+ li li a:visited {
+ color: $gray-medium;
+
+ &:hover,
+ &:focus {
+ color: $black;
+ }
+ }
+
+ a {
+ @include display-flex();
+ padding: 0.5em 0;
+ }
+
+ .site-related {
+ display: block;
+ background-color: $gray-light;
+ padding: 1em 1em 0.5em 1em;
+ margin-bottom: 1em; /* so the box shadow doesn't get cut off */
+
+ h3 {
+ margin-top: 0;
+ }
+
+ li a {
+ &.active {
+ color: $black;
+ }
+ }
+ }
+
+ #toc {
+ display: block;
+
+ .toplevel {
+ > a {
+ font-weight: bold;
+ color: $black;
+
+ &.hastoc::after {
+ content: '+';
+ @include flex(1);
+ text-align: right;
+ }
+ }
+ }
+
+ //> .toc > li {
+ .toplevel {
+ &.active {
+ .toc {
+ display: block;
+ }
+ > a.hastoc {
+ &::after {
+ content: ''; /* don't make it look like you can toggle it closed */
+ }
+ }
+ }
+ }
+ }
+
+ .toc {
+ margin: 0;
+ padding: 0.3em 0 0 0;
+ border-top: $default-border;
+
+ .toc {
+ display: block;
+ }
+
+ .toc li {
+ padding-left: 1em;
+ border-bottom: $default-border;
+ }
+ }
+}
// TOC on article pages
.inline-toc {
@@ -34,6 +129,13 @@ $toc-margin-left: 195px;
.related {
display: block;
+ background-color: $gray-light;
+ padding: 1em 1em 0.5em 1em;
+ margin-bottom: 1em; /* so the box shadow doesn't get cut off */
+
+ h3 {
+ margin-top: 0;
+ }
li a {
&.active {
@@ -46,7 +148,7 @@ $toc-margin-left: 195px;
display: none;
.toplevel {
- > a {
+ > a {
font-weight: bold;
color: $black;
@@ -108,28 +210,43 @@ $toc-margin-left: 195px;
// Large than mobile.
@media only screen and (min-width: $break-small) {
- .inline-toc {
+ .inline-site-toc {
position: absolute;
top: 0;
- left: 50%;
- margin-left: $toc-margin-left;
width: $toc-width;
overflow: auto;
overflow-x: hidden;
- // scroll.js adds and removes the floating class depending on the scroll position.
- &.sticky {
- position: fixed;
+ #toc {
+ display: block;
+ // article.js adds and removes the fixed nav depending on the scroll position.
}
+ }
+
+ .inline-toc {
+ position: absolute;
+ top: 0;
+ right: 0;
+ width: $toc-width;
+ overflow: auto;
+ overflow-x: hidden;
#toc {
display: block;
+ // article.js adds and removes the fixed nav depending on the scroll position.
+ &.sticky {
+ top: 0;
+ position: fixed;
+ -webkit-transform: translateZ(0); /* repaint issue */
+ }
}
}
.article-content {
- width: 70%;
+ //position: fixed;
+ width: $article-width;
padding-right: 5%;
border-right: 1px solid $gray-light;
+ min-height: 250px; /* add min-height so fatnav and sidenav doesn't overlap badly */
}
.cc-logo {
margin: 0 0 0 auto;
@@ -140,7 +257,7 @@ $toc-margin-left: 195px;
@media only screen and (min-width: $break-small + 1) and (max-width: $break-large) {
.inline-toc {
width: $toc-width - 40;
- margin-left: $toc-margin-left - 30;
+ //margin-left: $toc-margin-left - 30;
}
}
@@ -178,5 +295,8 @@ $toc-margin-left: 195px;
margin: $default-padding 0;
}
}
+ .hidden {
+ display: none
+ }
}

Powered by Google App Engine
This is Rietveld 408576698