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(component): support slider without interaction #4413

Merged
merged 7 commits into from
Dec 8, 2022
Merged

feat(component): support slider without interaction #4413

merged 7 commits into from
Dec 8, 2022

Conversation

visiky
Copy link
Member

@visiky visiky commented Dec 6, 2022

  • 接入 slider
  • slider 文档
  • Chart API 增加 slider
  • sliderFilter 下一个 PR

详情见: #4396

image

@Aarebecca
Copy link
Contributor

文档我晚上来补吧

@visiky
Copy link
Member Author

visiky commented Dec 6, 2022

文档我晚上来补吧

discussion中已经把文档写好了,只缺配置项部分而已

src/runtime/component.ts Outdated Show resolved Hide resolved
Copy link
Member

@pearmini pearmini left a comment

Choose a reason for hiding this comment

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

这个 PR 只是在一种情况下把 slider 展示出来了,还有很多问题需要考虑:

  • slider 受 transpose 的坐标系影响?
  • view 里面 slider 的传递?
  • slider 的同步问题?
  • 分面里面的 slider 的传递?

不过这些问题暂时可以忽略,先满足需求就可以。

src/runtime/component.ts Outdated Show resolved Hide resolved
__tests__/integration/charts/aapl-line-slider.ts Outdated Show resolved Hide resolved
src/runtime/component.ts Outdated Show resolved Hide resolved
src/runtime/component.ts Outdated Show resolved Hide resolved
@visiky
Copy link
Member Author

visiky commented Dec 6, 2022

  • view 里面 slider 的传递?
  • slider 的同步问题?
  • 分面里面的 slider 的传递?

✅ transpose 下的 slider

待办:

  • view 里面 slider 的传递?
  • slider 的同步问题?
  • 分面里面的 slider 的传递?
  • slider里边的迷你趋势图

@pearmini
Copy link
Member

pearmini commented Dec 7, 2022

还有一个问题,就是 Chart API 也需要更新一下?

Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

There are accessibility issues in these changes.

site/docs/api/component/slider.zh.md Outdated Show resolved Hide resolved
site/docs/api/component/slider.zh.md Outdated Show resolved Hide resolved
Copy link
Member

@pearmini pearmini left a comment

Choose a reason for hiding this comment

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

  • api 增加了 mark.slider,同时增加一下单测
  • 处理 一下 accesslint 的问题

@visiky visiky merged commit 9140d13 into v5 Dec 8, 2022
@visiky visiky deleted the feat/slider branch December 8, 2022 03:39
@pearmini pearmini mentioned this pull request Dec 15, 2022
hustcc pushed a commit that referenced this pull request May 16, 2023
* feat(component): support slider without interaction

* fix(slider): code review suggestions

* feat(slider): support transposed slider

* feat: add slider api

* feat(slider): support band scale & add slider docs

* fix: lint

* test: add slider chart api test case
hustcc pushed a commit that referenced this pull request May 16, 2023
* feat(component): support slider without interaction

* fix(slider): code review suggestions

* feat(slider): support transposed slider

* feat: add slider api

* feat(slider): support band scale & add slider docs

* fix: lint

* test: add slider chart api test case
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

4 participants