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

Unified Diff: skin/options.css

Issue 29338983: issue 3741 - Add "remove" option to list items in new options page (Closed)
Patch Set: Created March 23, 2016, 4:17 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
« options.js ('K') | « 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/options.css
===================================================================
--- a/skin/options.css
+++ b/skin/options.css
@@ -100,7 +100,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;
}
@@ -401,23 +399,17 @@
.table button.delete
{
background-color: transparent;
- background-position: -9px -32px;
+ background-position: -50px -52px;
border: 0px;
height: 10px;
- margin-top: 5px;
- -moz-margin-end: 20px;
- -webkit-margin-end: 20px;
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: -59px -52px;
}
.tabs.horizontal
@@ -642,12 +634,6 @@
display: inline-block;
}
-#all-filter-lists .table li
-{
- padding-left: 16px;
- padding-right: 16px;
-}
-
#all-filter-lists .table li > div
{
display: flex;
« options.js ('K') | « options.js ('k') | skin/options-sprite.png » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld