Skip to content

fix(visualmap): add textShadowColor, make text shadow work on visualMap. close #14440#16679

Merged
pissang merged 4 commits intoapache:masterfrom
fuchunhui:fix-14440
Mar 17, 2022
Merged

fix(visualmap): add textShadowColor, make text shadow work on visualMap. close #14440#16679
pissang merged 4 commits intoapache:masterfrom
fuchunhui:fix-14440

Conversation

@fuchunhui
Copy link
Copy Markdown
Contributor

@fuchunhui fuchunhui commented Mar 14, 2022

Brief Information

This pull request is in the type of:

  • bug fixing
  • new feature
  • others

What does this PR do?

Fix a bug that some text styles are not working on visualMap.

Fixed issues

Details

visualMap-continuous

Before After
image image

visualMap-piecewise

Before After
image image

Misc

  • The API has been changed (apache/echarts-doc#xxx).
  • This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs

  • test/visualMap-continuous.html
  • test/visualMap-pieces.html

Others

Merging options

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

Other information

@echarts-bot
Copy link
Copy Markdown

echarts-bot bot commented Mar 14, 2022

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.

@pull-request-size pull-request-size bot added size/M and removed size/S labels Mar 14, 2022
plainheart
plainheart previously approved these changes Mar 14, 2022
Copy link
Copy Markdown
Member

@plainheart plainheart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Member

@plainheart plainheart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One more question,
Should we also apply the text style on handleLabel and indicatorLabel?

@plainheart plainheart linked an issue Mar 14, 2022 that may be closed by this pull request
Ovilia
Ovilia previously approved these changes Mar 14, 2022
Copy link
Copy Markdown
Contributor

@Ovilia Ovilia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fuchunhui
Copy link
Copy Markdown
Contributor Author

@plainheart Your editing is great, more smarter.

@pissang
Copy link
Copy Markdown
Contributor

pissang commented Mar 15, 2022

Should we also apply the text style on handleLabel and indicatorLabel?

@fuchunhui So are we planning to add this in this PR as @plainheart suggested?

@fuchunhui fuchunhui dismissed stale reviews from Ovilia and plainheart via a255fa1 March 15, 2022 02:47
@pull-request-size pull-request-size bot added size/M and removed size/S labels Mar 15, 2022
@pissang pissang added this to the 5.3.2 milestone Mar 15, 2022
Copy link
Copy Markdown
Member

@plainheart plainheart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pissang pissang merged commit 7bf453a into apache:master Mar 17, 2022
@echarts-bot
Copy link
Copy Markdown

echarts-bot bot commented Mar 17, 2022

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Text shadow is not working on visualMap

4 participants