From 979fe4e9e655b78ec092c77805f4e9567d4bc941 Mon Sep 17 00:00:00 2001
From: sz-p <sz_p@outlook.com>
Date: Mon, 23 Dec 2024 16:39:36 +0800
Subject: [PATCH 1/2] fix(legend): symbolRotate for legend is neither inherited
 nor being set #20562

---
 src/component/legend/LegendView.ts | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/src/component/legend/LegendView.ts b/src/component/legend/LegendView.ts
index 9016468240..7d1b34777d 100644
--- a/src/component/legend/LegendView.ts
+++ b/src/component/legend/LegendView.ts
@@ -404,12 +404,7 @@ class LegendView extends ComponentView {
         }
         else {
             // Use default legend icon policy for most series
-            const rotate = legendIconType === 'inherit' && seriesModel.getData().getVisual('symbol')
-                ? (iconRotate === 'inherit'
-                    ? seriesModel.getData().getVisual('symbolRotate')
-                    : iconRotate
-                )
-                : 0; // No rotation for no icon
+            const rotate = zrUtil.retrieve2(seriesModel.getData().getVisual('symbolRotate'), iconRotate);
             itemGroup.add(getDefaultLegendIcon({
                 itemWidth,
                 itemHeight,

From c55256efb6c21746b06b260fc0ea7083980a87e5 Mon Sep 17 00:00:00 2001
From: sz-p <sz_p@outlook.com>
Date: Mon, 23 Dec 2024 19:23:11 +0800
Subject: [PATCH 2/2] test(legend): symbolRotate for legend is neither
 inherited nor being set #20562

---
 test/legend-style.html | 160 ++++++++++++++++++++++++++++++++++++++++-
 1 file changed, 159 insertions(+), 1 deletion(-)

diff --git a/test/legend-style.html b/test/legend-style.html
index 803dac2ee0..146dae3fb9 100644
--- a/test/legend-style.html
+++ b/test/legend-style.html
@@ -37,6 +37,10 @@
         <div id="main1"></div>
         <div id="main2"></div>
         <div id="main3"></div>
+        <div id="main4"></div>
+        <div id="main5"></div>
+        <div id="main6"></div>
+        <div id="main7"></div>
 
         <script>
             function getData(seriesId) {
@@ -46,9 +50,16 @@
                 }
                 return data;
             }
+            function getScatterData(seriesId) {
+                var data = [];
+                for (var i = 0; i < 7; ++i) {
+                    data.push([Math.random() * 100, Math.random() * 100]);
+                }
+                return data;
+            }
         </script>
 
-        <script>
+         <script>
         require(['echarts'], function (echarts) {
             var option = {
                 xAxis: {
@@ -331,5 +342,152 @@
             });
         });
     </script>
+
+        <script>
+        require(['echarts'], function (echarts) {
+            var option = {
+                textStyle: {
+                    color: 'green'
+                },
+                legend: {
+                    show: true,
+                    symbolRotate: 'inherit',
+                    itemWidth:15,
+                    itemHeight:15
+                },
+                xAxis: {},
+                yAxis: {},
+                series: [
+                    {
+                    name: 'triangle',
+                    symbolSize: 20,
+                    symbol: 'triangle',
+                    symbolRotate: 180,
+                    data: getScatterData(0),
+                    type: 'scatter'
+                    },
+                ],
+            };
+
+            var chart = testHelper.create(echarts, 'main4', {
+                title: [
+                    'Symbol should have **rotate inherit** in legend'
+                ],
+                option: option
+            });
+        });
+        </script>
+
+        <script>
+            require(['echarts'], function (echarts) {
+                var option = {
+                    textStyle: {
+                        color: 'green'
+                    },
+                    legend: {
+                        show: true,
+                        symbolRotate: 45,
+                        itemWidth:15,
+                        itemHeight:15
+                    },
+                    xAxis: {},
+                    yAxis: {},
+                    series: [
+                        {
+                        name: 'rect',
+                        symbolSize: 20,
+                        symbol: 'rect',
+                        data: getScatterData(0),
+                        type: 'scatter'
+                        },
+                    ],
+                };
+
+                var chart = testHelper.create(echarts, 'main5', {
+                    title: [
+                        'Symbol should have **rotate 45** in legend'
+                    ],
+                    option: option
+                });
+            });
+        </script>
+
+        <script>
+            require(['echarts'], function (echarts) {
+                var option = {
+                    textStyle: {
+                        color: 'green'
+                    },
+                    legend: {
+                        show: true,
+                        itemWidth:15,
+                        itemHeight:15
+                    },
+                    xAxis: {},
+                    yAxis: {},
+                    series: [
+                        {
+                        name: 'arrow',
+                        symbolSize: 20,
+                        symbol: 'arrow',
+                        data: getScatterData(0),
+                        type: 'scatter'
+                        },
+                    ],
+                };
+
+                var chart = testHelper.create(echarts, 'main6', {
+                    title: [
+                        'Symbol should have **rotate inherit** as **default** in legend'
+                    ],
+                    option: option
+                });
+            });
+        </script>
+
+        <script>
+            require(['echarts'], function (echarts) {
+                var option = {
+                    tooltip: {
+                        trigger: 'item'
+                    },
+                    legend: {
+                        orient: 'vertical',
+                        verticalAlign:"middle",
+                        left: 'right',
+                        top:"middle",
+                        symbolRotate: 45
+                    },
+                    series: [
+                        {
+                        name: 'Access From',
+                        type: 'pie',
+                        radius: '50%',
+                        data: [
+                            { value: 1048, name: 'Search Engine' },
+                            { value: 735, name: 'Direct' },
+                            { value: 580, name: 'Email' },
+                            { value: 484, name: 'Union Ads' },
+                            { value: 300, name: 'Video Ads' }
+                        ],
+                        emphasis: {
+                            itemStyle: {
+                            shadowBlur: 10,
+                            shadowOffsetX: 0,
+                            shadowColor: 'rgba(0, 0, 0, 0.5)'
+                            }
+                        }
+                        }
+                    ]
+                };
+
+                var chart = testHelper.create(echarts, 'main7', {
+                    title: [
+                        'Symbol should have **rotate 45** in legend'
+                    ],
+                    option: option
+                });
+            });
+        </script>
     </body>
 </html>