Issue 5549 - Implement missing error handlings for custom filter lists

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 201 matching lines @@
 button.list:hover
 {
   box-shadow: inset 0 0 0 3px #077CA6;
   border-color: #077CA6;
 }
 
 .side-controls:not(.wrap)
 {
   margin: 0.8rem 0rem;
   display: flex;
+  align-self: center;
   justify-content: flex-end;
 }
 
 .side-controls button
 {
   margin: 0rem;
   -moz-margin-start: 1rem;
   -webkit-margin-start: 1rem;
 }
 
@@ skipping 305 matching lines @@
 
 #sidebar,
 #sidebar .fixed,
 [role="tablist"]
 {
   width: 14.3rem;
 }
 
 #sidebar
 {
-  flex-shrink: 0; 
+  flex-shrink: 0;
 }
 
 #sidebar .fixed
 {
   top: 1.2rem;
   bottom: 0rem;
   height: auto;
 }
 
 #sidebar header
@@ skipping 95 matching lines @@
   #sidebar .fixed
   {
     position: fixed;
   }
 
   #sidebar footer
   {
     bottom: 0px;
     position: absolute;
   }
+

[saroyanm, 2018/02/02 16:57:20]

Nit: I think this is again an IDE auto-correction.

 }
 
 /*
   Main content
  */
 
 body[data-tab|="general"] #content-general,
 body[data-tab|="advanced"] #content-advanced,
 body[data-tab|="whitelist"] #content-whitelist,
 body[data-tab|="help"] #content-help
@@ skipping 454 matching lines @@
 /*
   Advanced tab content
 */
 
 #all-filter-lists-table li.show-message .last-update,
 #all-filter-lists-table li:not(.show-message) .message,
 #custom-filters:not([data-mode="empty"]) #empty-custom-filters,
 #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,
+#custom-filters-error,
 .state span,
 #acceptable-ads:not(.show-dnt-notification) #dnt
 {
   display: none;
 }
 
-#all-filter-lists-table button[role="checkbox"][aria-checked="true"] + .state #state-active
-{
-  display: inline;
-}
-
+#all-filter-lists-table button[role="checkbox"][aria-checked="true"] + .state #state-active,
 #all-filter-lists-table button[role="checkbox"][aria-checked="false"] + .state #state-disabled
 {
   display: inline;
 }
 
 #all-filter-lists-table
 {
   margin-bottom: 0.8rem;
 }
 
+#custom-filters-edit
+{
+  white-space: nowrap;
+}
+
+#custom-filters-edit-area
+{
+  display: flex;
+  width: 100%;
+}
+
+/*
+  avoid doubled margin glitchy behavior
+  as shown in visual specifications
+*/
+#custom-filters-edit-area .side-controls .side-controls
+{
+  margin: 0;
+}
+
+/*
+  visibility: hidden;
+  preserves the flex-box meaning,
+  keeping buttons on the right.
+*/
+#custom-filters-edit-error
+{
+  visibility: hidden;
+  display: flex;
+  align-self: center;
+  justify-content: flex-start;
+  font-weight: bold;
+}
+
+#custom-filters.warning #custom-filters-edit-error,
+#custom-filters-error
+{
+  color: #C11D26;
+  padding: 0.2rem 0.8rem;
+}
+
+#custom-filters.warning #custom-filters-edit-error
+{
+  visibility: visible;
+}
+
+#custom-filters-error
+{
+  display: block;
+  list-style: none;
+  white-space: pre;
+  overflow: auto;
+}
+
+#custom-filters-error.many
+{
+  border: 1px solid silver;
+  max-height: 6.2rem;
+}
+
 #custom-filters-raw
 {
   width: 100%;
   height: 23.6rem;
-  padding: 0.8rem; 
+  padding: 0.8rem;
 }
 
 #custom-filters-raw:focus
 {
   outline: none;
   border: 2px solid #077CA6;
 }
 
+#custom-filters.warning #custom-filters-raw
+{
+  outline: none;
+  border: 2px solid #C11D26;
+}
+
 #empty-custom-filters
 {
   padding: 1.5rem;
   border: 1px solid #CDCDCD;
 }
 
 /*
   Context menu
  */
 
@@ skipping 158 matching lines @@
 
 #social ul
 {
   list-style: none;
   padding: 0px;
 }
 
 #social ul li
 {
   display: inline-block;
+  margin-right: 1rem;

[saroyanm, 2018/02/02 16:57:20]

Why was this change necessary ?
We already had end margin specified.

[a.giammarchi, 2018/02/02 17:22:00]

We cannot use margin-end because it's not compatible with Edge.
This part was not covered by the other ticket that will remove following
margin-end from this file.

I'm fixing issues knowing already there will be one (tested on Edge)

[saroyanm, 2018/02/02 17:58:57]

Right, we have created issue #5813 to tackle that.
The problem with tackling it here, we will need to be sure that we consider RTL
languages, so this will probably require having a hack like:
html[dir="rtl"] #social ul li
{
  margin-left: 1rem;
}

and also removal of margin-end. While we need to tackle that not only for social
media buttons, but we are using margin-end and padding-end in a lot of places,
I'll suggest to leave it for #5813.

   -moz-margin-end: 1rem;
   -webkit-margin-end: 1rem;
 }
 
 #social ul li a
 {
   display: block;
   text-align: center;
   text-decoration: none;
 }
@@ skipping 152 matching lines @@
   font-size: 1rem;
   color: #077CA6;
   background-color: #E1F2FA;
 }
 
 #notification strong
 {
   flex: 1;
   text-align: center;
 }