Skip to content
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

[5.0] [FEATURE] more configurable styles for lineStyle & borderStyle. #12961

Merged
merged 33 commits into from Aug 28, 2020

Conversation

plainheart
Copy link
Member

@plainheart plainheart commented Jul 13, 2020

Brief Information

This pull request is in the type of:

  • bug fixing
  • new feature
  • others

What does this PR do?

Support for configuring the following options for lineStyle.

  • lineDashArray (dashArray)
  • lineDashOffset (dashOffset)
  • lineCap (cap)
  • lineJoin (join)
  • miterLimit (miterLimit)

Support for configuring the following options for border.

  • lineDashArray (borderDashArray)
  • lineDashOffset (borderDashOffset)
  • lineCap (borderCap) (NOT for label text)
  • lineJoin (borderJoin) (NOT for label text)
  • miterLimit (borderMiterLimit) (NOT for label text)

TODOs:

Fixed issues

N/A now.

Details

Before: What was the problem?

The line and border lack support for dash styles, such as dashArray, dashOffset, cap, join and etc.

After: How is it fixed in this PR?

Added some new options about the dash style, as mentioned above.

image

image

Usage

Are there any API changes?

  • The API has been changed.

Refer to the mentioned above.

Related test cases or examples to use the new APIs

test/line-style.html

Others

Merging options

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

Other information

@echarts-bot
Copy link

echarts-bot bot commented Jul 13, 2020

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.

@pissang
Copy link
Contributor

pissang commented Jul 15, 2020

Looks great!

@pull-request-size pull-request-size bot added size/M and removed size/S labels Jul 16, 2020
@pull-request-size pull-request-size bot added size/L and removed size/M labels Jul 16, 2020
@plainheart plainheart changed the title feat: more configurable styles for lineStyle. feat: more configurable styles for lineStyle & borderStyle. Jul 16, 2020
@pull-request-size pull-request-size bot added size/M and removed size/L labels Jul 20, 2020
@pull-request-size pull-request-size bot added size/L and removed size/M labels Jul 20, 2020
@plainheart plainheart added this to the 5.0.0 milestone Jul 31, 2020
@plainheart plainheart marked this pull request as ready for review August 9, 2020 02:37
src/model/mixin/itemStyle.ts Outdated Show resolved Hide resolved
@pissang pissang merged commit a6300c9 into apache:next Aug 28, 2020
@echarts-bot
Copy link

echarts-bot bot commented Aug 28, 2020

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

@pissang pissang changed the title feat: more configurable styles for lineStyle & borderStyle. [5.0] [FEATURE] more configurable styles for lineStyle & borderStyle. Oct 27, 2020
@echarts-bot echarts-bot bot added the PR: awaiting doc Document changes is required for this PR. label Oct 27, 2020
@plainheart plainheart deleted the more-line-type branch March 1, 2021 09:07
@plainheart plainheart removed the PR: awaiting doc Document changes is required for this PR. label Mar 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants