PR-50 Fixed the moreLinks element and its children

* fixed classes * gave it an arrow cursor instead of the text cursor Related #50
OXID-eSales · Aug 1, 2019 · 6818e23 · 6818e23
1 parent dd95638
commit 6818e23
Show file tree

Hide file tree

Showing 5 changed files with 11,179 additions and 34 deletions.
diff --git a/build/js/main.js b/build/js/main.js
@@ -271,10 +271,10 @@ $(function () {
                     oMoreLinkAElem = document.createElement('a'),
                     oMoreLinkUlElem = document.createElement('ul');
 
-                oMoreLinkElem.className = 'dropdown moreLinks';
+                oMoreLinkElem.className = 'dropdown moreLinks nav-item';
 
-                oMoreLinkAElem.className = 'dropdown-toggle';
-                oMoreLinkAElem.innerHTML = 'Mehr <span class="caret"></span>';
+                oMoreLinkAElem.className = 'dropdown-toggle nav-link';
+                oMoreLinkAElem.innerHTML = 'Mehr <i class="fa fa-angle-down"></i>';
                 oMoreLinkAElem.setAttribute('data-toggle', 'dropdown');
 
                 oMoreLinkUlElem.className = 'dropdown-menu';
@@ -299,6 +299,8 @@ $(function () {
                     iNavItemsWidth += $this.outerWidth();
 
                     if (iNavItemsWidth > iMainNavWidth) {
+                        $this.attr('class', 'dropdown-item');
+                        $this.find('.nav-link').attr('class', 'dropdown-link');
                         aMoreLinkElems.push($this);
                     }
                 }

diff --git a/build/scss/layout/_header.scss b/build/scss/layout/_header.scss
@@ -266,6 +266,14 @@
       &-link {
         color: $brand-primary;
       }
+
+      .moreLinks > .nav-link {
+        cursor: default;
+      }
+
+      .moreLinks > .nav-link::after {
+        display: none;
+      }
     }
   }
 }