Issue 5326 - General tab (HTML, strings and functionality)

skin/new-options.css

Index: skin/new-options.css
===================================================================
--- a/skin/new-options.css
+++ b/skin/new-options.css
@@ -100,7 +100,8 @@
   box-sizing: border-box;
 }
 
-button[role="checkbox"]
+button[role="checkbox"],
+#dialog-body .table button[role="checkbox"]

[Thomas Greiner, 2017/08/09 18:14:50]

Detail: Why are those duplicated selectors for checkboxes necessary? On first
glance it appears to work without it.

[saroyanm, 2017/08/14 14:00:11]

I'll recheck, but I almost sure I did it because of specificity

[Thomas Greiner, 2017/08/15 17:10:30]

Acknowledged.

[saroyanm, 2017/08/16 14:17:39]

Yes it is because of specificity, anyway I'll revisit the Dialog styles
separately.

[Thomas Greiner, 2017/08/16 17:57:08]

Acknowledged.

 {
   width: 18px;
   height: 18px;
@@ -113,25 +114,12 @@
   background-position: -51px 0px;
 }
 
-button[role="checkbox"][aria-checked="true"]
+button[role="checkbox"][aria-checked="true"],
+#dialog-body .table button[role="checkbox"][aria-checked="true"]
 {
   background-position: -68px 0px;
 }
 
-.option-name
-{
-  display: flex;
-  margin-bottom: 16px;
-}
-
-.option-name > :first-child
-{
-  flex: 1;
-  overflow: hidden;
-  text-overflow: ellipsis;
-  white-space: nowrap;
-}
-
 #nav-sidebar
 {
   min-width: 198px;
@@ -163,17 +151,6 @@
   padding: 8px 0px 16px 0px;
 }
 
-.hbox
-{
-  display: flex;
-  flex-direction: row;
-}
-
-.hbox > div
-{
-  width: 400px;
-}
-
 .tabs li
 {
   cursor: pointer;
@@ -375,14 +352,12 @@
   border-radius: 3px;
 }
 
-.table.list,
-#custom-wrapper
+.table.list
 {
   border-bottom: 1px solid #CDCDCD;
 }
 
-.table.list li .display,
-.fill
+.table.list li .display
 {
   flex: 1;
   line-height: 16px;
@@ -397,13 +372,22 @@
   padding-bottom: 6px;
 }
 
-.table.list li:nth-of-type(odd),
-.table.cols li:nth-of-type(odd):not([role="menuitem"]),
-.table li.empty-placeholder
+  .table li.empty-placeholder
 {
   background-color: #F5F5F5;
 }
 
+.table li [data-single='visible'],
+.table li:first-child:nth-last-child(2) [data-single='hidden']

[Thomas Greiner, 2017/08/09 18:14:51]

Detail: It's a nice trick to select an element if it's the only one but it's not
easy to understand because you'd have to know that the last element will always
be the `<template>` element.

Therefore let's use `li:first-of-type:last-of-type` instead because it makes
clear that we're only selecting the element if it's both the first and last
`<li>`.

[saroyanm, 2017/08/14 14:00:10]

I'll try that.

[saroyanm, 2017/08/16 14:17:38]

Done.

+{
+  display: none;
+}
+
+.table li:first-child:nth-last-child(2) [data-single='visible']

[Thomas Greiner, 2017/08/09 18:14:50]

Suggestion: You could move that selector into the rule above using
`li:not(:first-of-type:last-of-type)` to avoid having to know whether it's
supposed to be a block or an inline element.

[saroyanm, 2017/08/14 14:00:11]

Acknowledged.

[saroyanm, 2017/08/16 14:17:38]

Apparently it's not possible to have two pseudo classes in the :not

[Thomas Greiner, 2017/08/16 17:57:08]

Interesting, I wasn't aware of that. They only accept simple selectors according
to https://www.w3.org/TR/css3-selectors/#negation so you're right.

FYI: The upcoming Selectors Level 4 standard appears to change that so it
accepts a selector list instead. See https://www.w3.org/TR/selectors4/#negation

+{
+  display: block;
+}
+
 .table label
 {
   vertical-align: top;
@@ -439,10 +423,10 @@
 
 .icon,
 button[role="checkbox"],
+#dialog-body .table button[role="checkbox"],
 .table button.delete,
 #content-help a::before,
 #dialog-close::before,
-#dialog-body button::before,
 #all-filter-lists-table .arrow,
 .context-menu .content a::before
 {
@@ -482,8 +466,7 @@
   width: 12px;
 }
 
-#content-help a::before,
-#dialog-body button::before
+#content-help a::before
 {
   background-position: 0px -42px;
   content: "";
@@ -537,11 +520,7 @@
   General tab content
 */
 
-#acceptableads-table,
-#recommend-list-table,
-#blocking-languages-dialog-table,
-#custom-wrapper .table,
-#all-lang-table
+#blocking-languages-dialog-table
 {
   border-bottom: none;
 }
@@ -553,18 +532,53 @@
   -webkit-margin-end: 40px;
 }
 
-#custom-wrapper
+#whitelisting .controls.mode-edit > button,
+#whitelisting .controls:not(.mode-edit) > div
 {
-  height: 290px;
-  overflow: auto;
+  display: none;
 }
 
-#custom-wrapper .table
+#whitelisting .controls input[type="text"]
 {
+  border: 0px;
+  border-bottom: 1px solid #A1A1A1;
+  -moz-padding-end: 25px;
+  -webkit-padding-end: 25px;
+  -moz-margin-start: 14px;
+  -webkit-margin-start: 14px;
+  outline: 0px;
+  padding-bottom: 5px;
+  vertical-align: text-bottom;
   width: 100%;
 }
 
-.button-add
+#whitelisting .controls .button-add span
+{
+  -moz-margin-start: 5px;
+  -webkit-margin-start: 5px;
+}
+
+#whitelisting .button-add
+{
+  -moz-margin-start: 32px;
+  -webkit-margin-start: 32px;
+}
+
+#whitelisting-add-button + span
+{
+  flex: 1;
+}
+
+.icon-enter
+{
+  background-position: -19px -33px;
+  position: absolute;
+  bottom: 9px;
+  -moz-margin-start: -20px;
+  -webkit-margin-start: -20px;
+}
+
+.button-add, .cancel-button
 {
   background-color: transparent;
   border: 0px;
@@ -582,7 +596,8 @@
 #custom-filters[data-mode="empty"] #custom-filters-raw,
 #custom-filters:not([data-mode="write"]) #custom-filters-raw-controls,
 #custom-filters:not([data-mode="read"]) #custom-filters-edit,
-.state span
+.state span,
+#acceptable-ads:not(.show-dnt-notification) #no-dnt
 {
   display: none;
 }
@@ -728,13 +743,6 @@
   z-index: 1;
 }
 
-html[dir="ltr"] div[role="tooltip"].flip-horizontal,
-html[dir="rtl"] div[role="tooltip"]:not(.flip-horizontal)
-{
-  left: inherit;
-  right: -20px;
-}
-
 .tooltip:not(:hover) > div[role="tooltip"],
 div[role="tooltip"]:hover
 {
@@ -846,13 +854,6 @@
   background-position: -71px -34px;
 }
 
-html[dir="ltr"] div[role="tooltip"].flip-horizontal::before,
-html[dir="rtl"] div[role="tooltip"]:not(.flip-horizontal)::before
-{
-  left: inherit;
-  right: 30px;
-}
-
 div[role="tooltip"] img
 {
   float: left;
@@ -1024,11 +1025,6 @@
   width: 100%;
 }
 
-#dialog #dialog-content-language .table
-{
-  overflow: auto;
-}
-
 #dialog .section:not(:first-child)
 {
   margin-top: 24px;
@@ -1071,48 +1067,13 @@
 
 body:not([data-dialog="custom"]) #dialog-title-custom,
 body:not([data-dialog="custom"]) #dialog-content-custom,
-body:not([data-dialog="language"]) #dialog-title-language,
-body:not([data-dialog="language"]) #dialog-content-language,
+body:not([data-dialog="language-add"]) #dialog-title-language-add,
+body:not([data-dialog="language-add"]) #dialog-content-language-add,
+body:not([data-dialog="language-change"]) #dialog-title-language-change,
+body:not([data-dialog="language-change"]) #dialog-content-language-change,
 body:not([data-dialog="predefined"]) #dialog-title-predefined,
 body:not([data-dialog="predefined"]) #dialog-content-predefined,
 body:not([data-dialog]) #dialog
 {
   display: none;
 }
-
-#dialog-content-language .dialog-content-block
-{
-  display: flex;
-  flex-direction: column;
-  height: 120px;
-}
-
-#dialog-content-language #other-language
-{
-  height: 200px;
-}
-
-#other-language .button-add
-{
-  background-color: transparent;
-  margin: 0px;
-  padding: 0px;
-  width: auto;
-  -webkit-border-end: 1px solid #CDCDCD;
-  -moz-border-end: 1px solid #CDCDCD;
-  -webkit-padding-end: 10px;
-  -moz-padding-end: 10px;
-  -webkit-padding-start: 0px;
-  -moz-padding-start: 0px;
-}
-
-#other-language .button-add::before
-{
-  display: none;
-}
-
-#other-language .display
-{
-  -webkit-margin-start: 10px;
-  -moz-margin-start: 10px;
-}