Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 @import url(https://fonts.googleapis.com/css?family=Open+Sans:400,700|Montserrat :400,700); | 1 @import url(https://fonts.googleapis.com/css?family=Roboto:300,400|Montserrat:40 0,700); |
| 2 | 2 |
| 3 @import "compass/reset"; | 3 @import "compass/reset"; |
| 4 @import "compass/typography/text/replacement"; | 4 @import "compass/typography/text/replacement"; |
| 5 | 5 |
| 6 @import "syntax"; | 6 @import "partials/variables"; |
| 7 @import "partials/syntax"; | |
| 7 | 8 |
| 8 $baseFontSize: 14px; | 9 @import "bootstrap/theme"; |
| 9 $baseLineHeight: 24px; | 10 @import "bootstrap"; |
| 10 | 11 |
| 11 $sansFontFamily: "Open Sans", "Helvetica Neue", Helvetica, Arial, sans-serif; | 12 //// Don't require nasty <table class="table"> that bootstrap expects. |
|
Bob Nystrom
2013/11/26 19:00:11
Why ////?
Andrei Mouravski
2013/11/29 21:21:21
:g/^/norm i//
| |
| 12 | |
| 13 $iconSpritePath: 'img/glyphicons-halflings.png'; | |
| 14 $iconWhiteSpritePath: 'img/glyphicons-halflings-white.png'; | |
| 15 | |
| 16 @import "compass_twitter_bootstrap"; | |
| 17 | |
| 18 // Don't require nasty <table class="table"> that bootstrap expects. | |
| 19 table { | 13 table { |
| 20 @extend .table; | 14 @extend .table; |
| 21 } | 15 } |
| 22 | 16 |
| 17 | |
| 18 // TODO(amouravski): chuck these when everything else has been updated. | |
| 19 //$sansFontFamily: "Open Sans", "Helvetica Neue", Helvetica, Arial, sans-serif; | |
| 20 // | |
| 21 //$iconWhiteSpritePath: 'img/glyphicons-halflings-white.png'; | |
|
Bob Nystrom
2013/11/26 19:00:11
If they're commented out, can you chuck them now?
Andrei Mouravski
2013/11/29 21:21:21
Not yet, since I want to make sure the new stuff w
Bob Nystrom
2013/12/02 22:46:55
If the new stuff doesn't work, then you shouldn't
| |
| 22 | |
| 23 // Navbar | |
| 24 // ------------------------- | |
| 25 | |
| 26 $navbar-font-family: "Montserrat", sans-serif; | |
| 27 | |
| 28 .navbar { | |
| 29 border: none; | |
| 30 $shadow: 0 2px 2px rgba(50, 50, 50, 0.15); | |
| 31 @include box-shadow($shadow); | |
| 32 } | |
| 33 | |
| 34 .navbar-brand { | |
| 35 font-family: $navbar-font-family; | |
| 36 } | |
| 37 | |
| 38 .navbar-nav > li > a { | |
| 39 padding-top: 25px; | |
| 40 font-family: $navbar-font-family; | |
| 41 font-size: $font-size-small; | |
| 42 text-transform: uppercase; | |
| 43 } | |
| 44 | |
| 45 // TODO(amouravski): Fix up brand to match dartlang.org. | |
| 46 // TODO(amouravski): Upload bigger image and retina-ize it. | |
| 47 // TODO(amouravski): Spruce up search box. | |
|
Bob Nystrom
2013/11/26 19:00:11
These are distinct enough that it probably makes m
Andrei Mouravski
2013/11/29 21:21:21
Done.
| |
| 48 | |
| 49 // Body | |
| 50 // ------------------------- | |
| 51 | |
| 23 body { | 52 body { |
| 24 // Account for the fixed navbar height. | 53 // Account for the fixed navbar height. |
| 25 padding-top: 60px; | 54 padding-top: 65px; |
| 55 | |
| 56 -webkit-font-smoothing: antialiased; | |
| 26 } | 57 } |
| 27 | |
| 28 // Don't word wrap inline code. | 58 // Don't word wrap inline code. |
| 29 code { | 59 code { |
| 30 white-space: nowrap; | 60 white-space: nowrap; |
| 31 } | 61 } |
| 32 | 62 |
| 33 pre code { | 63 pre code { |
| 34 white-space: inherit; | 64 white-space: inherit; |
| 35 } | 65 } |
| 36 | 66 |
| 37 pre b { | 67 pre b { |
| 38 font-weight: bold; | 68 font-weight: bold; |
| 39 } | 69 } |
| 40 | 70 |
| 41 a.permalink { | 71 a.permalink { |
| 42 margin-left: 0.5em; | 72 margin-left: 0.5em; |
| 43 display: none; | 73 display: none; |
| 44 } | 74 } |
| 45 | 75 |
| 46 article { | 76 article { |
| 47 @extend .span8; | 77 @extend .col-md-8; |
| 48 | 78 |
| 49 position: relative; | 79 position: relative; |
| 50 | 80 |
| 51 h2, h3, h4 { | 81 h2, h3, h4 { |
| 52 // Ensure that the headers don't get covered up by the navbar when the user | 82 // Ensure that the headers don't get covered up by the navbar when the user |
| 53 // clicks on the permalink. | 83 // clicks on the permalink. |
| 54 padding-top: 40px; | 84 padding-top: 40px; |
| 55 margin-top: -40px; | 85 margin-top: -40px; |
| 56 | 86 |
| 57 &:hover a.permalink { | 87 &:hover a.permalink { |
| 58 display: inline; | 88 display: inline; |
| 59 } | 89 } |
| 60 } | 90 } |
| 61 } | 91 } |
| 62 | 92 |
| 63 .navbar-search { | 93 //.navbar-form { |
| 64 margin-top: 4px; | 94 // .search-query { |
| 65 | 95 // @include border-radius(4px, 4px); |
| 66 .search-query { | 96 // } |
| 67 border-radius: 4px; | 97 //} |
|
Bob Nystrom
2013/11/26 19:00:11
Delete all this commented out stuff.
Andrei Mouravski
2013/11/29 21:21:21
Done.
| |
| 68 -moz-border-radius: 4px; | |
| 69 -webkit-border-radius: 4px; | |
| 70 height: 20px; | |
| 71 line-height: inherit; | |
| 72 } | |
| 73 } | |
| 74 | 98 |
| 75 .has-permalink:hover > a.permalink { | 99 .has-permalink:hover > a.permalink { |
| 76 display:inline; | 100 display:inline; |
| 77 } | 101 } |
| 78 | 102 |
| 79 // Don't make tab bars so huge. | 103 //// Don't make tab bars so huge. |
| 80 .nav-tabs > li > a { | 104 //.nav-tabs > li > a { |
| 81 padding-top: 4px; | 105 // padding-top: 4px; |
| 82 padding-bottom: 4px; | 106 // padding-bottom: 4px; |
| 83 } | 107 //} |
| 84 | 108 |
| 85 .nav { | 109 //.nav { |
| 86 margin-bottom: 8px; | 110 // margin-bottom: 8px; |
| 87 } | 111 //} |
| 88 | 112 |
| 89 .package-sidebar { | 113 .package-sidebar { |
| 90 padding-top: 50px; | 114 padding-top: 50px; |
| 91 } | 115 } |
| 92 | 116 |
| 93 tt { | 117 // TODO(amouravski): add this back in? |
| 94 font-family: $monoFontFamily; | 118 //tt { |
| 95 padding: 0 3px; | 119 // font-family: $monoFontFamily; |
| 96 } | 120 // padding: 0 3px; |
| 121 //} | |
| 97 | 122 |
| 98 footer { | 123 footer { |
| 99 @extend .well; | 124 @extend .well; |
| 100 @extend .well-small; | 125 @extend .well-sm; |
| 101 font-size: 13px; | 126 font-size: 13px; |
| 102 text-align: center; | 127 text-align: center; |
| 103 | 128 |
| 104 .login, .logout { | 129 .login, .logout { |
| 105 display: inline-block; | 130 display: inline-block; |
| 106 float: right; | 131 float: right; |
| 107 // #f5f5f5 is the background-color of the footer, which isn't a variable | 132 // #f5f5f5 is the background-color of the footer, which isn't a variable |
| 108 // for some reason. | 133 // for some reason. |
| 109 color: scale-color(#f5f5f5, $lightness: -15%); | 134 color: scale-color(#f5f5f5, $lightness: -15%); |
| 110 } | 135 } |
| 111 } | 136 } |
| 112 | 137 |
| 113 form#private-key textarea { | 138 // TODO(amouravski): add this back in? |
| 114 width: 550px; | 139 //form#private-key textarea { |
| 115 height: 420px; | 140 // width: 550px; |
| 116 font-family: $monoFontFamily; | 141 // height: 420px; |
| 117 } | 142 // font-family: $monoFontFamily; |
| 118 | 143 //} |
| 119 // We have two brands: the dart logo and "pub.dartlang.org". This keeps them | |
| 120 // from being too spread out. | |
| 121 .brand + .brand { | |
| 122 padding-left: 10px; | |
| 123 } | |
| 124 | |
| 125 // Highlight the brand text on hover. This matches other navbar buttons and | |
| 126 // makes it clearer that the logo links to something different. | |
| 127 a.brand:hover { | |
| 128 color: white; | |
| 129 } | |
| 130 | 144 |
| 131 .dart-logo { | 145 .dart-logo { |
| 132 @include replace-text("/static/img/dart.png", 0, 0); | 146 @include replace-text-with-dimensions("dart.png", 0, 0); |
| 133 width: 21px; | |
| 134 } | 147 } |
| 135 | 148 |
| 136 .intro { | 149 .intro { |
| 137 @extend .well; | 150 @extend .well; |
| 138 } | 151 } |
| 139 | 152 |
| 140 .get-started { | 153 .get-started { |
| 141 @extend .btn; | 154 @extend .btn; |
| 142 @extend .btn-primary; | 155 @extend .btn-primary; |
| 143 } | 156 } |
| (...skipping 22 matching lines...) Expand all Loading... | |
| 166 margin: 3px 0 0 2px; | 179 margin: 3px 0 0 2px; |
| 167 } | 180 } |
| 168 | 181 |
| 169 .author { | 182 .author { |
| 170 white-space: nowrap; | 183 white-space: nowrap; |
| 171 } | 184 } |
| 172 | 185 |
| 173 .learn-more { | 186 .learn-more { |
| 174 a { | 187 a { |
| 175 @extend .btn; | 188 @extend .btn; |
| 176 @extend .btn-small; | 189 @extend .btn-sm; |
| 177 | 190 |
| 178 color: $linkColor; | 191 color: $link-color; |
| 179 | 192 |
| 180 float: right; | 193 float: right; |
| 181 position: absolute; | 194 position: absolute; |
| 182 width: 270px; | 195 width: 270px; |
| 183 right: -320px; | 196 right: -320px; |
| 184 | 197 |
| 185 display: inline-block; | 198 display: inline-block; |
| 186 } | 199 } |
| 187 } | 200 } |
| 188 | 201 |
| (...skipping 13 matching lines...) Expand all Loading... | |
| 202 font-size: 13px; | 215 font-size: 13px; |
| 203 line-height: 21px; | 216 line-height: 21px; |
| 204 } | 217 } |
| 205 } | 218 } |
| 206 | 219 |
| 207 .toc:before { | 220 .toc:before { |
| 208 @extend h3; | 221 @extend h3; |
| 209 content: "Contents"; | 222 content: "Contents"; |
| 210 } | 223 } |
| 211 | 224 |
| 212 .pagination { | |
| 213 @extend .pagination-centered; | |
| 214 } | |
| 215 | |
| 216 .progress { | 225 .progress { |
| 217 @extend .progress-striped; | 226 @extend .progress-striped; |
| 218 } | 227 } |
| 219 | 228 |
| 220 [class^="icon-"], [class*=" icon-"] { | 229 [class^="icon-"], [class*=" icon-"] { |
| 221 @include hide-text; | 230 @include hide-text; |
| 222 } | 231 } |
| 223 | 232 |
| 224 ul ul { | 233 ul ul { |
| 225 list-style-type: circle; | 234 list-style-type: circle; |
| 226 } | 235 } |
| 227 | 236 // TODO(amouravski): Fix admin page. |
| 228 .admin { | 237 //.admin { |
| 229 @extend .tabbable, .tabs-left; | 238 // @extend .tabbable, .tabs-left; |
| 230 | 239 // |
| 231 > ul { | 240 // > ul { |
| 232 @extend .nav, .nav-tabs; | 241 // @extend .nav, .nav-tabs; |
| 233 } | 242 // } |
| 234 | 243 // |
| 235 > div { | 244 // > div { |
| 236 @extend .tab-content; | 245 // @extend .tab-content; |
| 237 | 246 // |
| 238 > div { | 247 // > div { |
| 239 @extend .tab-pane, .container-fluid; | 248 // @extend .tab-pane, .container; |
| 240 } | 249 // } |
| 241 } | 250 // } |
| 242 | 251 // |
| 243 form button { | 252 // form button { |
| 244 @extend .btn; | 253 // @extend .btn; |
| 245 } | 254 // } |
| 246 } | 255 //} |
|
Bob Nystrom
2013/11/26 19:00:11
You should fix this before the patch goes in. I kn
Andrei Mouravski
2013/11/29 21:21:21
Well, this is on the dev branch. I just want to ma
Bob Nystrom
2013/12/02 22:46:55
Ah, OK. Are you planning to work in a branch for a
| |
| OLD | NEW |