Merge pull request #22 from MEOM/tweak/sub-nav

Note about  CSS needed for the SVG icons.

Author: samikeijonen

changelog.md

@@ -1,4 +1,7 @@
 # Changelog
+## Version 1.0.3 released November 3, 2021
+### Added
+- Note about `pointer-events: none;` CSS needed for the SVG icons.
 ## Version 1.0.2 released November 2, 2021
 ### Added
 - Demo for language dropdown.

dist/index.esm.min.js

@@ -1,6 +1,6 @@
 {
   "name": "@meom/navigation",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "MEOM navigation",
   "homepage": "https://github.com/MEOM/navigation",
   "repository": {

dist/index.min.js

@@ -114,6 +114,22 @@ if (!navElement || !navToggle) {
 new Navigation(navElement, navToggle);
 ```
 
+## Issues opening and closing the sub menus?
+First try adding `pointer-events: none;` CSS to SVG icon inside the sub menu toggles.
+
+For example:
+
+```css
+.site-nav__sub-toggle svg,
+.site-nav__sub-sub-toggle svg {
+    pointer-events: none;
+}
+```
+
+This prevents SVG icon being the clicked target element. In other words it should always the `<button>` itself. 
+
+Hopefully this will be fixed in the JS at some point.
+
 ## Options
 
 There are several settings (object) which you can pass in as third argument: