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

feat: support polygon crosshair for angleAxis in polar coordinate #3604

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

xiaoluoHe
Copy link
Contributor

[中文版模板 / Chinese template]

🤔 This is a ...

  • New feature
  • Bug fix
  • TypeScript definition update
  • Bundle size optimization
  • Performance optimization
  • Enhancement feature
  • Refactoring
  • Update dependency
  • Code style optimization
  • Test Case
  • Branch merge
  • Release
  • Site / documentation update
  • Demo update
  • Workflow
  • Other (about what?)

🔗 Related issue link

🔗 Related PR link

🐞 Bugserver case id

💡 Background and solution

📝 Changelog

Language Changelog
🇺🇸 English
🇨🇳 Chinese

☑️ Self-Check before Merge

⚠️ Please check all items below before requesting a reviewing. ⚠️

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • TypeScript definition is updated/provided or not needed
  • Changelog is provided or not needed

🚀 Summary

copilot:summary

🔍 Walkthrough

copilot:walkthrough

@xiaoluoHe xiaoluoHe modified the milestones: 1.13.3, 1.13.4 Dec 25, 2024
@xiaoluoHe xiaoluoHe changed the title [WIP] feat: support polygon crosshair for angleAxis in polar coordinate feat: support polygon crosshair for angleAxis in polar coordinate Jan 20, 2025
@xiaoluoHe xiaoluoHe modified the milestones: 1.13.4, 1.13.5 Jan 21, 2025
@@ -16,7 +16,8 @@ Only effective for `type: 'line'`, `width` represents the width of the auxiliary

#${prefix} smooth(boolean)

Only effective for `type: 'line'`, whether to draw smoothly under the polar coordinate system or not.
Whether to draw smoothly under the polar coordinate system or not.
Effective for `type: 'polygon'` since version `1.13.3`
Copy link
Contributor

Choose a reason for hiding this comment

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

版本需要更新一下

@@ -32,8 +38,8 @@ export class PolarCrossHair<T extends IPolarCrosshairSpec = IPolarCrosshairSpec>
private _currValueAngle: AxisCurrentValueMap;
private _currValueRadius: AxisCurrentValueMap;

private _angleHair: IHair | undefined;
private _radiusHair: IHairRadius | undefined;
private _angleHair: IPolarHair | undefined;
Copy link
Contributor

Choose a reason for hiding this comment

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

crosshair有更新,这里也需要更新一下

@xiaoluoHe xiaoluoHe force-pushed the feat/polar-polyon-crosshair branch from 59c296a to f80d29a Compare January 26, 2025 07:46
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.

2 participants