feat(boxplot) correctly handle series.encode with category axis

apache · Sep 5, 2024 · bc5c888 · bc5c888
1 parent 0ffc94b
commit bc5c888
Show file tree

Hide file tree

Showing 2 changed files with 150 additions and 3 deletions.
diff --git a/src/chart/helper/whiskerBoxCommon.ts b/src/chart/helper/whiskerBoxCommon.ts
@@ -52,6 +52,16 @@ class WhiskerBoxCommonMixin<Opts extends CommonOption> {
 
     defaultValueDimensions: CoordDimensionDefinition['dimsDef'];
 
+    /**
+     * @private
+     */
+    _hasEncodeRule(key: string) {
+        const encodeRules = this.getEncode();
+        if(encodeRules?.data?.has(key))
+            return encodeRules.data.get(key) !== null;
+        return false;
+    }
+
     /**
      * @override
      */
@@ -74,12 +84,12 @@ class WhiskerBoxCommonMixin<Opts extends CommonOption> {
         if (xAxisType === 'category') {
             option.layout = 'horizontal';
             ordinalMeta = xAxisModel.getOrdinalMeta();
-            addOrdinal = true;
+            addOrdinal = !this._hasEncodeRule('x');
         }
         else if (yAxisType === 'category') {
             option.layout = 'vertical';
             ordinalMeta = yAxisModel.getOrdinalMeta();
-            addOrdinal = true;
+            addOrdinal = !this._hasEncodeRule('y');
         }
         else {
             option.layout = option.layout || 'horizontal';
@@ -161,4 +171,4 @@ class WhiskerBoxCommonMixin<Opts extends CommonOption> {
 };
 
 
-export {WhiskerBoxCommonMixin};
+export {WhiskerBoxCommonMixin};
diff --git a/test/boxplot-category.html b/test/boxplot-category.html