Merge remote-tracking branch 'origin/master' into cam/7334/switch-geodata-providers

Author: camdecoster

.github/workflows/test.yml

@@ -0,0 +1,33 @@
+# This workflow will do a clean installation of node dependencies, cache/restore them, build the source code and run tests across different versions of node
+# For more information see: https://docs.github.com/en/actions/automating-builds-and-tests/building-and-testing-nodejs
+
+name: No CI Test
+
+on: push
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        node-version: [18.x]
+
+    steps:
+    - uses: browser-actions/setup-chrome@v1
+    - uses: actions/checkout@v4
+    - name: Use Node.js ${{ matrix.node-version }}
+      uses: actions/setup-node@v4
+      with:
+        node-version: ${{ matrix.node-version }}
+        cache: 'npm'
+    - run: chrome --version
+    - run: ls
+    - run: npm run pretest
+    - run: npm ci
+    - run: npm run cibuild
+    - name: Run headless test
+      uses: coactions/setup-xvfb@v1
+      with:
+        run: ./tasks/noci_test.sh jasmine

devtools/regl_codegen/devtools.js

@@ -158,5 +158,3 @@ function handleOnLoad() {
         window.close();
     });
 }
-
-module.exports = Tabs;

devtools/test_dashboard/devtools.js

@@ -268,4 +268,3 @@ function handleOnLoad() {
     plotFromHash();
 }
 
-module.exports = Tabs;

draftlogs/7280_add.md

@@ -0,0 +1 @@
+- Add `pattern.path` attribute as an alternative to the preset `pattern.shape` values, so you can use any SVG path string as a pattern fill. [[#7280](https://github.com/plotly/plotly.js/pull/7280)]

draftlogs/7407_fix.md

@@ -0,0 +1 @@
+- Move css-loader and other dependencies into devDependencies section [[#7407](https://github.com/plotly/plotly.js/pull/7407)]

draftlogs/7420_add.md

@@ -0,0 +1,2 @@
+ - Add support for ticklabelposition "inside"/"outside" for category axes with `tickson` set to "boundaries" [[#7420](https://github.com/plotly/plotly.js/pull/7420)],
+   with thanks to @my-tien for the contribution!

package-lock.json

@@ -86,15 +86,13 @@
     "color-parse": "2.0.0",
     "color-rgba": "3.0.0",
     "country-regex": "^1.1.0",
-    "css-loader": "^7.1.2",
     "d3-force": "^1.2.1",
     "d3-format": "^1.4.5",
     "d3-geo-projection": "^2.9.0",
     "d3-hierarchy": "^1.1.9",
     "d3-interpolate": "^3.0.1",
     "d3-time": "^1.1.0",
     "d3-time-format": "^2.2.3",
-    "esbuild-style-plugin": "^1.6.3",
     "fast-isnumeric": "^1.1.4",
     "gl-mat4": "^1.2.0",
     "gl-text": "^1.4.0",
@@ -115,7 +113,6 @@
     "regl-scatter2d": "^3.3.1",
     "regl-splom": "^1.0.14",
     "strongly-connected-components": "^1.0.1",
-    "style-loader": "^4.0.0",
     "superscript-text": "^1.0.0",
     "svg-path-sdf": "^1.1.3",
     "tinycolor2": "^1.4.2",
@@ -138,13 +135,15 @@
     "canvas": "^3.1.0",
     "check-node-version": "^4.2.1",
     "chttps": "^1.0.6",
+    "css-loader": "^7.1.2",
     "d3-geo": "^3.1.1",
     "deep-equal": "^2.2.3",
     "ecstatic": "^4.1.4",
     "esbuild": "^0.25.5",
     "esbuild-plugin-browserify-adapter": "^0.1.4",
     "esbuild-plugin-environment": "^0.4.0",
     "esbuild-plugin-glsl": "^1.2.2",
+    "esbuild-style-plugin": "^1.6.3",
     "extra-iterable": "^2.5.22",
     "falafel": "^2.2.5",
     "fs-extra": "^10.1.0",
@@ -180,6 +179,7 @@
     "run-series": "^1.1.9",
     "sass": "^1.78.0",
     "stream-browserify": "^3.0.0",
+    "style-loader": "^4.0.0",
     "through2": "^4.0.2",
     "topojson": "^1.6.27",
     "transform-loader": "^0.2.4",

package.json

@@ -28,6 +28,16 @@ exports.pattern = {
             'By default, no pattern is used for filling the area.',
         ].join(' ')
     },
+    path: {
+        valType: 'string',
+        arrayOk: true,
+        editType: 'style',
+        description: [
+            'Sets a custom path for pattern fill.',
+            'Use with no `shape` or `solidity`, provide an SVG path string for',
+            'the regions of the square from (0,0) to (`size`,`size`) to color.'
+        ].join(' ')
+    },
     fillmode: {
         valType: 'enumerated',
         values: ['replace', 'overlay'],

src/components/drawing/attributes.js

@@ -218,13 +218,14 @@ drawing.dashStyle = function(dash, lineWidth) {
 function setFillStyle(sel, trace, gd, forLegend) {
     var markerPattern = trace.fillpattern;
     var fillgradient = trace.fillgradient;
-    var patternShape = markerPattern && drawing.getPatternAttr(markerPattern.shape, 0, '');
+    var pAttr = drawing.getPatternAttr;
+    var patternShape = markerPattern && (pAttr(markerPattern.shape, 0, '') || pAttr(markerPattern.path, 0, ''));
     if(patternShape) {
-        var patternBGColor = drawing.getPatternAttr(markerPattern.bgcolor, 0, null);
-        var patternFGColor = drawing.getPatternAttr(markerPattern.fgcolor, 0, null);
+        var patternBGColor = pAttr(markerPattern.bgcolor, 0, null);
+        var patternFGColor = pAttr(markerPattern.fgcolor, 0, null);
         var patternFGOpacity = markerPattern.fgopacity;
-        var patternSize = drawing.getPatternAttr(markerPattern.size, 0, 8);
-        var patternSolidity = drawing.getPatternAttr(markerPattern.solidity, 0, 0.3);
+        var patternSize = pAttr(markerPattern.size, 0, 8);
+        var patternSolidity = pAttr(markerPattern.solidity, 0, 0.3);
         var patternID = trace.uid;
         drawing.pattern(sel, 'point', gd, patternID,
             patternShape, patternSize, patternSolidity,
@@ -662,6 +663,16 @@ drawing.pattern = function(sel, calledBy, gd, patternID, shape, size, solidity,
                 fill: fgRGB
             };
             break;
+        default:
+            width = size;
+            height = size;
+            patternTag = 'path';
+            patternAttrs = {
+                d: shape,
+                opacity: opacity,
+                fill: fgRGB
+            };
+            break;
     }
 
     var str = [
@@ -869,7 +880,10 @@ drawing.singlePointStyle = function(d, sel, trace, fns, gd, pt) {
         }
 
         var markerPattern = marker.pattern;
-        var patternShape = markerPattern && drawing.getPatternAttr(markerPattern.shape, d.i, '');
+        var pAttr = drawing.getPatternAttr;
+        var patternShape = markerPattern && (
+            pAttr(markerPattern.shape, d.i, '') || pAttr(markerPattern.path, d.i, '')
+        );
 
         if(gradientType && gradientType !== 'none') {
             var gradientColor = d.mgc;
@@ -888,14 +902,15 @@ drawing.singlePointStyle = function(d, sel, trace, fns, gd, pt) {
                 fgcolor = pt.color;
                 perPointPattern = true;
             }
-            var patternFGColor = drawing.getPatternAttr(fgcolor, d.i, (pt && pt.color) || null);
+            var patternFGColor = pAttr(fgcolor, d.i, (pt && pt.color) || null);
 
-            var patternBGColor = drawing.getPatternAttr(markerPattern.bgcolor, d.i, null);
+            var patternBGColor = pAttr(markerPattern.bgcolor, d.i, null);
             var patternFGOpacity = markerPattern.fgopacity;
-            var patternSize = drawing.getPatternAttr(markerPattern.size, d.i, 8);
-            var patternSolidity = drawing.getPatternAttr(markerPattern.solidity, d.i, 0.3);
+            var patternSize = pAttr(markerPattern.size, d.i, 8);
+            var patternSolidity = pAttr(markerPattern.solidity, d.i, 0.3);
             perPointPattern = perPointPattern || d.mcc ||
                 Lib.isArrayOrTypedArray(markerPattern.shape) ||
+                Lib.isArrayOrTypedArray(markerPattern.path) ||
                 Lib.isArrayOrTypedArray(markerPattern.bgcolor) ||
                 Lib.isArrayOrTypedArray(markerPattern.fgcolor) ||
                 Lib.isArrayOrTypedArray(markerPattern.size) ||