Issue 5872 - apply style changes

skin/desktop-options.css

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-present 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 38 matching lines @@
   font-size: 16px;
 }
 
 body
 {
   background-color: #F3F3F3;
   display: flex;
   justify-content: center;
   /* We force vertical scrollbars to keep the content centered */
   overflow-y: scroll;
   margin: 1.2rem 0.3rem;

[ire, 2017/11/15 13:49:28]

The vertical margin should be 1rem

[saroyanm, 2017/11/15 14:51:23]

Done.

   font-family: "Source Sans Pro", sans-serif;
-  font-size: 1.25rem;
-  color: #494949;
+  font-size: 1rem;
+  line-height: 1.5rem;
+  color: #4A4A4A;
 }
 
 h1
 {
   font-size: 3rem;
   font-weight: 300;
+  line-height: 3rem;
 }
 
 h2
 {
-  font-size: 1.375rem;
+  font-size: 1.125rem;
   font-weight: 700;
 }
 
 a
 {
-  color: #099CD0;
-  text-decoration: none;
+  color: #077CA6;
 }
 
 a:hover
 {
   color: #5CBCE1;
 }
 
 ul
 {
   margin: 0rem;
 }
 
 [aria-hidden="true"]
 {
   display: none !important;
 }
 
 input[type="text"],
 input[type="url"],
 textarea,
 main
 {
   -webkit-box-sizing: border-box;
   -moz-box-sizing: border-box;
   box-sizing: border-box;
 }
 
 /*
+  Normalization
+ */
+
+input,
+button
+{
+  font-family: inherit;
+}
+
+button
+{
+  border-radius: 0rem;
+}
+
+/*
   Buttons and links
  */
 
 button,
 .button
 {
   display: block;
-  padding: 0.8rem 1.2rem;
+  padding: 0.6rem 0.8rem;
   background-color: transparent;
-  font-size: 1.125rem;
+  font-size: 1rem;
   font-weight: 700;
   text-decoration: none;
   text-transform: uppercase;
   cursor: pointer;
 }
 
 button.primary,
 .button.primary
 {
   border: 0px;
   color: #FFF;
-  background-color: #099CD0;
+  background-color: #077CA6;
 }
 
 button.primary:not([disabled]):hover,
 .button.primary:hover
 {
   box-shadow: inset 0 0 0 3px #005D80;
 }
 
 button.primary[disabled]
 {
   background-color: #5CBCE1;
 }
 
 button.secondary,
 .button.secondary
 {
-  border: 1px solid #099CD0;
-  color: #099CD0;
+  border: 2px solid #077CA6;
+  color: #077CA6;
 }
 
 button.secondary:hover,
 .button.secondary:hover
 {
-  box-shadow: inset 0 0 0 2px #099CD0;
+  box-shadow: inset 0 0 0 1px #077CA6;
 }
 
 button[role="checkbox"]
 {
-  width: 18px;
-  height: 18px;
+  width: 1.2rem;
+  height: 1.2rem;
   padding: 0px;
   border: 0px;
   background-color: transparent;
 }
 
 button[role="checkbox"]:not(.toggle)
 {
-  background-image: url(icons/checkbox.png);
+  background-image: url(icons/checkbox.svg#off);
   display: inline-block;
 }
 
 button[role="checkbox"][aria-checked="true"]:not(.toggle)
 {
-  background-position: 0px 18px;
+  background-image: url(icons/checkbox.svg#on);
 }
 
 button[role="checkbox"][disabled],
 button[role="checkbox"][aria-disabled="true"]
 {
   border-radius: 2px;
   background-color: #ccc;
 }
 
 button[role="checkbox"].toggle
 {
   background: url(icons/toggle.svg#on);
 }
 
 button[role="checkbox"][aria-checked="false"].toggle
 {
   background: url(icons/toggle.svg#off);
 }
 
 button[role="checkbox"][aria-checked="true"].toggle
 {
   background: url(icons/toggle.svg#on);
 }
 
 button[role="checkbox"].toggle
 {
   background-position: initial;
-  width: 2.25rem;
-  height: 1.3rem;
+  width: 1.9rem;
+  height: 1rem;
+  vertical-align: middle;
 }
 
 button[role="checkbox"][disabled].toggle
 {
   background-image: none;
 }
 
 button.delete::before
 {
   background-image: url(icons/trash.svg#default);
@@ skipping 20 matching lines @@
   border: 0px;
   padding: 0px;
   background-color: transparent;
 }
 
 button.gear::before,
 button.delete::before
 {
   content: "";
   display: block;
-  height: 1.9rem;
-  width: 1.9rem;
+  height: 1rem;
+  width: 1rem;
+  border: 0.2rem solid transparent;
+  background-repeat: no-repeat;
+  background-position: center;
 }
 
 button.link,
 button.list
 {
-  color: #099CD0;
+  color: #077CA6;
 }
 
 button.link
 {
   border: 0px;
   background-color: transparent;
-  font-weight: 300;
+  font-weight: 400;
   font-family: inherit;
   text-transform: none;
+  text-decoration: underline;
   padding: 0.2rem;
 }
 
 button.link:hover
 {
   color: #5CBCE1;
 }
 
 button.list
 {
   border-style: solid;;
   border-color: #CDCDCD;
-  border-width: 0px 1px 1px 1px;
+  border-width: 1px;
   width: 100%;
   background-color: #E1F2FA;
   text-align: initial;
 }
 
 button.list:hover
 {
-  box-shadow: inset 0 0 0 3px #099CD0;
-  border-color: #099CD0;
+  box-shadow: inset 0 0 0 3px #077CA6;
+  border-color: #077CA6;
 }
 
 .side-controls:not(.wrap)
 {
   margin: 0.8rem 0rem;
   display: flex;
   justify-content: flex-end;
 }
 
 .side-controls button
@@ skipping 19 matching lines @@
   position: relative;
   padding-top: 0.7rem;
   margin: 1.8rem 0rem 0.5rem 0rem;
 }
 
 .floating-input input
 {
   border-color: #CDCDCD;
   border-width: 0px 0px 1px 0px;
   outline: none;
-  font-size: 1.3rem;
+  font-size: 1rem;
   padding: 5px;
   width: 100%;
 }
 
 .floating-input input:placeholder-shown ~ label,
 .floating-input input + label,
 .floating-input input:focus + label
 {
   position: absolute;
   top: 0.9rem;
   left: 0.3rem;
-  font-size: 1.3rem;
+  font-size: 1rem;
 }
 
 .floating-input input + label,
 .floating-input input:focus + label
 {
   top: -0.5rem;
   font-size: 0.9rem;
 }
 
 html[dir="rtl"] .floating-input input:placeholder-shown ~ label,
 html[dir="rtl"] .floating-input input ~ label,
 html[dir="rtl"] .floating-input input:focus ~ label
 {
   right: 0.3rem;
   left: auto;
 }
 
 [data-validation] .floating-input input:focus:invalid
 {
   border-color: #C11D27;
 }
 
 [data-validation] .floating-input input:focus:invalid ~ .attention::before,
 [data-validation] .floating-input input:valid ~ .attention::before

[ire, 2017/11/15 13:49:28]

This element is too close to the input underline. There should be more space
between the two

[saroyanm, 2017/11/15 14:51:23]

Done.

 {
   content: "";
   position: absolute;
   display: block;
   margin: 0.5rem;
   height: 1.5rem;
   width: 1.5rem;
   border: 0rem;
   top: 0.5rem;
   right: 0rem;
@@ skipping 72 matching lines @@
 
 #sidebar header h1 strong
 {
   font-weight: 700;
 }
 
 #sidebar header p
 {
   margin: 0rem;
   font-size: 2.4rem;
+  line-height: 2.6rem;
 }
 
 html[dir="rtl"] #sidebar header
 {
   text-align: left;
 }
 
 #sidebar-logo
 {
   width: 3rem;
   margin-bottom: 0.7rem;
 }
 
 #sidebar nav,
 #sidebar footer
 {
   margin: 1.4rem 0rem;
 }
 
 [role="tablist"]
 {
   list-style: none;
   margin: 0rem;
   padding: 0rem;
   position: relative;
-  font-size: 1.25rem;
+  font-size: 1rem;
 }
 
 [role="tablist"] li a
 {
   display: flex;
   margin-top: -1px;
   padding: 1rem 0.8rem;
   -moz-margin-end: -1px;
   -webkit-margin-end: -1px;
   -moz-margin-start: -1px;
@@ skipping 22 matching lines @@
 
 #sidebar footer
 {
   width: 100%;
 }
 
 #sidebar footer p
 {
   display: flex;
   justify-content: center;
-  margin: 1.2rem 0rem;
+  margin: 1rem 0rem;
 }
 
 /* This is a stopgap solution of footer overlapping tabs on low resolutions */
 @media (min-height: 37rem)
 {
   #sidebar .fixed
   {
     position: fixed;
   }
 
@@ skipping 13 matching lines @@
 body[data-tab|="whitelist"] #content-whitelist,
 body[data-tab|="help"] #content-help
 {
   display: block;
 }
 
 main
 {
   background-color: #FFFFFF;
   border: 1px solid #CDCDCD;
-  max-width: 46.3rem;
+  width: 46.3rem;
   padding: 0px 0rem 1.4rem 0rem;
 }
 
 main > div
 {
   display: none;
 }
 
 main p
 {
   margin: 0.8rem 0rem;
 }
 
 /*
   Sections
  */
 
 [role="tabpanel"] > section,
 [role="tabpanel"] > .section
 {
-  padding: 2rem;
+  padding: 1.4rem 2rem;
   border-top: 1px solid #CDCDCD;
 }
 
 [role="tabpanel"] > header h1,
 [role="tabpanel"] > header p
 {
   padding: 0rem 2rem;
   margin: 1.4rem 0rem;
 }
 
@@ skipping 31 matching lines @@
   flex: 3;
 }
 
 /*
   Acceptable ads
  */
 
 #acceptable-ads ul
 {
   position: relative;
-  padding-left: 2rem;
+  padding-left: 2.2rem;
   list-style: none;
 }
 
 html[dir="rtl"] #acceptable-ads ul
 {
   padding-left: 0rem;
-  padding-right: 2rem;
+  padding-right: 2.2rem;
 }
 
 #acceptable-ads button
 {
   position: absolute;
   margin-top: 0.3rem;
   left: 0rem;
 }
 
 html[dir="rtl"] button
 {
   left: auto;
   right: 0rem;
 }
 
 #acceptable-ads label
 {
   font-weight: 700;
-  font-size: 1.375rem;
+  font-size: 1rem;
+  -moz-margin-end: 0.5rem;
+  -webkit-margin-end: 0.5rem;
 }
 
 #dnt
 {
   padding: 0.8rem;
-  border: 1px solid #099CD0;
+  border: 1px solid #077CA6;
 }
 
 .new
 {
   display: inline-block;
-  margin: 0rem 0.5rem;
-  padding: 0.3rem 0.6rem;
+  padding: 0.2rem 0.5rem;
   border-radius: 0.2rem;
-  background-color: #099CD0;
+  background-color: #077CA6;
   color: #FFF;
   line-height: 100%;
-  font-size: 1rem;
+  font-size: 0.8rem;
   text-transform: uppercase;
 }
 
 /*
   Tables
  */
 
 ul.table,
 ul.list
 {
@@ skipping 13 matching lines @@
 {
   margin: 0rem;
   border-style: solid;
   border-color: #CDCDCD;
   border-width: 0px 1px 1px 1px;
 }
 
 .list li
 {
   padding: 0rem;
-  margin: 0rem 0rem 1.3rem 0rem;
+  margin-bottom: 0.8rem;
 }
 
 .list li [role="checkbox"]
 {
   flex-shrink: 0;
 }
 
 .table li:first-of-type
 {
   border-top: 1px solid #CDCDCD;
 }
 
 .table.list li
 {
-  padding: 0.5rem 1.1rem;
+  padding: 0.5rem 1rem;
   margin: 0rem;
 }
 
+.table.list.bottom-control li:last-of-type
+{
+  border-bottom: 0px;
+}
+
 li .display
 {
   margin: 0rem 1rem;
 }
 
 .table.list li .display
 {
   flex: 1;
+  margin: 0rem;
 }
 
 .table.list li.empty-placeholder
 {
   padding: 1.3em 1.9em;

[ire, 2017/11/15 13:49:28]

This padding should be the same as `padding: 0.5rem 1rem;`

[saroyanm, 2017/11/15 14:51:23]

According to specs, it's 1rem 1.4rem, I've updated accordingly (
https://bitbucket.org/adblockplus/spec/src/b140349a623ca03a314b03009cecbff948...
)

 }
 
 .table.list li.empty-placeholder:not(:last-of-type)
 {
   border-bottom: 0px;
 }
 
-#blocking-languages-table .display
-{
-  font-weight: 700;
-}
-
 .table.list button.link
 {
   font-weight: 700;
   text-transform: uppercase;
+  text-decoration: none;
 }
 
 .table:not(.list):not(.cols) li
 {
   padding-top: 0px;
   padding-bottom: 6px;
 }
 
 .table li [data-single="visible"],
 .table li:first-of-type:last-of-type [data-single="hidden"]
 {
   display: none;
 }
 
 .table li:first-of-type:last-of-type [data-single="visible"]
 {
   display: block;
 }
 
 .th
 {
   display: flex;
 }
 
-.col4 > *
+.col5 > *
 {
   display: inline-block;
 }
 
-.col4
+.col5
 {
-  margin: 0rem 0.9rem;
+  margin: 0rem 1rem;
 }
 
-.th .col4:nth-of-type(1),
-.table .col4:nth-of-type(1)
+.th .col5:nth-of-type(1),
+.table .col5:nth-of-type(1)
 {
-  flex: 2;
+  flex: 4;
 }
 
-.th .col4:nth-of-type(2),
-.table .col4:nth-of-type(2)
+.th .col5:nth-of-type(2),
+.table .col5:nth-of-type(2)
 {
-  flex: 3;
+  flex: 8;
 }
 
-.th .col4:nth-of-type(3),
-.table .col4:nth-of-type(3)
+.th .col5:nth-of-type(3),
+.table .col5:nth-of-type(3)
 {
-  flex: 2;
+  flex: 4;
 }
 
-.th .col4:nth-of-type(4),
-.table .col4:nth-of-type(4)
+.th .col5:nth-of-type(4),
+.table .col5:nth-of-type(4)
 {
   flex: 1;
 }
 
-.table.cols
+.th .col5:nth-of-type(5),
+.table .col5:nth-of-type(5)
 {
-  font-size: 1.1rem;
+  flex: 1;
+  -moz-margin-start: 0;
+  -webkit-margin-start: 0;
+  -moz-margin-end: 1.8rem;
+  -webkit-margin-end: 1.8rem;
 }
 
 .table.cols .display
 {
   margin: 0rem;
 }
 
 .table.cols li
 {
   padding: 0.9rem 0rem;
@@ skipping 44 matching lines @@
 
 #dialog .table.list li button[aria-checked="true"]::before
 {
   content: "";
   width: 1.3rem;
   height: 1.3rem;
   background-image: url(icons/checkmark.svg#default);
   margin: 0rem;
 }
 
 #dialog .table.list li button .display

[ire, 2017/11/15 13:49:28]

This element is currently in bold weight since it's a button, but I think it
should be same as body font weight

[saroyanm, 2017/11/15 14:51:23]

Right, done.

 {
   flex: none;
   margin: 0rem 0.8rem;
   text-transform: none;
 }
 
 /*
   Tooltips
 */
 
 .tooltip
 {
   display: inline-block;
   position: relative;
   margin: 0rem;
   -moz-margin-end: 1rem;
   -webkit-margin-end: 1rem;
   line-height: 1.5rem;
   text-decoration: none;
   cursor: help;
 }
 
 .tooltip::before
 {
   content: "";
-  width: 1.3rem;
-  height: 1.3rem;
+  width: 1.1rem;
+  height: 1.1rem;
   display: block;
   background-image: url(icons/tooltip.svg);
 }
 
 /* 
   Used for translatable screen reader only conten.
   e.g.: Use instead of aria-label to avoid complex attribute value translation
 */
 .sr-only
 {
@@ skipping 29 matching lines @@
  */
 
 #content-whitelist form
 {
   display: flex;
 }
 
 #content-whitelist form input
 {
   flex: 1;
+  height: 100%;
   padding: 0.5rem 1rem;
-  font-size: 1.25rem;
-  border: 2px solid #099CD0;
+  font-size: 1rem;
+  border: 2px solid #077CA6;
 }
 
 #content-whitelist form button
 {
   -moz-margin-start: 0.7rem;
   -webkit-margin-start: 0.7rem;
 }
 
 #whitelisting-table li
 {
+  padding-left: 1.4rem;
+  padding-right: 1.4rem;
   border-left: 0rem;
   border-right: 0rem; 
 }
 
 /*
   Advanced tab content
 */
 
 #all-filter-lists-table li.show-message .last-update,
 #all-filter-lists-table li:not(.show-message) .message,
@@ skipping 30 matching lines @@
 #custom-filters-raw
 {
   width: 100%;
   height: 23.6rem;
   padding: 0.8rem 1.3rem; 
 }
 
 #custom-filters-raw:focus
 {
   outline: none;
-  border: 2px solid #099CD0;
+  border: 2px solid #077CA6;
 }
 
 #empty-custom-filters
 {
   padding: 1.5rem;
   border: 1px solid #CDCDCD;
 }
 
 /*
   Context menu
  */
 
 li [role="menubar"]
 {
   position: relative;
 }
 
 [role="tooltip"]
 {
   right: -1rem;
   margin-top: 1rem;
   opacity: 1;
-  padding: 1.3rem;
+  padding: 0.2rem 1rem;
   position: absolute;
   -webkit-transition: opacity 200ms ease-in-out 400ms,
     visibility 0ms linear 400ms;
   transition: opacity 200ms ease-in-out 400ms,
     visibility 0ms linear 400ms;
   visibility: visible;
   width: 15rem;
   z-index: 1;
 }
 
 html[dir="rtl"] [role="tooltip"]
 {
   right: auto;
   left: -1rem;
 }
 
 [role="tooltip"],
 .context-menu .content
 {
-  border: 1px solid #099CD0;
+  border: 1px solid #077CA6;
   border-radius: 3px;
   background-color: #FFFFFF;
 }
 
 
 .tooltip:not(:hover) > [role="tooltip"],
 [role="tooltip"]:hover
 {
   visibility: hidden;
   opacity: 0;
   -webkit-transition-delay: 0ms;
   transition-delay: 0ms;
 }
 
 .context-menu
 {
   position: absolute;
-  right: 3rem;
-  top: -3rem;
+  right: 2.5rem;
+  top: -2.7rem;
   z-index: 1;
   visibility: hidden;
 }
 
 html[dir="rtl"] .context-menu
 {
   right: auto;
-  left: 3rem;
+  left: 2.5rem;
 }
 
 .context-menu .content
 {
   padding: 0rem;
   position: relative;
   cursor: default;
 }
 
 li.show-context-menu .context-menu
 {
   visibility: visible;
 }
 
 [role="tooltip"]::before,
 .context-menu::before
 {
   content: "";
   width: 0rem;
   height: 0rem;
   position: absolute;
 }
 
 [role="tooltip"]::before
 {
   border-left: 10px solid transparent;
   border-right: 10px solid transparent;
-  border-bottom: 10px solid #099CD0;
+  border-bottom: 10px solid #077CA6;
   top: -10px;
   right: 15px;
 }
 
 html[dir="rtl"] [role="tooltip"]::before
 {
   right: auto;
   left: 15px;
 }
 
 .context-menu::before
 {
-  border-left: 10px solid #099CD0;
+  border-left: 10px solid #077CA6;
   border-top: 10px solid transparent;
   border-bottom: 10px solid transparent;
   top: 22px;
   right: -0.5rem;
 }
 
 html[dir="rtl"] .context-menu::before
 {
   border-left: 0rem;
-  border-right: 10px solid #099CD0;
+  border-right: 10px solid #077CA6;
   right: auto;
   left: -0.5rem;
 }
 
 .context-menu li[role="menuitem"]
 {
   width: 12.2rem;
   border: 0rem;
   padding: 0rem;
 }
 
 .context-menu li[role="menuitem"] > *
 {
   width: 100%;
   display: flex;
   border: 0rem;
-  padding: 1rem 0rem;
-  color: #099CD0;
-  font-size: 1.1rem;
-  font-weight: 700;
+  padding: 0.7rem 0rem;
+  color: #077CA6;
+  align-items: center;
+  font-size: 1rem;
+  font-weight: 400;
   text-transform: none;
+  text-decoration: none;
 }
 
 .context-menu li[role="menuitem"] > *:hover,
 .context-menu li[role="menuitem"] > *:focus
 {
   background-color: #E1F2FA;
   cursor: pointer;
 }
 
 .context-menu li[role="menuitem"] > *::before
 {
   content: "";
-  height: 1.5rem;
-  width: 1.5rem;
+  height: 1rem;
+  width: 1rem;
   margin: 0rem 1.1rem;
+  border: 0rem;
 }
 
 .context-menu .update-subscription::before
 {
   background-image: url(icons/reload.svg);
 }
 
 .context-menu .website::before
 {
   background-image: url(icons/globe.svg);
@@ skipping 29 matching lines @@
 {
   display: inline-block;
   -moz-margin-end: 1rem;
   -webkit-margin-end: 1rem;
 }
 
 #social ul li a
 {
   display: block;
   text-align: center;
+  text-decoration: none;
 }
 
 #social ul li a::before
 {
   display: block;
   margin: 0em auto;
   width: 2.5rem;
   height: 2.5rem;
   content: "";
 }
@@ skipping 45 matching lines @@
   width: 50vw;
   margin: auto;
   border-radius: 3px;
   background-color: #FFFFFF;
 }
 
 #dialog header
 {
   display: flex;
   padding: 0.7rem 1.5rem;
-  background-color: #099CD0;
+  background-color: #077CA6;
 }
 
 #dialog header h3
 {
   margin: 0rem;
-  font-size: 1.375rem;
+  font-size: 1rem;
   font-weight: 700;
 }
 
 #dialog-close,
 #hide-notification
 {
   border: 0rem;
   padding: 0rem;
   margin: 0rem;
   background-color: transparent;
@@ skipping 41 matching lines @@
 [data-dialog="about"] button
 {
   margin: 1.5rem auto 1.8rem auto;
 }
 
 [data-dialog="about"] p
 {
   margin: 0.5rem 0rem;
 }
 
-#abp-version
-{
-  margin-bottom: 2rem;
-}
-
 [data-dialog="import"] .side-controls
 {
   margin-top: 2.45rem;
 }
 
 #dialog .table
 {
   width: 100%;
 }
 
@@ skipping 45 matching lines @@
 {
   position: fixed;
   top: 0rem;
   left: 0rem;
   display: flex;
   padding: 1rem 1.9rem;
   width: 100%;
   box-sizing: border-box;
   opacity: 0.8;
   font-size: 1rem;
-  color: #099CD0;
+  color: #077CA6;
   background-color: #E1F2FA;
 }
 
 #notification strong
 {
   flex: 1;
   text-align: center;
 }
 
 #hide-notification
 {
   margin: 0rem 1rem;
 }
 
 #hide-notification::after
 {
   background-image: url(icons/delete.svg#secondary);
 }
 
 #hide-notification:hover::after
 {
   background-image: url(icons/delete.svg#secondary-hover);
 }