Fix Stylelint errors

app/Resources/static/themes/material/css/article.scss

@@ -54,7 +54,7 @@
     }
 
     a {
-      border-bottom: 1px dotted $blueAccentColor;
+      border-bottom: 1px dotted $blue-accent-color;
       text-decoration: none;
     }
 
@@ -105,7 +105,7 @@
     margin: 2.1rem 0 0.68rem;
   }
 
-  aside {
+  .entry-info {
     .tools {
       display: flex;
       flex-flow: row wrap;

app/Resources/static/themes/material/css/cards.scss

@@ -73,7 +73,7 @@ main {
 
   .card-entry-labels-hidden li {
     display: inline-block;
-    background-color: $blueAccentColor;
+    background-color: $blue-accent-color;
     margin: 0 5px;
     padding: 5px 12px;
     border-radius: 3px;
@@ -85,10 +85,6 @@ main {
     white-space: nowrap;
   }
 
-  .card-content .estimatedTime {
-    margin-bottom: 10px;
-  }
-
   .card-action {
     padding: 10px 10px 10px 15px;
 
@@ -160,7 +156,7 @@ a.original:not(.waves-effect) {
 .card-tag-labels li {
   margin: 10px 10px 10px auto;
   padding: 5px 12px 5px 16px !important;
-  background-color: $blueAccentColor;
+  background-color: $blue-accent-color;
   border-radius: 3px;
   color: #fff;
   cursor: default;
@@ -251,7 +247,7 @@ a.original:not(.waves-effect) {
     }
 
     .chip {
-      background-color: $blueAccentColor;
+      background-color: $blue-accent-color;
       padding: 0 7px;
       margin: auto 1px;
       border-radius: 6px;
@@ -315,7 +311,7 @@ a.original:not(.waves-effect) {
   color: #fff !important;
 }
 
-.settings .div_tabs {
+.settings .tabs-container {
   padding-bottom: 15px;
 }
 

app/Resources/static/themes/material/css/dark_theme.scss

@@ -131,7 +131,7 @@
     background-color: #282c34;
   }
 
-  nav input {
+  nav input {
     color: #abb2bf;
   }
 

app/Resources/static/themes/material/css/icons.scss

@@ -27,7 +27,7 @@
   -webkit-font-smoothing: antialiased;
 
   /* Support for Safari and Chrome. */
-  text-rendering: optimizeLegibility;
+  text-rendering: optimizelegibility;
 
   /* Support for Firefox. */
   -moz-osx-font-smoothing: grayscale;

app/Resources/static/themes/material/css/layout.scss

@@ -19,7 +19,7 @@ body {
 }
 
 a {
-  color: $blueAccentColor;
+  color: $blue-accent-color;
 }
 
 main,

app/Resources/static/themes/material/css/nav.scss

@@ -131,11 +131,11 @@ nav {
   margin: 0 1%;
 }
 
-#button_filters {
+.button-filters {
   display: none;
 }
 
-#button_export {
+.button-export {
   display: none;
 }
 

app/Resources/static/themes/material/css/print.scss

@@ -18,20 +18,16 @@
 
   /* Hide useless blocks */
   body > header,
-  #article_toolbar,
-  #links,
-  #sort,
   body > footer,
-  .top_link,
+  .entry-tools,
-  div.tools,
   header div,
   .messages,
   .entry + .results,
-  #slide-out,
+  .left-bar,
   .progress,
   .hide-on-large-only,
-  #article > aside,
+  .entry-info,
-  #article .mbm a {
+  .title-edit {
     display: none !important;
   }
 
@@ -39,7 +35,7 @@
     padding-left: 0 !important;
   }
 
-  #article {
+  .article {
     margin: inherit !important;
   }
 

app/Resources/static/themes/material/css/sidenav.scss

@@ -45,6 +45,6 @@
   font-weight: bold;
 }
 
-span.numberItems {
+.items-number {
   float: right;
 }

app/Resources/static/themes/material/css/variables.scss

@@ -2,4 +2,4 @@
    Variables
    ========================================================================== */
 
-$blueAccentColor: #00acc1;
+$blue-accent-color: #00acc1;

src/Wallabag/CoreBundle/Resources/views/Config/index.html.twig

@@ -9,7 +9,7 @@
             <div class="card-panel settings">
 
                 <div class="row">
-                    <div class="div_tabs col s12">
+                    <div class="tabs-container col s12">
                         <ul class="tabs">
                             <li class="tab col s12 m6 l3"><a class="active" href="#set1">{{ 'config.tab_menu.settings'|trans }}</a></li>
                             <li class="tab col s12 m6 l3"><a href="#set2">{{ 'config.tab_menu.feed'|trans }}</a></li>

src/Wallabag/CoreBundle/Resources/views/Entry/entry.html.twig

@@ -36,7 +36,7 @@
             </ul>
         </div>
     </nav>
-    <ul id="slide-out" class="collapsible side-nav fixed reader-mode" data-collapsible="accordion">
+    <ul id="slide-out" class="left-bar collapsible side-nav fixed reader-mode" data-collapsible="accordion">
         <li class="bold border-bottom hide-on-med-and-down">
             <a class="waves-effect collapsible-header" href="{{ path('homepage') }}">
                 <i class="material-icons small">arrow_back</i>
@@ -232,12 +232,12 @@
 {% endblock %}
 
 {% block content %}
-    <div id="article">
+    <div id="article" class="article">
         <header class="mbm">
-            <h1><span{% if entry.language is defined and entry.language is not null %} lang="{{ entry.getHTMLLanguage() }}"{% endif %}>{{ entry.title|striptags|default('entry.default_title'|trans)|raw }}</span> <a href="{{ path('edit', {'id': entry.id}) }}" title="{{ 'entry.view.edit_title'|trans }}"><i class="material-icons grey-text">create</i></a></h1>
+            <h1><span{% if entry.language is defined and entry.language is not null %} lang="{{ entry.getHTMLLanguage() }}"{% endif %}>{{ entry.title|striptags|default('entry.default_title'|trans)|raw }}</span> <a class="title-edit" href="{{ path('edit', {'id': entry.id}) }}" title="{{ 'entry.view.edit_title'|trans }}"><i class="material-icons grey-text">create</i></a></h1>
         </header>
-        <aside>
+        <aside class="entry-info">
-            <div class="tools grey-text" dir="auto">
+            <div class="tools entry-tools grey-text" dir="auto">
                 <ul class="stats">
                     <li>
                         {% include "@WallabagCore/Entry/_reading_time.html.twig" with {'entry': entry} only %}
@@ -273,7 +273,7 @@
                     {% if entry.originUrl is not empty %}
                         <li>
                             <i class="material-icons grey-text" title="{{ 'entry.view.provided_by'|trans }}">launch</i>
-                            <a href="{{ entry.originUrl|e }}" target="_blank" rel="noopener" class="tool grey-text">
+                            <a href="{{ entry.originUrl|e }}" target="_blank" rel="noopener" class="tool grey-text" data-tests="entry-origin-url">
                                 {{ entry.originUrl|striptags|removeSchemeAndWww|u.truncate(32) }}
                             </a>
                         </li>

src/Wallabag/CoreBundle/Resources/views/Static/about.html.twig

@@ -9,7 +9,7 @@
             <div class="card-panel settings">
 
                 <div class="row">
-                    <div class="div_tabs col s12">
+                    <div class="tabs-container col s12">
                         <ul class="tabs">
                         <li class="tab col s12 m6 l3"><a class="active" href="#set1">{{ 'about.top_menu.who_behind_wallabag'|trans }}</a></li>
                         <li class="tab col s12 m6 l3"><a href="#set2">{{ 'about.top_menu.getting_help'|trans }}</a></li>

src/Wallabag/CoreBundle/Resources/views/Static/howto.html.twig

@@ -9,7 +9,7 @@
             <div class="card-panel settings">
 
                 <div class="row">
-                    <div class="div_tabs col s12">
+                    <div class="tabs-container col s12">
                         <ul class="tabs">
                             <li class="tab col s12 m6 l3"><a class="active" href="#set1">{{ 'howto.tab_menu.add_link'|trans }}</a></li>
                             <li class="tab col s12 m6 l3"><a href="#set2">{{ 'howto.tab_menu.shortcuts'|trans }}</a></li>

src/Wallabag/CoreBundle/Resources/views/layout.html.twig

@@ -25,7 +25,7 @@
 
 {% block menu %}
     <nav class="cyan darken-1">
-        <ul id="slide-out" class="side-nav fixed">
+        <ul id="slide-out" class="left-bar side-nav fixed">
             {% block logo %}
                 <li class="logo border-bottom">
                     <a title="{{ 'menu.left.back_to_unread'|trans }}" href="{{ path('unread') }}">
@@ -53,22 +53,22 @@
             {% endif %}
 
             <li class="bold {% if active_route == 'unread' %}active{% endif %}">
-                <a class="waves-effect" href="{{ path('unread') }}">{{ 'menu.left.unread'|trans }} <span class="numberItems grey-text">{{ count_entries('unread') }}</span></a>
+                <a class="waves-effect" href="{{ path('unread') }}">{{ 'menu.left.unread'|trans }} <span class="items-number grey-text">{{ count_entries('unread') }}</span></a>
             </li>
             <li class="bold {% if active_route == 'starred' %}active{% endif %}">
-                <a class="waves-effect" href="{{ path('starred') }}">{{ 'menu.left.starred'|trans }} <span class="numberItems grey-text">{{ count_entries('starred') }}</span></a>
+                <a class="waves-effect" href="{{ path('starred') }}">{{ 'menu.left.starred'|trans }} <span class="items-number grey-text">{{ count_entries('starred') }}</span></a>
             </li>
             <li class="bold {% if active_route == 'archive' %}active{% endif %}">
-                <a class="waves-effect" href="{{ path('archive') }}">{{ 'menu.left.archive'|trans }} <span class="numberItems grey-text">{{ count_entries('archive') }}</span></a>
+                <a class="waves-effect" href="{{ path('archive') }}">{{ 'menu.left.archive'|trans }} <span class="items-number grey-text">{{ count_entries('archive') }}</span></a>
             </li>
             <li class="bold {% if active_route == 'annotated' %}active{% endif %}">
-                <a class="waves-effect" href="{{ path('annotated') }}">{{ 'menu.left.with_annotations'|trans }} <span class="numberItems grey-text">{{ count_entries('annotated') }}</span></a>
+                <a class="waves-effect" href="{{ path('annotated') }}">{{ 'menu.left.with_annotations'|trans }} <span class="items-number grey-text">{{ count_entries('annotated') }}</span></a>
             </li>
             <li class="bold {% if active_route == 'all' %}active{% endif %}">
-                <a class="waves-effect" href="{{ path('all') }}">{{ 'menu.left.all_articles'|trans }} <span class="numberItems grey-text">{{ count_entries('all') }}</span></a>
+                <a class="waves-effect" href="{{ path('all') }}">{{ 'menu.left.all_articles'|trans }} <span class="items-number grey-text">{{ count_entries('all') }}</span></a>
             </li>
             <li class="bold {% if current_route == 'tags' %}active{% endif %}">
-                <a class="waves-effect" href="{{ path('tag') }}">{{ 'menu.left.tags'|trans }} <span class="numberItems grey-text">{{ count_tags() }}</span></a>
+                <a class="waves-effect" href="{{ path('tag') }}">{{ 'menu.left.tags'|trans }} <span class="items-number grey-text">{{ count_tags() }}</span></a>
             </li>
         </ul>
         <div class="nav-panels">
@@ -98,12 +98,12 @@
                         </a>
                     </li>
                     {% endif %}
-                    <li id="button_filters">
+                    <li id="button_filters" class="button-filters">
                         <a class="nav-panel-menu button-collapse-right tooltipped js-filters-action" data-position="bottom" data-delay="50" data-tooltip="{{ 'menu.top.filter_entries'|trans }}" href="#" data-activates="filters">
                             <i class="material-icons">filter_list</i>
                         </a>
                     </li>
-                    <li id="button_export">
+                    <li id="button_export" class="button-export">
                         <a class="nav-panel-menu button-collapse-right tooltipped js-export-action" data-position="bottom" data-delay="50" data-tooltip="{{ 'menu.top.export'|trans }}" href="#" data-activates="export">
                             <i class="material-icons">file_download</i>
                         </a>

tests/Wallabag/CoreBundle/Controller/EntryControllerTest.php

@@ -591,8 +591,8 @@ class EntryControllerTest extends WallabagCoreTestCase
 
         $this->assertGreaterThan(1, $title = $crawler->filter('div[id=article] h1')->extract(['_text']));
         $this->assertStringContainsString('My updated title hehe :)', $title[0]);
-        $this->assertGreaterThan(1, $stats = $crawler->filter('div[class="tools grey-text"] ul[class=stats] li a[class="tool grey-text"]')->extract(['_text']));
+        $originUrl = $crawler->filter('[data-tests="entry-origin-url"]')->text();
-        $this->assertStringContainsString('example.io', trim($stats[1]));
+        $this->assertStringContainsString('example.io', $originUrl);
     }
 
     public function testEditRemoveOriginUrl()
@@ -626,9 +626,8 @@ class EntryControllerTest extends WallabagCoreTestCase
         $this->assertGreaterThan(1, $title);
         $this->assertStringContainsString('My updated title hehe :)', $title[0]);
 
-        $stats = $crawler->filter('div[class="tools grey-text"] ul[class=stats] li a[class="tool grey-text"]')->extract(['_text']);
+        $originUrl = $crawler->filter('[data-tests="entry-origin-url"]')->extract(['_text']);
-        $this->assertCount(1, $stats);
+        $this->assertCount(0, $originUrl);
-        $this->assertStringNotContainsString('example.io', trim($stats[0]));
     }
 
     public function testToggleArchive()