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

Delta Between Two Patch Sets: skin/desktop-options.css

Issue 29674584: Issue 5549 - Implement missing error handlings for custom filter lists (Closed)
Left Patch Set: Created Jan. 19, 2018, 11:46 a.m.
Right Patch Set: changed custom-filters-edit-area to custom-filters-control as suggested Created Feb. 5, 2018, 9:05 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « messageResponder.js ('k') | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 /* 1 /*
2 * This file is part of Adblock Plus <https://adblockplus.org/>, 2 * This file is part of Adblock Plus <https://adblockplus.org/>,
3 * Copyright (C) 2006-present eyeo GmbH 3 * Copyright (C) 2006-present eyeo GmbH
4 * 4 *
5 * Adblock Plus is free software: you can redistribute it and/or modify 5 * Adblock Plus is free software: you can redistribute it and/or modify
6 * it under the terms of the GNU General Public License version 3 as 6 * it under the terms of the GNU General Public License version 3 as
7 * published by the Free Software Foundation. 7 * published by the Free Software Foundation.
8 * 8 *
9 * Adblock Plus is distributed in the hope that it will be useful, 9 * Adblock Plus 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 527 matching lines...) Expand 10 before | Expand all | Expand 10 after
538 538
539 #sidebar, 539 #sidebar,
540 #sidebar .fixed, 540 #sidebar .fixed,
541 [role="tablist"] 541 [role="tablist"]
542 { 542 {
543 width: 14.3rem; 543 width: 14.3rem;
544 } 544 }
545 545
546 #sidebar 546 #sidebar
547 { 547 {
548 flex-shrink: 0; 548 flex-shrink: 0;
549 } 549 }
550 550
551 #sidebar .fixed 551 #sidebar .fixed
552 { 552 {
553 top: 1.2rem; 553 top: 1.2rem;
554 bottom: 0rem; 554 bottom: 0rem;
555 height: auto; 555 height: auto;
556 } 556 }
557 557
558 #sidebar header 558 #sidebar header
(...skipping 569 matching lines...) Expand 10 before | Expand all | Expand 10 after
1128 /* 1128 /*
1129 Advanced tab content 1129 Advanced tab content
1130 */ 1130 */
1131 1131
1132 #all-filter-lists-table li.show-message .last-update, 1132 #all-filter-lists-table li.show-message .last-update,
1133 #all-filter-lists-table li:not(.show-message) .message, 1133 #all-filter-lists-table li:not(.show-message) .message,
1134 #custom-filters:not([data-mode="empty"]) #empty-custom-filters, 1134 #custom-filters:not([data-mode="empty"]) #empty-custom-filters,
1135 #custom-filters[data-mode="empty"] #custom-filters-raw, 1135 #custom-filters[data-mode="empty"] #custom-filters-raw,
1136 #custom-filters:not([data-mode="write"]) #custom-filters-raw-controls, 1136 #custom-filters:not([data-mode="write"]) #custom-filters-raw-controls,
1137 #custom-filters:not([data-mode="read"]) #custom-filters-edit, 1137 #custom-filters:not([data-mode="read"]) #custom-filters-edit,
1138 #custom-filters-edit-filters, 1138 #custom-filters-error,
1139 #link-filters-on-edit-error,
1140 .state span, 1139 .state span,
1141 #acceptable-ads:not(.show-dnt-notification) #dnt 1140 #acceptable-ads:not(.show-dnt-notification) #dnt
1142 { 1141 {
1143 display: none; 1142 display: none;
1144 } 1143 }
1145 1144
1146 #all-filter-lists-table button[role="checkbox"][aria-checked="true"] + .state #s tate-active, 1145 #all-filter-lists-table button[role="checkbox"][aria-checked="true"] + .state #s tate-active,
1147 #all-filter-lists-table button[role="checkbox"][aria-checked="false"] + .state # state-disabled, 1146 #all-filter-lists-table button[role="checkbox"][aria-checked="false"] + .state # state-disabled
1148 #link-filters-on-edit-error.visible
1149 { 1147 {
1150 display: inline; 1148 display: inline;
1151 } 1149 }
1152 1150
1153 #all-filter-lists-table 1151 #all-filter-lists-table
1154 { 1152 {
1155 margin-bottom: 0.8rem; 1153 margin-bottom: 0.8rem;
1156 } 1154 }
1157 1155
1158 #custom-filters-edit { 1156 #custom-filters-edit
1157 {
1159 white-space: nowrap; 1158 white-space: nowrap;
1160 } 1159 }
1161 1160
1162 #custom-filters-edit-area { 1161 #custom-filters-control
1162 {
1163 display: flex; 1163 display: flex;
1164 width: 100%; 1164 width: 100%;
1165 } 1165 }
1166 1166
1167 /* 1167 /*
1168 avoid doubled margin glitchy behavior 1168 avoid doubled margin glitchy behavior
1169 as shown in visual specifications 1169 as shown in visual specifications
1170 */ 1170 */
1171 #custom-filters-edit-area .side-controls .side-controls { 1171 #custom-filters-control .side-controls .side-controls
1172 {
1172 margin: 0; 1173 margin: 0;
1173 } 1174 }
1174 1175
1175 /* 1176 /*
1176 visibility: hidden; 1177 visibility: hidden;
1177 preserves the flex-box meaning, 1178 preserves the flex-box meaning,
1178 keeping buttons on the right. 1179 keeping buttons on the right.
1179 */ 1180 */
1180 #custom-filters-edit-error { 1181 #custom-filters-edit-error
1182 {
1181 visibility: hidden; 1183 visibility: hidden;
1182 display: flex; 1184 display: flex;
1183 align-self: center; 1185 align-self: center;
1184 justify-content: flex-start; 1186 justify-content: flex-start;
1185 font-weight: bold; 1187 font-weight: bold;
1186 } 1188 }
1187 1189
1188 #custom-filters-edit-error.warning, 1190 #custom-filters.warning #custom-filters-edit-error,
1189 #custom-filters-edit-filters 1191 #custom-filters-error
1190 { 1192 {
1191 color: #C11D26; 1193 color: #C11D26;
1192 padding: 0.2rem 0.8rem; 1194 padding: 0.2rem 0.8rem;
1193 } 1195 }
1194 1196
1195 #custom-filters-edit-error.warning { 1197 #custom-filters.warning #custom-filters-edit-error
1198 {
1196 visibility: visible; 1199 visibility: visible;
1197 } 1200 }
1198 1201
1199 #custom-filters-edit-filters { 1202 #custom-filters-error
1203 {
1200 display: block; 1204 display: block;
1205 list-style: none;
1201 white-space: pre; 1206 white-space: pre;
1202 overflow: auto; 1207 overflow: auto;
1203 } 1208 }
1204 1209
1205 #custom-filters-edit-filters.many { 1210 #custom-filters-error.many
1211 {
1206 border: 1px solid silver; 1212 border: 1px solid silver;
1207 max-height: 6.2rem; 1213 max-height: 6.2rem;
1208 } 1214 }
1209 1215
1210 #custom-filters-raw 1216 #custom-filters-raw
1211 { 1217 {
1212 width: 100%; 1218 width: 100%;
1213 height: 23.6rem; 1219 height: 23.6rem;
1214 padding: 0.8rem; 1220 padding: 0.8rem;
1215 } 1221 }
1216 1222
1217 #custom-filters-raw:focus 1223 #custom-filters-raw:focus
1218 { 1224 {
1219 outline: none; 1225 outline: none;
1220 border: 2px solid #077CA6; 1226 border: 2px solid #077CA6;
1221 } 1227 }
1222 1228
1223 #custom-filters-raw.warning 1229 #custom-filters.warning #custom-filters-raw
1224 { 1230 {
1225 outline: none; 1231 outline: none;
1226 border: 2px solid #C11D26; 1232 border: 2px solid #C11D26;
1227 } 1233 }
1228 1234
1229 #empty-custom-filters 1235 #empty-custom-filters
1230 { 1236 {
1231 padding: 1.5rem; 1237 padding: 1.5rem;
1232 border: 1px solid #CDCDCD; 1238 border: 1px solid #CDCDCD;
1233 } 1239 }
(...skipping 335 matching lines...) Expand 10 before | Expand all | Expand 10 after
1569 font-size: 1rem; 1575 font-size: 1rem;
1570 color: #077CA6; 1576 color: #077CA6;
1571 background-color: #E1F2FA; 1577 background-color: #E1F2FA;
1572 } 1578 }
1573 1579
1574 #notification strong 1580 #notification strong
1575 { 1581 {
1576 flex: 1; 1582 flex: 1;
1577 text-align: center; 1583 text-align: center;
1578 } 1584 }
LEFTRIGHT

Powered by Google App Engine
This is Rietveld