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

Refactor/chart view #38

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Refactor/chart view #38

wants to merge 1 commit into from

Conversation

zChanges
Copy link
Collaborator

No description provided.

@changeset-bot
Copy link

changeset-bot bot commented Dec 12, 2022

⚠️ No Changeset found

Latest commit: 64b2530

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Contributor

github-actions bot commented Dec 12, 2022

Deploy preview for alauda-chart ready!

✅ Preview
https://alauda-chart-8z55fnovl-jounqin.vercel.app

Built with commit 64b2530.
This pull request is being automatically deployed with vercel-action

@zChanges zChanges force-pushed the refactor/chart-view branch 3 times, most recently from 12d64d9 to 31b683e Compare December 13, 2022 06:02
@JounQin JounQin changed the base branch from next to main December 8, 2023 06:34
if (seriesIdx === 1) {
hRect = null;

const cx = u.cursor.left * pxRatio;

Check warning

Code scanning / CodeQL

Implicit operand conversion Warning

This expression will be implicitly converted from undefined to number.
hRect = null;

const cx = u.cursor.left * pxRatio;
const cy = u.cursor.top * pxRatio;

Check warning

Code scanning / CodeQL

Implicit operand conversion Warning

This expression will be implicitly converted from undefined to number.
auto: notAuto,
range: (_u: uPlot, dataMin: number, dataMax: number) => {
const minV = min
? Math.max(dataMin, min || 0)

Check warning

Code scanning / CodeQL

Useless conditional Warning

This use of variable 'min' always evaluates to true.
range: (_u: uPlot, dataMin: number, dataMax: number) => {
const minV = min
? Math.max(dataMin, min || 0)
: Math.min(dataMin, min || 0);

Check warning

Code scanning / CodeQL

Useless conditional Warning

This use of variable 'min' always evaluates to false.
if (!titleFormatter) {
this.title.text(context.title as string);
return;
let tpl: string = title || NOT_AVAILABLE;

Check warning

Code scanning / CodeQL

Useless conditional Warning

This use of variable 'title' always evaluates to true.
@@ -6,3 +6,4 @@ lib
.vercel
.*cache
*.tsbuildinfo
.git
Copy link
Member

Choose a reason for hiding this comment

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

revert

@@ -6,3 +6,4 @@ LICENSE
*.sh
*.tsbuildinfo
*.lock
docs
Copy link
Member

Choose a reason for hiding this comment

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

revert

]
}
]
"selector-type-no-unknown": null,
Copy link
Member

Choose a reason for hiding this comment

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

revert

@@ -1,7 +1,9 @@
{
"extends": "@1stg/eslint-config/loose",
"rules": {
"regexp/strict": "off"
"regexp/strict": "off",
Copy link
Member

Choose a reason for hiding this comment

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

revert

# reactive

> 为了支持配置式 options 更新,图表动态响应更新
> reactive 由 on-change 库完成
Copy link
Member

Choose a reason for hiding this comment

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

底层换成 @vue/reactivity

@@ -65,6 +63,12 @@
"optional": true
}
},
"dependencies": {
"aphrodite": "^2.4.0",
"on-change": "^4.0.2",
Copy link
Member

Choose a reason for hiding this comment

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

换成 @vue/reactivity

"cache": true,
"detail": true,
"ignoreAsAssertion": true,
"ignoreNested": true,
"ignoreNonNullAssertion": true,
"showRelativePath": true,
"strict": true,
"update": true
"update": true,
"ignoreFiles": [
Copy link
Member

Choose a reason for hiding this comment

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

revert

@JounQin
Copy link
Member

JounQin commented Dec 8, 2023

本地先把 lint 修复完再提交

@@ -65,6 +63,12 @@
"optional": true
}
},
"dependencies": {
"aphrodite": "^2.4.0",
Copy link
Member

Choose a reason for hiding this comment

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

为啥用这个?我看都不维护了,建议用纯 scss 就行了

"dependencies": {
"aphrodite": "^2.4.0",
"on-change": "^4.0.2",
"placement.js": "^1.0.0-beta.5",
Copy link
Member

@JounQin JounQin Dec 8, 2023

Choose a reason for hiding this comment

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

这个也过时了,不要用,试试 https://popper.js.org

"aphrodite": "^2.4.0",
"on-change": "^4.0.2",
"placement.js": "^1.0.0-beta.5",
"uplot": "^1.6.26"
Copy link
Member

Choose a reason for hiding this comment

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

好奇为啥要有两套图表引擎?

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.

2 participants