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

Unified Diff: skin/new-options.css

Issue 29338983: issue 3741 - Add "remove" option to list items in new options page (Closed)
Patch Set: Removed label's "for" attribute, added "aria-lable" to addItem Created June 24, 2016, 4:35 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « new-options.js ('k') | skin/options-sprite.png » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: skin/new-options.css
===================================================================
--- a/skin/new-options.css
+++ b/skin/new-options.css
@@ -102,7 +102,6 @@
button[role="checkbox"]
{
- vertical-align: top;
width: 18px;
height: 18px;
margin-top: 0px;
@@ -359,9 +358,8 @@
.table li
{
display: flex;
- padding: 12px 0px;
- -webkit-padding-start: 16px;
- -moz-padding-start: 16px;
+ align-items: center;
+ padding: 12px 16px;
border-radius: 3px;
}
@@ -414,20 +412,17 @@
.table button.delete
{
background-color: transparent;
- background-position: -10px -33px;
+ background-position: -50px -51px;
border: 0px;
- margin-top: 5px;
- -moz-margin-end: 20px;
- -webkit-margin-end: 20px;
+ height: 10px;
padding: 0px;
+ cursor: pointer;
+ width: 10px;
}
-.table .popular
+.table button.delete:hover
{
- color: #1E8728;
- font-size: 12px;
- -moz-padding-end: 12px;
- -webkit-padding-end: 12px;
+ background-position: -61px -51px;
}
.tabs.horizontal
@@ -506,8 +501,7 @@
}
.icon-enter,
-#custom-filters-add button::after,
-.table button.delete
+#custom-filters-add button::after
{
height: 8px;
width: 8px;
@@ -703,10 +697,9 @@
font-weight: 600;
}
-#all-filter-lists .table li
+#all-filter-lists .table
{
- padding-left: 16px;
- padding-right: 16px;
+ display: inline-block;
}
#all-filter-lists .table li > div
« no previous file with comments | « new-options.js ('k') | skin/options-sprite.png » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld