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

Side by Side Diff: stylesheets/style.scss

Issue 85333007: Begin the redesign process, starting with the navbar. (Closed) Base URL: git@github.com:dart-lang/pub-dartlang.git@master
Patch Set: Created 7 years 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 unified diff | Download patch
OLDNEW
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
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
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
OLDNEW
« stylesheets/partials/_variables.scss ('K') | « stylesheets/partials/_variables.scss ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698