Issue 5482 - Sidebar and about ABP dialog

skin/new-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 20 matching lines @@
   src: url(fonts/SourceSansPro-Regular.woff);
   /* local("Ø") forces using no local font called Source Sans Pro */
   src: local("Ø"), url(fonts/SourceSansPro-Regular.woff) format("woff");
   font-weight: 400;
   font-style: normal;
 }
 
 @font-face
 {
   font-family: "Source Sans Pro";
-  src: url(fonts/SourceSansPro-Semibold.woff);
+  src: url(fonts/SourceSansPro-bold.woff);
   /* local("Ø") forces using no local font called Source Sans Pro */
-  src: local("Ø"), url(fonts/SourceSansPro-Semibold.woff) format("woff");
-  font-weight: 600;
+  src: local("Ø"), url(fonts/SourceSansPro-bold.woff) format("woff");
+  font-weight: 700;
   font-style: normal;
 }
 
 html
 {
   font-size: 20px;
 }
 
 body
 {
   background-color: #F5F5F5;
   display: flex;
   margin: 1.2rem 0.3rem;
   font-family: "Source Sans Pro", sans-serif;
   font-weight: 300;

[juliandoucette, 2017/08/23 18:11:11]

Note: This issue is still non-blocking and unresolved (I suggested 400).

[saroyanm, 2017/08/23 20:57:54]

I'll change to 400

[saroyanm, 2017/08/24 17:39:44]

I didn't touch the font size, as I still have a clarification question ->
https://bitbucket.org/adblockplus/spec/issues/69 and question about which font
to use -> https://bitbucket.org/adblockplus/spec/issues/69#comment-39300762

Let's keep yet with old plan, not make this blocker yet, we will fix this in
current review or separately depending on the ticket resolution.

   font-size: 1rem;
   color: #494949;
 }
 
 h1
 {
   font-size: 3rem;
   font-weight: 300;
 }
 
 h2
 {
   font-size: 1.375rem;
-  font-weight: 600;
+  font-weight: 700;
 }
 
 p
 {
   font-weight: 300;
 }
 
 [aria-hidden="true"]
 {
   display: none !important;
@@ skipping 10 matching lines @@
 
 /*
   Buttons and links
  */
 
 button,
 .button
 {
   padding: 0.8rem 1.2rem;
   font-size: 1.125rem;
-  font-weight: 600;
+  font-weight: 700;
   text-decoration: none;
   text-transform: uppercase;
   cursor: pointer;
 }
 
 button.primary,
 .button.primary
 {
   border: 0px;
   color: #FFF;
@@ skipping 14 matching lines @@
 button.secondary,
 .button.secondary
 {
   border: 1px solid #0A9DD1;
   color: #099CD0;
 }
 
 button.secondary:hover,
 .button.secondary:hover
 {
-  box-shadow: inset 0 0 0 1px #099CD0;
+  box-shadow: inset 0 0 0 4px #099CD0;
 }
 
 button[role="checkbox"],
 #dialog-body .table button[role="checkbox"]
 {
   width: 18px;
   height: 18px;
   margin-top: 0px;
   -moz-margin-end: 20px;
   -webkit-margin-end: 20px;
@@ skipping 52 matching lines @@
 
 html[dir="rtl"] #sidebar header
 {
   margin-left: 2rem;
 }
 
 #sidebar header h1
 {
   margin: 0rem;
   font-size: 1.5rem;
+}
+
+#sidebar header h1 strong
+{
+  font-weight: 700;
 }
 
 #sidebar header p
 {
   margin: 0rem;
   font-size: 2.4rem;
 }
 
 html[dir="rtl"] #sidebar header
 {
@@ skipping 41 matching lines @@
 [role="tablist"] a[role="tab"]:focus
 {
   outline: none;
   text-shadow: 0px 0px 1px #888;
 }
 
 li a[role="tab"][aria-selected]
 {
   -moz-border-start-color: #CDCDCD;
   -webkit-border-start-color: #CDCDCD;
-  font-weight: 600;
+  font-weight: 700;
   background-color: #FFF;
 }
 
 #sidebar footer
 {
   bottom: 0px;
   position: absolute;
   width: 100%;
 }
 
 #sidebar footer p
 {
   display: flex;
   justify-content: center;
   margin: 1.2rem 0rem;
 }
 
 main
 {
   background-color: #FFFFFF;
   border: 1px solid #CDCDCD;
   max-width: 46.3rem;

[juliandoucette, 2017/08/23 18:11:11]

Note: The spec doesn't seem to say anything about this max-width.

[juliandoucette, 2017/08/23 18:11:11]

Note: I didn't re-calculate all of the horizontal widths to see if they add up
now.

[saroyanm, 2017/08/23 20:57:54]

Not sure what you mean, but it's border-box element, so paddings are included in
size calculation, if that what you were referring to.

[saroyanm, 2017/08/23 20:57:55]

It does ->
https://bytebucket.org/adblockplus/spec/raw/f20ab3573d020065e415dca85961f3143...


It's states that the width is 46.3em

[juliandoucette, 2017/08/23 22:00:46]

Yes. I was referring to my previous comment where I pointed out that the total
horizontal width of the whole layout was probably wider than you/Jeen intended
and speculated about why that was. I was only pointing out that I didn't
re-check the width.

   padding: 0px 2rem 1.4rem 2rem;
 }
 
 main > div
 {
   display: none;
 }
 
 main h1
 {
@@ skipping 98 matching lines @@
 
 .table button.delete:hover
 {
   background-position: -61px -51px;
 }
 
 .icon,
 button[role="checkbox"],
 #dialog-body .table button[role="checkbox"],
 .table button.delete,
-#content-help a::before,
 #dialog-close::before,
 #all-filter-lists-table .arrow,
 .context-menu .content a::before
 {
   background-image: url(options-sprite.png);
   display: inline-block;
 }
 
 .icon-add,
 .icon-update
@@ skipping 20 matching lines @@
   cursor: pointer;
 }
 
 #dialog-close::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;
 }
@@ skipping 27 matching lines @@
   background: none;
   cursor: pointer;
 }
 
 .controls button span:not(.icon)
 {
   -moz-margin-start: 16px;
   -webkit-margin-start: 16px;
   color: #3A7BA6;
   vertical-align: top;
+}
+
+/* 
+  Used for translatable screen reader only conten.
+  e.g.: Use instead of aria-label to avoid complex attribute value translation
+*/
+.sr-only
+{
+  position: absolute;
+  overflow: hidden;
+  clip: rect(0, 0, 0, 0);
+  width: 1px;
+  height: 1px;
+  margin: -1px;
+  padding: 0px;
+  border: 0px;
 }
 
 /*
   General tab content
 */
 
 #blocking-languages-dialog-table
 {
   border-bottom: none;
 }
@@ skipping 364 matching lines @@
 
 div[role="tooltip"] .notes p
 {
   font-weight: 300;
 }
 
 /*
   Help tab content
 */
 
-#content-help
-{
-  counter-reset: section;
-}
-
-#content-help h1::before
-{
-  counter-increment: section;
-  content: counter(section) ". ";
-}
-
-#content-help a
-{
-  color: #3A7BA6;
+html:not([lang="zh"]) #social-chinese,
+html[lang="zh"] #social-general
+{
+  display: none;
+}
+
+#social ul
+{
+  list-style: none;
+}
+
+#social ul li
+{
   display: inline-block;
-  text-decoration: none;
-  -moz-margin-end: 16px;
-  -webkit-margin-end: 16px;
-  vertical-align: top;
-}
-
-#content-help a::before
-{
-  -moz-margin-end: 6px;
-  -webkit-margin-end: 6px;
-}
-
-#share-general:lang(zh),
-#share-chinese:not(zh)
-{
-  display: none;
+}
+
+#social ul li a
+{
+  display: block;
+  text-align: center;
+}
+
+#social ul li a::before
+{
+  display: block;
+  margin: 0em auto;
+  width: 2.5rem;
+  height: 2.5rem;
+  content: "";
+  background-color: #099DD1;
+}
+
+#twitter::before
+{
+  -webkit-mask: url("social/twitter.svg");
+  mask: url("social/twitter.svg");
+}
+
+#facebook::before
+{
+  -webkit-mask: url("social/facebook.svg");
+  mask: url("social/facebook.svg");
+}
+
+#google-plus::before
+{
+  -webkit-mask: url("social/googleplus.svg");
+  mask: url("social/googleplus.svg");
 }
 
 /*
   Dialog
 */
 
 #dialog-background
 {
   display: none;
   position: fixed;
@@ skipping 116 matching lines @@
 body:not([data-dialog="language-change"]) #dialog-title-language-change,
 body:not([data-dialog="language-add"]):not([data-dialog="language-change"]) #dialog-content-language-add,
 body:not([data-dialog="language-add"]) #dialog-body button.add,
 body:not([data-dialog="language-change"]) #dialog-body button.change,
 body:not([data-dialog="predefined"]) #dialog-title-predefined,
 body:not([data-dialog="predefined"]) #dialog-content-predefined,
 body:not([data-dialog]) #dialog
 {
   display: none;
 }
+
+/*
+  Notification
+*/
+
+#notification
+{
+  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;
+  background-color: #E1F2FA;
+}
+
+#notification strong
+{
+  flex: 1;
+  text-align: center;
+}
+
+#hide-notification
+{
+  border: 0rem;
+  padding: 0rem;
+  margin: 0rem 1rem;
+  background-color: transparent;
+  cursor: pointer;
+}
+
+#hide-notification::after
+{
+  content: "";
+  display: block;
+  height: 1rem;
+  width: 1rem;
+  border: 0rem;
+  background-color: #099DD1;
+  -webkit-mask: url(delete.svg);
+  mask: url(delete.svg);
+}
+
+#hide-notification:hover::after
+{
+  background-color: #5CBCE1;
+}