ecomfe / vue-echarts

Vue.js component for Apache ECharts™.
https://vue-echarts.dev
MIT License
9.6k stars 1.49k forks source link

v6.7.3 Hides Charts #796

Closed davidkassa closed 1 month ago

davidkassa commented 2 months ago

Confirmation

How are you introducing Vue-ECharts into your project?

ES Module imports

Versions

dvc-rofr-stats@0.1.0 C:\Code\dvc-rofr-stats
├─┬ @vitejs/plugin-vue2@2.3.1              
│ └── vue@2.7.16 deduped                   
├─┬ @vue/test-utils@1.3.6                  
│ └── vue@2.7.16 deduped                   
├─┬ buefy@0.9.29                           
│ └── vue@2.7.16 deduped                   
├── echarts@5.5.1                          
├─┬ vue-echarts@6.7.3                      
│ ├── echarts@5.5.1 deduped                
│ ├─┬ vue-demi@0.13.11                     
│ │ └── vue@2.7.16 deduped                 
│ └── vue@2.7.16 deduped                   
├─┬ vue-json-csv@1.2.12                    
│ └── vue@2.7.16 deduped                   
├─┬ vue-multi-select@4.6.0                 
│ └── vue@2.7.16 deduped                   
└── vue@2.7.16

Details

When upgrading from 6.7.2 to 6.7.3 my charts disappear. When I downgrade they show back up.

This PR shows the only diff is the update with a preview URL.

Reproduction

https://github.com/davidkassa/dvc-rofr-stats/pull/2822

kingyue737 commented 2 months ago

https://github.com/davidkassa/dvc-rofr-stats/blob/7a6878824d07bc9e1ae039019560f60ff7804673/src/views/WaitTimes.vue#L69

Because you set vue-echarts container as inline-block. v6.7.3 introduced a regression that chart container became flex. This will be reverted in v7. Currently you can downgrade to v6.7.2 or replace inline-block with inline-flex.

Justineo commented 1 month ago

As @kingyue737 said. You may try vue-echarts@7.0.0-beta.0 to see if the problem still exists.