Merge remote-tracking branch 'origin/master' into lab/modal-without-bottom-buttons

elm.json

@@ -3,7 +3,7 @@
     "name": "NoRedInk/noredink-ui",
     "summary": "UI Widgets we use at NRI",
     "license": "BSD-3-Clause",
-    "version": "7.4.0",
+    "version": "7.4.1",
     "exposed-modules": [
         "Nri.Ui",
         "Nri.Ui.Alert.V2",

src/Nri/Ui/Icon/V3.elm

@@ -894,4 +894,9 @@ svgStyle =
     [ RootAttr.style "fill" "currentColor"
     , RootAttr.style "width" "100%"
     , RootAttr.style "height" "100%"
+
+    -- don't allow SVG to capture pointer events. This means that we can use it
+    -- in links. See https://css-tricks.com/links-inline-svg-staying-target-events/
+    -- for more details.
+    , RootAttr.style "pointer-events" "none"
     ]

src/Nri/Ui/Icon/V4.elm

@@ -941,4 +941,9 @@ svgStyle =
     [ RootAttr.style "fill" "currentColor"
     , RootAttr.style "width" "100%"
     , RootAttr.style "height" "100%"
+
+    -- don't allow SVG to capture pointer events. This means that we can use it
+    -- in links. See https://css-tricks.com/links-inline-svg-staying-target-events/
+    -- for more details.
+    , RootAttr.style "pointer-events" "none"
     ]

src/Nri/Ui/Icon/V5.elm

@@ -746,4 +746,9 @@ svgStyle =
     [ RootAttr.style "fill" "currentColor"
     , RootAttr.style "width" "100%"
     , RootAttr.style "height" "100%"
+
+    -- don't allow SVG to capture pointer events. This means that we can use it
+    -- in links. See https://css-tricks.com/links-inline-svg-staying-target-events/
+    -- for more details.
+    , RootAttr.style "pointer-events" "none"
     ]