Re: [PR] feat: new theme for ECharts 6.0 [echarts]

[via GitHub]

Ovilia commented on PR #20865:
URL: https://github.com/apache/echarts/pull/20865#issuecomment-2861723772

   In the above commit, I added a new option `grid.containDataZoom`. Similar to 
`containLabel`, it adjust grid position to make room for dataZoom. @100pah Do 
you feel this is the correct way to fix the problem?
   
   ## v5
   
   <img width="1046" alt="image" 
src="https://github.com/user-attachments/assets/dd6bb5fe-aeed-424c-86e2-a05ea8b8c679";
 />
   
   ## v6
   
   <img width="1046" alt="image" 
src="https://github.com/user-attachments/assets/c46df2b2-c71f-4200-b450-433670d60685";
 />
   
   Option is:
   
   ```js
   {
     show: true,
     realtime: true,
     start: 30,
     end: 70
   },
   {
     show: true,
     realtime: true,
     yAxisIndex: 0
   }
   ```
   
   and
   
   ```js
   {
     show: true,
     realtime: true,
     start: 30,
     end: 70,
     top: 50
   },
   {
     show: true,
     realtime: true,
     yAxisIndex: 0,
     left: 25
   }
   ```
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@echarts.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@echarts.apache.org
For additional commands, e-mail: commits-h...@echarts.apache.org