issue 2377 - Finish design of Advanced tab of new options page

skin/options.css

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2016 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 37 matching lines @@
 body
 {
   background-color: #F5F5F5;
   display: flex;
   margin: 20px 10px;
   font-family: "Source Sans Pro", sans-serif;
   font-size: 14px;
   color: #494949;
 }
 
-h1 
+h1
 {
   font-size: 24px;
   line-height: 1em;
   font-weight: 300;
 }
 
 h2
 {
   font-size: 16px;
   font-weight: 600;
@@ skipping 15 matching lines @@
 [aria-hidden="true"]
 {
   display: none !important;
 }
 
 input[type="search"]::-webkit-search-cancel-button
 {
   display: none;
 }
 
-input[type="text"], input[type="search"]
+input[type="text"],
+input[type="search"],
+textarea
 {
   -webkit-box-sizing: border-box;
   -moz-box-sizing: border-box;
   box-sizing: border-box;
 }
 
 button[role="checkbox"]
 {
   vertical-align: top;
   width: 18px;
   height: 18px;
   margin-top: 0px;
   -moz-margin-end: 20px;
   -webkit-margin-end: 20px;
   padding: 0px;
   border: none;
   background-color: transparent;
   background-position: -51px 0px;
 }
 
 button[role="checkbox"][aria-checked="true"]
 {
   background-position: -68px 0px;
 }
 
 .option-name
 {
   display: flex;
   margin-bottom: 16px;
-  margin-top: 24px;
 }
 
 .option-name > :first-child
 {
   flex: 1;
   overflow: hidden;
   text-overflow: ellipsis;
   white-space: nowrap;
 }
 
@@ skipping 165 matching lines @@
   border: 1px solid #CDCDCD;
   border-radius: 8px;
   box-sizing: border-box;
   min-width: 960px;
   padding: 0px 60px 40px 60px;
 }
 
 #content h1
 {
   border-bottom: 1px solid #CDCDCD;
-  margin: 0px;
+  margin: 0px 0px 24px 0px;
   padding: 40px 0px 16px 0px;
 }
 
 #link-version
 {
   display: flex;
   margin: 12px 20px;
   color: #3A7BA6;
   text-decoration: none;
 }
@@ skipping 37 matching lines @@
 
 .table li
 {
   display: flex;
   padding: 12px 0px;
   -webkit-padding-start: 16px;
   -moz-padding-start: 16px;
   border-radius: 3px;
 }
 
+.table.list,
+#custom-wrapper
+{
+  border-bottom: 1px solid #CDCDCD;
+}
+
 .table.list li .display
 {
   flex: 1;
   line-height: 16px;
   overflow: hidden;
   text-overflow: ellipsis;
   white-space: nowrap;
 }
 
+.table:not(.list):not(.cols) li
+{
+  padding-top: 0px;
+  padding-bottom: 6px;
+}
+
 .table.list li:nth-of-type(odd),
 .table.cols li:nth-of-type(odd),
 .table li.empty-placeholder
 {
   background-color: #F5F5F5;
 }
 
 .table label
 {
   vertical-align: top;
 }
 
 .table.cols
 {
   border-bottom: 1px solid #CDCDCD;
   border-top: 1px solid #CDCDCD;
 }
 
 button[disabled="true"]
 {
   border-radius: 2px;
   background-color: #ccc;
 }
 
 .table button.delete
 {
   background-color: transparent;
-  background-position: -9px -32px;
+  background-position: -10px -33px;
   border: 0px;
-  height: 10px;
   margin-top: 5px;
   -moz-margin-end: 20px;
   -webkit-margin-end: 20px;
   padding: 0px;
-  cursor: pointer;
-  width: 10px;
 }
 
 .table .popular
 {
   color: #1E8728;
   font-size: 12px;
   -moz-padding-end: 12px;
   -webkit-padding-end: 12px;
 }
 
 .tabs.horizontal
 {
   display: flex;
+  margin-top: 0px;
   margin-bottom: 0px;
   padding: 0px;
+  border-bottom: 1px solid #CDCDCD;
 }
 
 .tabs.horizontal li
 {
   display: inline-block;
-  border-bottom: 1px solid #A1A1A1;
-  padding: 10px 46px;
+  padding: 0px 46px 10px 46px;
   color: #3A7BA6;
   text-align: center;
 }
 
 .icon,
 button[role="checkbox"],
 .table button.delete,
 #content-help a::before,
 #dialog-close::before,
 #custom-filters-add button::after,
 #dialog-body button::before,
 .date::before,
 .time::before,
 #all-filter-lists .arrow,
 .context-menu .content a::before
 {
   background-image: url(options-sprite.png);
   display: inline-block;
 }
 
+.icon-add,
+.icon-update,
+.icon-edit
+{
+  height: 16px;
+  width: 16px;
+  cursor: pointer;
+}
+
 .icon-add
 {
-  background-position: -0px -0px;
-  cursor: pointer;
-  height: 18px;
-  min-width: 18px;
+  background-position: -1px -1px;
 }
 
 .icon-update
 {
-  background-position: -34px -0px;
-  cursor: pointer;
-  height: 18px;
-  width: 18px;
+  background-position: -35px -1px;
 }
 
 .icon-edit
 {
-  background-position: -17px -0px;
+  background-position: -18px -1px;
+}
+
+.icon-save,
+.icon-cancel
+{
+  height: 10px;
+  width: 10px;
   cursor: pointer;
-  height: 18px;
-  width: 18px;
+}
+
+.icon-save
+{
+  background-position: -1px -62px;
+}
+
+.icon-cancel
+{
+  background-position: -12px -62px;
+}
+
+.icon-enter,
+#custom-filters-add button::after,
+.table button.delete
+{
+  height: 8px;
+  width: 8px;
+  cursor: pointer;
+}
+
+#dialog-close::before,
+.date::before,
+.time::before
+{
+  content: "";
+  height: 12px;
+  width: 12px;
 }
 
 #content-help a::before,
 #dialog-body button::before
 {
   background-position: 0px -42px;
   content: "";
   cursor: pointer;
   height: 11px;
   vertical-align: middle;
   width: 7px;
   -moz-margin-end: 12px;
   -webkit-margin-end: 12px;
 }
 
 .controls
 {
-  border-top: 1px solid #CDCDCD;
-  padding-top: 8px;
-  -moz-padding-start: 16px;
-  -webkit-padding-start: 16px;
-  position: relative;
+  padding-top: 14px;
+  margin-left: 16px;
+  margin-right: 16px;
+}
+
+.controls.mode-edit
+{
+  -moz-margin-end: 0px;
+  -webkit-margin-end: 0px;
 }
 
 .controls > div
 {
   display: flex;
+  position: relative;
 }
 
 .controls button,
 #dialog-close
 {
+  display: flex;
+  border: none;
+  padding: 0px;
+  align-items: center;
   background: none;
-  border: none;
   cursor: pointer;
-  display: block;
-  padding: 0;
 }
 
 .controls button span:not(.icon)
 {
+  -moz-margin-start: 16px;
+  -webkit-margin-start: 16px;
   color: #3A7BA6;
-  display: inline-block;
-  -moz-margin-start: 15px;
-  -webkit-margin-start: 15px;
-  padding-top: 1px;
   vertical-align: top;
 }
 
 /*
   General tab content
 */
 
+#acceptableads-table,
+#recommend-list-table,
+#blocking-languages-dialog-table,
+#custom-wrapper .table,
+#all-lang-table
+{
+  border-bottom: none;
+}
+
 #blocking-languages,
 #acceptable-ads
 {
   -moz-margin-end: 40px;
   -webkit-margin-end: 40px;
 }
 
 #custom-wrapper
 {
   height: 290px;
   overflow: auto;
 }
 
 #custom-wrapper .table
 {
   width: 100%;
 }
 
-#whitelisting .controls
-{
-  -moz-padding-start: 12px;
-  -webkit-padding-start: 12px;
-}
-
 #whitelisting .controls.mode-edit > button,
 #whitelisting .controls:not(.mode-edit) > div
 {
   display: none;
 }
 
 #whitelisting .controls input[type="text"]
 {
   border: 0px;
   border-bottom: 1px solid #A1A1A1;
@@ skipping 19 matching lines @@
   -webkit-margin-start: 32px;
 }
 
 #whitelisting-add-button + span
 {
   flex: 1;
 }
 
 .icon-enter
 {
-  background-position: -18px -32px;
-  cursor: pointer;
-  height: 10px;
+  background-position: -19px -33px;
   position: absolute;
-  top: 10px;
+  bottom: 9px;
   -moz-margin-start: -20px;
   -webkit-margin-start: -20px;
-  width: 10px;
 }
 
 .button-add, .cancel-button
 {
   background-color: transparent;
   border: 0px;
   color: #3A7BA6;
   cursor: pointer;
 }
 
 /*
   Advanced tab content
 */
 
-#tweaks.table li
-{
-  padding-bottom: 0px;
-}
-
 #restart-safari
 {
   -moz-margin-start: 20px;
   -webkit-margin-start: 20px;
   color: red;
   font-weight: 600;
 }
 
-#filter-lists > div
+#filter-lists > div,
+#all-filter-lists li.show-message .date,
+#all-filter-lists li.show-message .time,
+#all-filter-lists li:not(.show-message) .message,
+#custom-filters:not(.mode-edit) #custom-filters-raw,
+#custom-filters:not(.mode-edit) #custom-filters-raw-controls,
+#custom-filters.mode-edit #custom-filters-show-edit,
+#custom-filters.mode-edit #custom-filters-list-wrapper
 {
   display: none;
 }
 
 body[data-tab="advanced-allFilterLists"] #all-filter-lists,
 body[data-tab="advanced-customFilters"] #custom-filters
 {
   display: block;
 }
 
+#all-filter-lists .table
+{
+  display: inline-block;
+}
+
 body[data-tab="advanced-allFilterLists"] #content-advanced [data-tab="advanced-allFilterLists"],
 body[data-tab="advanced-customFilters"] #content-advanced [data-tab="advanced-customFilters"]
 {
   border-bottom: 2px solid #1E8728;
   padding-bottom: 10px;
   color: black;
   font-weight: 600;
 }
 
-#all-filter-lists .table
-{
-  display: inline-block;
-}
-
 #all-filter-lists .table li
 {
   padding-left: 16px;
   padding-right: 16px;
 }
 
 #all-filter-lists .table li > div
 {
   display: flex;
   width: 330px;
 }
 
 .table-header
 {
   display: flex;
 }
     
-#filter-lists h3
+#filter-lists h2
 {
-  display: inline-block;
   margin-bottom: 7px;
   font-size: 14px;
 }
 
-.table-header h3:first-child
+.table-header h2:first-child
 {
   width: 330px;
   -webkit-padding-start: 54px;
   -moz-padding-start: 54px;
 }
 
 #all-filter-lists .table li span.display
 {
   cursor: pointer;
 }
@@ skipping 33 matching lines @@
   text-decoration: none;
   cursor: pointer;
 }
 
 #all-filter-lists .table li:last-of-type > span:last-child
 {
   -webkit-margin-start: auto;
   -moz-margin-start: auto;
 }
 
-#all-filter-lists li.show-message .date,
-#all-filter-lists li.show-message .time,
-#all-filter-lists li:not(.show-message) .message
-{
-  display: none;
-}
-
 .date::before
 {
-  content: "";
   -webkit-margin-end: 6px;
   -moz-margin-end: 6px;
-  height: 12px;
-  width: 12px;
   background-position: -7px -49px;
 }
 
 .time::before
 {
-  content: "";
   -webkit-margin-end: 6px;
   -moz-margin-end: 6px;
   -webkit-margin-start: 12px;
   -moz-margin-start: 12px;
-  height: 12px;
-  width: 12px;
   background-position: -20px -49px;
 }
 
+#all-filter-lists-table .static label
+{
+  font-weight: 600;
+}
+
+#custom-filters
+{
+  width: 838px;

[Thomas Greiner, 2016/04/28 10:48:44]

This is a workaround.

1. According to
https://issues.adblockplus.org/attachment/ticket/1526/options_page_styleguide...
the content's inner width should be `840px` (i.e. `900-2*60`). That is excluding
borders.

2. The width should be defined by the content element. So I looked into possible
solutions and found this to work best:

#content
{
-  min-width: 960px;
+  width: 960px;
}

It's still debateable whether we should use `960px` or `962px` due to the border
width - unlike in the style guide - being included in the width calculation of
the box model.

Note that I'm not sure why we've been using "min-width" in the first place.

[saroyanm, 2016/04/28 13:16:23]

Yes, true, Done.
Done.
Yes you are right it's bit tricky with the design example, I kept it 962px for
now, otherwise we should change paddings, or content should be 838. 

+}
+
 #custom-filters-header
 {
-  padding: 0px 20px;
+  padding: 0px 16px;
   margin-bottom: 10px;
 }
 
 #custom-filters-raw-controls
 {
-  display: flex;
-}
-
-#custom-filters:not(.mode-edit) #custom-filters-raw,
-#custom-filters:not(.mode-edit) #custom-filters-raw-controls,
-#custom-filters.mode-edit #custom-filters-show-edit,
-#custom-filters.mode-edit #custom-filters-list-wrapper
-{
-  display: none;
-}
-
-#custom-filters-raw-save
-{
-  padding: 0px 16px;
-}
-
-#custom-filters .table
-{
-  height: 290px;
-  overflow: auto;
-  width: auto;
-}
-
-#custom-filters-add
-{
-  display: flex;
-  padding: 0px;
-  border: none;
-}
-
-#custom-filters .controls
-{
-  border-top: none;
+  justify-content: space-between;
 }
 
 #custom-filters-raw
 {
   width: 100%;
   height: 100%;
+  padding: 2px 16px; 
 }
 
 #custom-filters-wrapper
 {
-  height: 330px;
+  height: 280px;
 }
 
-#custom-filters input[type="text"]
+#custom-filters-add
 {
+  display: flex;
   border-width: 1px 0px 1px 0px;
   border-bottom-style: solid;
   border-top-style: solid;
   border-color: #1E8728;
-  height: 25px;
-  -moz-padding-start: 10px;
-  -webkit-padding-start: 10px;
-  -moz-padding-end: 60px;
-  -webkit-padding-end: 60px;
-  padding-top: 18px;
-  padding-bottom: 18px;
-  width: 100%;
+  padding: 0px;
+  margin: 0px;
+  background-color: #F5F5F5;
 }
 
-#custom-filters input[type="text"]:focus
+#custom-filters-add input[type="text"]
+{
+  -moz-padding-start: 16px;
+  -webkit-padding-start: 16px;
+  padding-top: 10px;
+  padding-bottom: 10px;
+  border: none;
+  width: 100%;
+  background-color: transparent;
+}
+
+#custom-filters-add input[type="text"]:focus
 {
   outline: 0px;
 }
 
-#custom-filters-add input
-{
-  font-size: 13px;
-  background-color: #F5F5F5;
-}
-
-#custom-filters-add input::-webkit-input-placeholder
-{
-  font-weight: 600;
-}
-#custom-filters-add input::-moz-placeholder
-{
-  font-weight: 600;
-}
-
 #custom-filters-table
 {
+  height: 240px;
+  width: 100%;
   border-top: 1px solid #CDCDCD;
+  overflow: auto;
 }
 
 #custom-filters-add button
 {
   font-size: 14px;
-  -webkit-margin-start: -60px;
-  -moz-margin-start: -60px;
   -webkit-padding-end: 6px;
   -moz-padding-end: 6px;
   cursor: pointer;
   color: #3A7BA6;
 }
 
 #custom-filters-add button::before
 {
   content: "";
   display: inline-block;
   -moz-border-end: 1px solid #CDCDCD;
   -webkit-border-end: 1px solid #CDCDCD;
   height: 15px;
   -webkit-margin-end: 10px;
   -moz-margin-end: 10px;
   margin-bottom: -2px;
   width: 1px;
 }
 
 #custom-filters-add button::after
 {
   content: "";
-  background-position: -28px -32px;
-  cursor: pointer;
-  height: 10px;
+  background-position: -28px -33px;
   -webkit-margin-start: 6px;
   -moz-margin-start: 6px;
-  width: 10px;
 }
 
 /*
   Tooltips
 */
 
 .tooltip
 {
+  display: inline-block;
+  position: relative;
+  margin: 0px 4px;
   border-bottom: dashed 1px;
+  height: 15px;
   color: #3A7BA6;
+  font-size: 12px;
+  line-height: 19px;
+  text-decoration: none;
+  font-weight: 400;
   cursor: default;
-  font-size: 12px;
-  height: 15px;
-  line-height: 19px;
-  margin: 0px 4px;
-  position: relative;
-  text-decoration: none;
 }
 
 .context-menu
 {
   display: inline-block;
   position: relative;
   border-bottom: none;
 }
 
 .context-menu a
@@ skipping 279 matching lines @@
   display: inline-block;
   height: 20px;
   font-size: 15px;
   -moz-padding-start: 10px;
   -webkit-padding-start: 10px;
 }
 
 #dialog-close::before
 {
   background-position: -10px -33px;
-  content: "";
   cursor: pointer;
   height: 8px;
   width: 8px;
   vertical-align: middle;
   -moz-margin-end: 6px;
   -webkit-margin-end: 6px;
 }
 
 #dialog #dialog-body
 {
@@ skipping 107 matching lines @@
 #other-language .button-add::before
 {
   display: none;
 }
 
 #other-language .display
 {
   -webkit-margin-start: 10px;
   -moz-margin-start: 10px;
 }