@instructure/ui-menu 8.22.0 → 8.22.1-snapshot.21

package/LICENSE.md

@@ -0,0 +1,27 @@
+---
+title: The MIT License (MIT)
+category: Getting Started
+order: 9
+---
+
+# The MIT License (MIT)
+
+Copyright (c) 2015 Instructure, Inc.
+
+**Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions.**
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/es/Menu/index.js

@@ -69,6 +69,11 @@ let Menu = (_dec = withDeterministicId(), _dec2 = withStyle(generateStyle, gener
 
       if (typeof menuRef === 'function') {
         menuRef(el);
+      } // If there is no trigger `<ul>` is the ref, otherwise the trigger
+
+
+      if (!this.props.trigger) {
+        this.ref = el;
       }
     };
 
@@ -414,6 +419,9 @@ let Menu = (_dec = withDeterministicId(), _dec2 = withStyle(generateStyle, gener
       onMouseOver: onMouseOver,
       offsetX: offsetX,
       offsetY: offsetY,
+      elementRef: element => {
+        this.ref = element;
+      },
       ref: el => {
         this._popover = el;
 
@@ -424,7 +432,6 @@ let Menu = (_dec = withDeterministicId(), _dec2 = withStyle(generateStyle, gener
       renderTrigger: safeCloneElement(trigger, {
         ref: el => {
           this._trigger = el;
-          this.ref = (el === null || el === void 0 ? void 0 : el.ref) || el;
         },
         'aria-haspopup': true,
         id: this._labelId,

package/lib/Menu/index.js

@@ -92,6 +92,11 @@ let Menu = (_dec = (0, _withDeterministicId.withDeterministicId)(), _dec2 = (0,
 
       if (typeof menuRef === 'function') {
         menuRef(el);
+      } // If there is no trigger `<ul>` is the ref, otherwise the trigger
+
+
+      if (!this.props.trigger) {
+        this.ref = el;
       }
     };
 
@@ -437,6 +442,9 @@ let Menu = (_dec = (0, _withDeterministicId.withDeterministicId)(), _dec2 = (0,
       onMouseOver: onMouseOver,
       offsetX: offsetX,
       offsetY: offsetY,
+      elementRef: element => {
+        this.ref = element;
+      },
       ref: el => {
         this._popover = el;
 
@@ -447,7 +455,6 @@ let Menu = (_dec = (0, _withDeterministicId.withDeterministicId)(), _dec2 = (0,
       renderTrigger: (0, _safeCloneElement.safeCloneElement)(trigger, {
         ref: el => {
           this._trigger = el;
-          this.ref = (el === null || el === void 0 ? void 0 : el.ref) || el;
         },
         'aria-haspopup': true,
         id: this._labelId,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@instructure/ui-menu",
-  "version": "8.22.0",
+  "version": "8.22.1-snapshot.21+7bdf07c83",
   "description": "A dropdown menu component",
   "author": "Instructure, Inc. Engineering and Product Design",
   "module": "./es/index.js",
@@ -23,27 +23,27 @@
   },
   "license": "MIT",
   "devDependencies": {
-    "@instructure/ui-babel-preset": "8.22.0",
-    "@instructure/ui-color-utils": "8.22.0",
-    "@instructure/ui-test-locator": "8.22.0",
-    "@instructure/ui-test-queries": "8.22.0",
-    "@instructure/ui-test-utils": "8.22.0",
-    "@instructure/ui-themes": "8.22.0"
+    "@instructure/ui-babel-preset": "8.22.1-snapshot.21+7bdf07c83",
+    "@instructure/ui-color-utils": "8.22.1-snapshot.21+7bdf07c83",
+    "@instructure/ui-test-locator": "8.22.1-snapshot.21+7bdf07c83",
+    "@instructure/ui-test-queries": "8.22.1-snapshot.21+7bdf07c83",
+    "@instructure/ui-test-utils": "8.22.1-snapshot.21+7bdf07c83",
+    "@instructure/ui-themes": "8.22.1-snapshot.21+7bdf07c83"
   },
   "dependencies": {
     "@babel/runtime": "^7.13.10",
-    "@instructure/console": "8.22.0",
-    "@instructure/emotion": "8.22.0",
-    "@instructure/shared-types": "8.22.0",
-    "@instructure/ui-a11y-utils": "8.22.0",
-    "@instructure/ui-dom-utils": "8.22.0",
-    "@instructure/ui-icons": "8.22.0",
-    "@instructure/ui-popover": "8.22.0",
-    "@instructure/ui-position": "8.22.0",
-    "@instructure/ui-prop-types": "8.22.0",
-    "@instructure/ui-react-utils": "8.22.0",
-    "@instructure/ui-testable": "8.22.0",
-    "@instructure/ui-utils": "8.22.0",
+    "@instructure/console": "8.22.1-snapshot.21+7bdf07c83",
+    "@instructure/emotion": "8.22.1-snapshot.21+7bdf07c83",
+    "@instructure/shared-types": "8.22.1-snapshot.21+7bdf07c83",
+    "@instructure/ui-a11y-utils": "8.22.1-snapshot.21+7bdf07c83",
+    "@instructure/ui-dom-utils": "8.22.1-snapshot.21+7bdf07c83",
+    "@instructure/ui-icons": "8.22.1-snapshot.21+7bdf07c83",
+    "@instructure/ui-popover": "8.22.1-snapshot.21+7bdf07c83",
+    "@instructure/ui-position": "8.22.1-snapshot.21+7bdf07c83",
+    "@instructure/ui-prop-types": "8.22.1-snapshot.21+7bdf07c83",
+    "@instructure/ui-react-utils": "8.22.1-snapshot.21+7bdf07c83",
+    "@instructure/ui-testable": "8.22.1-snapshot.21+7bdf07c83",
+    "@instructure/ui-utils": "8.22.1-snapshot.21+7bdf07c83",
     "keycode": "^2",
     "prop-types": "^15"
   },
@@ -53,5 +53,6 @@
   "publishConfig": {
     "access": "public"
   },
-  "sideEffects": false
+  "sideEffects": false,
+  "gitHead": "7bdf07c834ffb614098415cbe0e3df6c0f6efdfb"
 }

package/src/Menu/index.tsx

@@ -110,6 +110,10 @@ class Menu extends Component<MenuProps> {
     if (typeof menuRef === 'function') {
       menuRef(el)
     }
+    // If there is no trigger `<ul>` is the ref, otherwise the trigger
+    if (!this.props.trigger) {
+      this.ref = el
+    }
   }
 
   constructor(props: MenuProps) {
@@ -487,6 +491,9 @@ class Menu extends Component<MenuProps> {
         onMouseOver={onMouseOver}
         offsetX={offsetX}
         offsetY={offsetY}
+        elementRef={(element) => {
+          this.ref = element
+        }}
         ref={(el) => {
           this._popover = el
           if (typeof popoverRef === 'function') {
@@ -496,7 +503,6 @@ class Menu extends Component<MenuProps> {
         renderTrigger={safeCloneElement(trigger as ReactElement, {
           ref: (el: (React.ReactInstance & { ref?: Element }) | null) => {
             this._trigger = el
-            this.ref = el?.ref || (el as Element)
           },
           'aria-haspopup': true,
           id: this._labelId,

package/src/Menu/props.ts

@@ -55,7 +55,8 @@ type MenuOwnProps = {
    */
   children?: React.ReactNode // TODO: oneOf(['MenuItem', 'MenuItemGroup', 'MenuItemSeparator', 'Menu'])
   /**
-   * Description of the `<Menu />`
+   * Description of the `<Menu />`. The component uses it to add its value to
+   * the `aria-label` attribute.
    */
   label?: string
   /**