Skip to content

feat(graph): new thumbnail component #17471

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 28 commits into from
Oct 31, 2022
Merged

feat(graph): new thumbnail component #17471

merged 28 commits into from
Oct 31, 2022
+915 −13

Conversation

Lruler
Copy link
Contributor

@Lruler Lruler commented Aug 4, 2022 •
edited
Loading

Brief Information

This pull request is in the type of:

  • bug fixing
  • new feature
  • others

What does this PR do?

add thumbnail component, it looks like

Fixed issues

#7521& #11352

Details

we could provide a thumbnail, we can give users the intuitive about the overall shapes of the whole series and the current visible area.
image
image

Before: What was the problem?

For series like maps, graphs, trees, sankeys, data shapes sometimes may be too large to displayed without overlap in a chart. Some charts supports roam (not sankey) but the user may not have an idea about the relationship between the current visible data and the overall data

Document Info

One of the following should be checked.

  • This PR doesn't relate to document changes
  • The document should be updated later
  • The document changes have been made in apache/echarts-doc#xxx

Misc

ZRender Changes

  • This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs

N.A.

Others

Merging options

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

Other information

@echarts-bot
Copy link

echarts-bot bot commented Aug 4, 2022

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.

⚠️ MISSING DOCUMENT INFO: Please make sure one of the document options are checked in this PR's description. Search "Document Info" in the description of this PR. This should be done either by the author or the reviewers of the PR.

@plainheart plainheart marked this pull request as draft August 4, 2022 08:28
@Ovilia
Copy link
Contributor

Ovilia commented Aug 4, 2022

Thanks for your contribution. I opened a related discussion at #17432. You are welcomed to do the thumbnail part and I will guide you along with @100pah . I will be responsible to make the roaming experience better.

@echarts-bot echarts-bot bot added PR: doc unchanged PR: awaiting doc Document changes is required for this PR. and removed PR: doc unchanged labels Aug 5, 2022
@echarts-bot
Copy link

echarts-bot bot commented Aug 5, 2022

Document changes are required in this PR. Please also make a PR to apache/echarts-doc for document changes and update the issue id in the PR description. When the doc PR is merged, the maintainers will remove the PR: awaiting doc label.

@echarts-bot echarts-bot bot added PR: doc unchanged and removed PR: awaiting doc Document changes is required for this PR. labels Sep 5, 2022
@Lruler Lruler marked this pull request as ready for review September 18, 2022 07:16
@Lruler Lruler mentioned this pull request Sep 19, 2022
Closed
@Ovilia Ovilia added this to the 5.4.1 milestone Sep 23, 2022
@pissang pissang modified the milestones: 5.4.1, 5.5.0 Oct 15, 2022
@Lruler Lruler changed the base branch from next to next-funnel October 31, 2022 02:05
@Lruler Lruler changed the base branch from next-funnel to next-thumbnail October 31, 2022 05:58
@pissang pissang changed the base branch from next-thumbnail to next October 31, 2022 13:51
@pissang pissang changed the base branch from next to next-thumbnail October 31, 2022 13:51
100pah
100pah approved these changes Oct 31, 2022
View reviewed changes
@100pah 100pah merged commit c2eb34a into apache:next-thumbnail Oct 31, 2022
@echarts-bot
Copy link

echarts-bot bot commented Oct 31, 2022

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

@plainheart plainheart added PR: awaiting doc Document changes is required for this PR. PR: doc ready and removed PR: doc unchanged PR: awaiting doc Document changes is required for this PR. labels Oct 31, 2022
@plainheart plainheart mentioned this pull request Jun 25, 2023
Merged
3 tasks
@Ovilia Ovilia mentioned this pull request Sep 22, 2023
Open
@Ovilia Ovilia modified the milestones: 5.5.0, 6.0.0 Jan 30, 2024
@Ovilia Ovilia mentioned this pull request Feb 4, 2024
Open
@Ovilia Ovilia mentioned this pull request Jul 22, 2024
Open
@kingyue737 kingyue737 mentioned this pull request Apr 21, 2025
Closed
9 tasks
@Ovilia Ovilia changed the title [feat]: add thumbnails components feat: thumbnail component Jun 21, 2025
@Ovilia Ovilia changed the title feat: thumbnail component feat(graph): new thumbnail component Jun 21, 2025
@100pah 100pah mentioned this pull request Jun 27, 2025
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pissang pissang pissang requested changes

@100pah 100pah 100pah approved these changes

@plainheart plainheart Awaiting requested review from plainheart

Assignees
No one assigned
Labels
PR: doc unchanged PR: first-time contributor size/L
Projects
ECharts 6.0
Status: Done
Milestone
6.0.0
Development

Successfully merging this pull request may close these issues.
5 participants
@Lruler @Ovilia @100pah @pissang @plainheart