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

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

Issue 291663002: Adjustments to the fatnav and sidenav, and other CSS updates (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Added missing copyright license. Fixed bad indentation on search.js Created 6 years, 6 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..c11e9b8f12672baaa13417782cee47f2bc36bb48 100644
--- a/chrome/common/extensions/docs/static/sass/_article.scss
+++ b/chrome/common/extensions/docs/static/sass/_article.scss
@@ -1,5 +1,5 @@
-$toc-width: 240px;
-$toc-margin-left: 195px;
+$article-width: 70%;
+$toc-width: 28%;
// TOC on article pages
.inline-toc {
@@ -34,11 +34,22 @@ $toc-margin-left: 195px;
.related {
display: block;
+ background-color: $gray-light;
+ box-shadow: $nav-box-shadow;
+ 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;
}
+ &:hover {
+ background-image: $nav-hover-gradient;
+ }
}
}
@@ -66,7 +77,7 @@ $toc-margin-left: 195px;
}
> a.hastoc {
&::after {
- content: '-';
+ content: ''; /* don't make it look like you can toggle it closed */
}
}
}
@@ -75,7 +86,7 @@ $toc-margin-left: 195px;
.toc {
margin: 0;
- padding: 0;
+ padding: 0.3em 0 0 0;
border-top: $default-border;
.toc {
@@ -111,39 +122,32 @@ $toc-margin-left: 195px;
.inline-toc {
position: absolute;
top: 0;
- left: 50%;
- margin-left: $toc-margin-left;
width: $toc-width;
+ right: 0;
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.
+ &.sticky {
+ top: 0;
+ position: fixed;
+ -webkit-transform: translateZ(0); /* repaint issue */
+ }
}
}
.article-content {
- width: 70%;
+ 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;
}
}
-// Tablet
-@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;
- }
-}
-
// Phone
@media only screen and (max-width: $break-small) {
.article-content [itemprop="articleBody"] {
« no previous file with comments | « chrome/common/extensions/docs/static/js/site.js ('k') | chrome/common/extensions/docs/static/sass/_html.scss » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698