fix: Remove invalid css declarations and lint leftovers

background-color/min-width do not accept 'none'; browsers ignored the
declarations, so removing them does not change rendering. Also an
unused event parameter and a stale eslint-disable directive.

Author: lfrancke

ui/src/css/base.css

@@ -33,10 +33,6 @@ a:hover {
   text-decoration: underline;
 }
 
-a:active {
-  background-color: none;
-}
-
 code,
 kbd,
 pre {

ui/src/css/header.css

@@ -254,13 +254,11 @@ body {
 
 .mobile-popup-menu .navbar-item.has-dropdown {
   flex-direction: column;
-  min-width: none;
 }
 
 .mobile-popup-menu .navbar-dropdown {
   display: inline;
   position: relative;
-  min-width: none;
 }
 
 /** indent sub items a little bit to distinguish them visually */

ui/src/js/03-fragment-jumper.js

@@ -21,7 +21,7 @@
     window.scrollTo(0, computePosition(this, 0) - toolbar.getBoundingClientRect().bottom)
   }
 
-  window.addEventListener('load', function jumpOnLoad (e) {
+  window.addEventListener('load', function jumpOnLoad () {
     var fragment, target
     if ((fragment = decodeFragment(window.location.hash)) && (target = document.getElementById(fragment))) {
       jumpToAnchor.bind(target)()

ui/src/js/06-copy-to-clipboard.js

@@ -67,7 +67,7 @@
     window.navigator.clipboard.writeText(text).then(
       function () {
         this.classList.add('clicked')
-        this.offsetHeight // eslint-disable-line no-unused-expressions
+        this.offsetHeight
         this.classList.remove('clicked')
       }.bind(this),
       function () {}