master-wowl, dropdown fixes round 2

addons/web/static/src/core/debug/debug_menu.xml

@@ -5,7 +5,7 @@
         <Dropdown class="o_debug_manager"
           beforeOpen="beforeOpenDropdown"
           t-on-dropdown-item-selected.stop="onDropdownItemSelected"
-          menuClass="'o_dropdown_menu_right'">
+          togglerClass="'btn btn-light'">
             <span class="fa fa-bug"/>
             <t t-set-slot="menu">
                 <t t-foreach="getElements()" t-as="element" t-key="element_index">

addons/web/static/src/core/dropdown/dropdown.scss

@@ -1,5 +1,5 @@
-// Default Boostrap Classes customizations
-//------------------------------------------------------------------------------
+// = Default Boostrap Classes customizations
+// ----------------------------------------------------------------------------
 .dropdown-menu {
   min-width: 100%;
   max-height: 90vh;
@@ -34,7 +34,9 @@
   }
 
   &.disabled a, &:disabled a {
-    @include o-hover-text-color($dropdown-link-disabled-color, $dropdown-link-disabled-color);
+    @include o-hover-text-color(
+      $dropdown-link-disabled-color,
+      $dropdown-link-disabled-color);
   }
 }
 
@@ -48,20 +50,23 @@
 }
 
 .dropdown-item-text {
-  // Commonly used in to wrap forms into dropdown menus.
-  // Specify a reasonable minimum-width to avoid inner elements
-  // to overlap each other.
+  // Commonly used in to wrap forms into dropdown menus. Specify a reasonable
+  // minimum-width to avoid that inner elements overlap each other.
   min-width: 15rem;
 }
 
+.dropdown-divider:first-child {
+  display: none;
+}
+
 .o_rtl {
   .dropdown-menu {
     @extend .dropdown-menu-right;
   }
 }
 
-// Extend Owl custom classes to visually match Boostrap ones
-//------------------------------------------------------------------------------
+// = Extend Owl custom classes to visually match Boostrap ones
+// ----------------------------------------------------------------------------
 .o_dropdown {
   @extend .dropdown;
 
@@ -82,8 +87,8 @@
   }
 }
 
-// Owl components specific rules
-//------------------------------------------------------------------------------
+// = Owl components specific rules
+// -----------------------------------------------------------------------------
 .o_dropdown_item:not(.disabled):not(:disabled) {
   // Needed 'cause <DropdownItem> generate <li> html tags that for browsers are
   // normally not clickable.

addons/web/static/src/legacy/scss/dropdown.scss

@@ -1,7 +1,13 @@
-.ui-widget.ui-autocomplete { // Copy style of bootstrap dropdown
+// = jQueryUI dropdown adaptations
+// ----------------------------------------------------------------------------
+.ui-widget.ui-autocomplete {
     @extend .dropdown-menu;
-    top: 0; // Necessary to correctly compute the initial position
-    min-width: 0; // Necessary to correctly compute the initial width
+    top: 0;       // Compute initial position
+    min-width: 0; // Compute initial width
+
+    // Needed because .ui-widget are rendered at a
+    // lower stacking contex compared to modals.
+    z-index: $zindex-modal + 1;
 
     .ui-menu-item {
         padding: 0;

addons/web/static/src/legacy/scss/navbar.scss

@@ -63,6 +63,7 @@
       background: $-backend-entry-hover-bg;
       outline: 1px solid $o-navbar-inverse-link-hover-bg;
       outline-offset: -1px;
+      color: $white;
     }
   }
 
@@ -76,7 +77,7 @@
 
   // = Owl <Dropdown> components
   // --------------------------------------------------------------------------
-  .o_dropdown_toggler {
+  .o_dropdown .o_dropdown_toggler {
     @extend %-main-navbar-entry-base;
     @extend %-main-navbar-entry-padding;
     @extend %-main-navbar-entry-bg-hover;

addons/web/static/tests/legacy/views/pivot_tests.js

@@ -2690,7 +2690,12 @@ QUnit.module('Views', {
             View: PivotView,
             model: "partner",
             data: this.data,
-            arch: `<pivot/>`,
+            // have at least a measure to have a separator in the Measures dropdown:
+            //
+            // Foo
+            // -----
+            // Count
+            arch: `<pivot><field name="foo" type="measure"/></pivot>`,
         });
 
         // open the "Measures" menu