Skip to content

feat(sector): enhance sector corner radius to support configuring any one corner.#865

Merged
pissang merged 16 commits into
nextfrom
enhance-sector-corner-radius
Jan 7, 2022
Merged

feat(sector): enhance sector corner radius to support configuring any one corner.#865
pissang merged 16 commits into
nextfrom
enhance-sector-corner-radius

Conversation

@plainheart
Copy link
Copy Markdown
Collaborator

The corner radius for the sector has been supported since v5, but it was limited to a pair of corners, either an inner pair or an outer pair. I can remember @Ovilia asked about it at that time.

This PR enhances this feature so that we can configure the radius of four corners at will. As shown below, the sector has four different corner radius.

A related test case is sector.html.


Moreover, in commit 53819e9, I tweaked the watch script to process lib extension immediately after the source was compiled.

Comment thread src/graphic/helper/roundSector.ts Outdated
Comment thread src/graphic/helper/roundSector.ts Outdated
Copy link
Copy Markdown
Member

@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.

So glad to see this PR. Two minor comments which I am not very sure of. @pissang Please have a double check.

Comment thread src/graphic/helper/roundSector.ts
Comment thread src/graphic/shape/Sector.ts
Comment thread src/graphic/helper/roundSector.ts Outdated
Comment thread src/graphic/helper/roundSector.ts
@pissang pissang merged commit db0029f into next Jan 7, 2022
@pissang pissang deleted the enhance-sector-corner-radius branch January 7, 2022 08:29
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.

3 participants