Skip to content

Fix warnings and errors in demo #839

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jun 24, 2025
Merged

Fix warnings and errors in demo #839

merged 2 commits into from
Jun 24, 2025

Conversation

kingyue737
Copy link
Member

Address some warnings and errors in the demo to enhance developer experience, making truly important alerts and errors more noticeable.

  1. Avoid reinitializing esbuild-wasm during HMR, which causes an error — this issue also occurred previously when using Webpack.

Screenshot 2025-06-24 114901

  1. normal has been removed since ECharts v4
  2. Add splitNumbers in radar to suppress alignTicks warning

Screenshot 2025-06-24 113414

Copy link

vercel bot commented Jun 24, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
vue-echarts ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 24, 2025 5:25am

@Justineo Justineo merged commit 9c3c230 into 8.0 Jun 24, 2025
4 checks passed
@kingyue737 kingyue737 deleted the demo-warning branch June 24, 2025 05:46
kingyue737 added a commit that referenced this pull request Jul 4, 2025
* chore: fix warnings and errors in demo (#839)

* chore: suppress warning in demo

* chore: prevent multiple intializations of esbuild-wasm in demo HMR

* feat: dynamically update the theme (#841)

Co-authored-by: GU Yiling <[email protected]>

* feat: add dataView slot

* vibe docs

---------

Co-authored-by: GU Yiling <[email protected]>
Justineo added a commit that referenced this pull request Jul 25, 2025
* feat: experimental component rendered tooltip

* revert slot in VChart

* feat: use tooltip composable

* feat: try createApp

* feat: use pie chart as tooltip

* feat: switch to createVNode

The limitation is that the tooltip detached from the current component tree, not provide/inject

will try teleport next

* feat: try component with teleport

* wip

* add xAxis example

* refactor with shallowReactive

* Support dynamic slot

* fix: fill empty elements with object in array

* shallow copy option along the path

* ssr friendly

* vibe docs

* typo

* update according to the review

* add dataView slot

* chore: fix warnings and errors in demo (#839)

* chore: suppress warning in demo

* chore: prevent multiple intializations of esbuild-wasm in demo HMR

* feat: dynamically update the theme (#841)

Co-authored-by: GU Yiling <[email protected]>

* feat: add dataView slot

* vibe docs

---------

Co-authored-by: GU Yiling <[email protected]>

* fix docs typo

* update according to the review

* small fix

* remove wrapper around slotProp

* update comments

* remove anys

* add tooltip slot prop type

* target to vue 3.3

* move slot related codes to slot.ts

---------

Co-authored-by: GU Yiling <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants