Skip to content
New issue

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

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

Already on GitHub? # to your account

fix(toolbox): fix MouseEvent detection error #19620

Merged
merged 1 commit into from
Feb 20, 2024
Merged

Conversation

plainheart
Copy link
Member

@plainheart plainheart commented Feb 16, 2024

Brief Information

This pull request is in the type of:

  • bug fixing
  • new feature
  • others

What does this PR do?

Fix an unexpected replacement in 53babd2#diff-0b91602cf047a98c67d6c4edaacbfa7399c8e0d2d4564b83ece8419d144aea84R64
- Should use the typeof operator to avoid uncaught reference error.

Fixed issues

Document Info

One of the following should be checked.

  • This PR doesn't relate to document changes
  • The document should be updated later
  • The document changes have been made in apache/echarts-doc#xxx

Misc

ZRender Changes

  • This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs

N.A.

Others

Merging options

  • Please squash the commits into a single one when merging.

Other information

@plainheart plainheart changed the title fix(toolbox): use typeof operator rather than isFunction util to detect if MouseEvent object is supported () fix(toolbox): use typeof operator rather than isFunction util to detect if MouseEvent object is supported Feb 16, 2024
Copy link

echarts-bot bot commented Feb 16, 2024

Thanks for your contribution!
The community will review it ASAP. In the meanwhile, please checkout the coding standard and Wiki about How to make a pull request.

The pull request is marked to be PR: author is committer because you are a committer of this project.

@plainheart plainheart added this to the 5.5.1 milestone Feb 16, 2024
Copy link
Contributor

The changes brought by this PR can be previewed at: https://echarts.apache.org/examples/editor?version=PR-19620@385a817

@plainheart plainheart requested a review from pissang February 16, 2024 13:16
@Ovilia Ovilia merged commit 706b74b into master Feb 20, 2024
2 checks passed
@Ovilia Ovilia deleted the fix/toolbox/saveAsImage branch February 20, 2024 08:20
Copy link

echarts-bot bot commented Feb 20, 2024

Congratulations! Your PR has been merged. Thanks for your contribution! 👍

@plainheart plainheart changed the title fix(toolbox): use typeof operator rather than isFunction util to detect if MouseEvent object is supported fix(toolbox): fix MouseEvent detection error Jun 18, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants