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: Rebase to changeset #83 Created May 10, 2016, 2: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
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;
}
@@ -413,20 +411,17 @@
.table button.delete
{
background-color: transparent;
- background-position: -10px -33px;
+ background-position: -50px -52px;
border: 0px;
- margin-top: 5px;
- -moz-margin-end: 20px;
- -webkit-margin-end: 20px;
+ height: 10px;
Thomas Greiner 2016/05/23 12:43:26 These dimensions are being overridden by a rule be
saroyanm 2016/06/08 15:21:07 Done.
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
@@ -702,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
« new-options.html ('K') | « new-options.js ('k') | skin/options-sprite.png » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld