100pah opened a new pull request #11574: fix: fix bar width strategy.
URL: https://github.com/apache/incubator-echarts/pull/11574
 
 
   
   fix: fix bar width strategy.
   
   (1) On value axis, the auto-calculated bar width might be near 0, which make 
the bar invisible.
   So we add option `barMinWidth` (default 1 in category and default null in 
other coord sys).
   
   (2) In #5316 (commit 2823ab502bf1af27c33de6d09729bc287ef0a185) the fix seems 
not correct.
   check the "main3" case in `echarts/test/bar-width.html`. That fix would made 
the `barMaxWidth` has
   higher priority than `barWidth`. But it only works when auto-calculated bar 
width is bigger than the 
   giving `barWidth`. So the behavior seems weird. 
   This fix revert it and use the strategy: `barWidth` always has higher 
priority than `barMaxWidth`.
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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

Reply via email to