Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: static/scss/_utilities.scss

Issue 29612659: Issue 6061 - website-defaults unstyled ul doesn't unstyle li (Closed) Base URL: https://hg.adblockplus.org/website-defaults
Patch Set: Removed includes for the sake of consistency (will separate separately) Created Nov. 20, 2017, 3:27 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« pages/utilities.html ('K') | « pages/utilities.html ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // This file is part of website-defaults 1 // This file is part of website-defaults
2 // Copyright (C) 2016-present eyeo GmbH 2 // Copyright (C) 2016-present eyeo GmbH
3 // 3 //
4 // website-defaults is free software: you can redistribute it and/or 4 // website-defaults is free software: you can redistribute it and/or
5 // modify it under the terms of the GNU General Public License as published by 5 // modify it under the terms of the GNU General Public License as published by
6 // the Free Software Foundation, either version 3 of the License, or 6 // the Free Software Foundation, either version 3 of the License, or
7 // (at your option) any later version. 7 // (at your option) any later version.
8 // 8 //
9 // website-defaults is distributed in the hope that it will be useful, 9 // website-defaults is distributed in the hope that it will be useful,
10 // but WITHOUT ANY WARRANTY; without even the implied warranty of 10 // but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 88 matching lines...) Expand 10 before | Expand all | Expand 10 after
99 margin: -1px; 99 margin: -1px;
100 padding: 0; 100 padding: 0;
101 border: 0; 101 border: 0;
102 } 102 }
103 103
104 /* Unstyled elements 104 /* Unstyled elements
105 ******************************************************************************/ 105 ******************************************************************************/
106 106
107 .unstyled, 107 .unstyled,
108 .unstyled *, 108 .unstyled *,
109 .content .unstyled 109 .content .unstyled,
110 .content .unstyled *
110 { 111 {
111 margin: 0; 112 margin: 0;
112 padding: 0; 113 padding: 0;
113 border: 0; 114 border: 0;
114 background: none; 115 background: none;
115 } 116 }
116 117
117 ul.unstyled 118 .unstyled ul,
119 ul.unstyled,
120 .unstyled li,
ire 2017/11/27 08:40:02 I don't think this selector is needed because if .
juliandoucette 2017/11/27 13:32:10 Good catch.
121 li.unstyled
ire 2017/11/27 08:40:02 NIT: I can't think of a case where only one list i
juliandoucette 2017/11/27 13:32:09 Acknowledged.
118 { 122 {
119 list-style: none; 123 list-style: none;
120 } 124 }
121 125
122 /* Backgrounds 126 /* Backgrounds
123 ******************************************************************************/ 127 ******************************************************************************/
124 128
125 .bg-primary 129 .bg-primary
126 { 130 {
127 color: $primary-light; 131 color: $primary-light;
(...skipping 10 matching lines...) Expand all
138 { 142 {
139 color: $accent-light; 143 color: $accent-light;
140 background-color: $accent-dark; 144 background-color: $accent-dark;
141 } 145 }
142 146
143 .bg-error 147 .bg-error
144 { 148 {
145 color: $error-light; 149 color: $error-light;
146 background-color: $error-dark; 150 background-color: $error-dark;
147 } 151 }
OLDNEW
« pages/utilities.html ('K') | « pages/utilities.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld