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

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

Issue 29338983: issue 3741 - Add "remove" option to list items in new options page (Closed)
Left Patch Set: Impoved accessibility, removed onclick assignments from the collections Created June 8, 2016, 3:02 p.m.
Right Patch Set: Removed label's "for" attribute, added "aria-lable" to addItem Created June 24, 2016, 4:35 p.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 | « new-options.js ('k') | skin/options-sprite.png » ('j') | 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-2016 Eyeo GmbH 3 * Copyright (C) 2006-2016 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 351 matching lines...) Expand 10 before | Expand all | Expand 10 after
362 padding: 12px 16px; 362 padding: 12px 16px;
363 border-radius: 3px; 363 border-radius: 3px;
364 } 364 }
365 365
366 .table.list, 366 .table.list,
367 #custom-wrapper 367 #custom-wrapper
368 { 368 {
369 border-bottom: 1px solid #CDCDCD; 369 border-bottom: 1px solid #CDCDCD;
370 } 370 }
371 371
372 .table.list li .display 372 .table.list li .display,
373 .fill
373 { 374 {
374 flex: 1; 375 flex: 1;
375 line-height: 16px; 376 line-height: 16px;
376 overflow: hidden; 377 overflow: hidden;
377 text-overflow: ellipsis; 378 text-overflow: ellipsis;
378 white-space: nowrap; 379 white-space: nowrap;
379 } 380 }
380 381
381 .table:not(.list):not(.cols) li 382 .table:not(.list):not(.cols) li
382 { 383 {
(...skipping 553 matching lines...) Expand 10 before | Expand all | Expand 10 after
936 position: absolute; 937 position: absolute;
937 -webkit-transition: opacity 200ms ease-in-out 400ms, 938 -webkit-transition: opacity 200ms ease-in-out 400ms,
938 visibility 0ms linear 400ms; 939 visibility 0ms linear 400ms;
939 transition: opacity 200ms ease-in-out 400ms, 940 transition: opacity 200ms ease-in-out 400ms,
940 visibility 0ms linear 400ms; 941 visibility 0ms linear 400ms;
941 visibility: visible; 942 visibility: visible;
942 width: 400px; 943 width: 400px;
943 z-index: 1; 944 z-index: 1;
944 } 945 }
945 946
946 html[dir="ltr"] div[role="tooltip"].flip-vertical, 947 html[dir="ltr"] div[role="tooltip"].flip-horizontal,
947 html[dir="rtl"] div[role="tooltip"]:not(.flip-vertical) 948 html[dir="rtl"] div[role="tooltip"]:not(.flip-horizontal)
948 { 949 {
949 left: inherit; 950 left: inherit;
950 right: -20px; 951 right: -20px;
951 } 952 }
952 953
953 .tooltip:not(:hover) > div[role="tooltip"], 954 .tooltip:not(:hover) > div[role="tooltip"],
954 div[role="tooltip"]:hover 955 div[role="tooltip"]:hover
955 { 956 {
956 visibility: hidden; 957 visibility: hidden;
957 opacity: 0; 958 opacity: 0;
(...skipping 96 matching lines...) Expand 10 before | Expand all | Expand 10 after
1054 .context-menu .source::before 1055 .context-menu .source::before
1055 { 1056 {
1056 background-position: -53px -34px; 1057 background-position: -53px -34px;
1057 } 1058 }
1058 1059
1059 .context-menu .delete::before 1060 .context-menu .delete::before
1060 { 1061 {
1061 background-position: -71px -34px; 1062 background-position: -71px -34px;
1062 } 1063 }
1063 1064
1064 html[dir="ltr"] div[role="tooltip"].flip-vertical::before, 1065 html[dir="ltr"] div[role="tooltip"].flip-horizontal::before,
1065 html[dir="rtl"] div[role="tooltip"]:not(.flip-vertical)::before 1066 html[dir="rtl"] div[role="tooltip"]:not(.flip-horizontal)::before
1066 { 1067 {
1067 left: inherit; 1068 left: inherit;
1068 right: 30px; 1069 right: 30px;
1069 } 1070 }
1070 1071
1071 div[role="tooltip"] img 1072 div[role="tooltip"] img
1072 { 1073 {
1073 float: left; 1074 float: left;
1074 height: 64px; 1075 height: 64px;
1075 margin-top: -2px; 1076 margin-top: -2px;
(...skipping 242 matching lines...) Expand 10 before | Expand all | Expand 10 after
1318 #other-language .button-add::before 1319 #other-language .button-add::before
1319 { 1320 {
1320 display: none; 1321 display: none;
1321 } 1322 }
1322 1323
1323 #other-language .display 1324 #other-language .display
1324 { 1325 {
1325 -webkit-margin-start: 10px; 1326 -webkit-margin-start: 10px;
1326 -moz-margin-start: 10px; 1327 -moz-margin-start: 10px;
1327 } 1328 }
LEFTRIGHT

Powered by Google App Engine
This is Rietveld