Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Adding Cluster HoverData#7297

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
BSd3v wants to merge1 commit intoplotly:master
base:master
Choose a base branch
Loading
fromBSd3v:adding-cluster-hoverdata

Conversation

@BSd3v
Copy link

Adding hover and click data to clusters for maps.

It currently only emits the plotly_hover and plotly_click events, instead of providing the hoverData directly on screen, automatically.

Looking for feedback and assistance with how to get thehoverData to show.

@alexcjohnson

AnnMarieW, archmoj, and ndrezn reacted with heart emojipip-install-python reacted with rocket emoji
@gvwilsongvwilson added featuresomething new communitycommunity contribution P2considered for next cycle labelsDec 4, 2024
@archmoj
Copy link
Contributor

Thanks very much for the PR.
Please adddraftlogs/7297_fix.md for this PR as describedhere.

Please add a jasmine test intest/jasmine/tests/map_test.js for this feature as well.

Wondering if this fix may also be applied tomapbox?

@BSd3v
Copy link
Author

@archmoj,

I believe that this can be applied tomapbox as well. From my understanding, it seems thatmap is an offshoot ofmapbox.

@archmoj
Copy link
Contributor

@archmoj,

I believe that this can be applied tomapbox as well. From my understanding, it seems thatmap is an offshoot ofmapbox.

That's true.
Although themapbox traces would be removed in plotly.js v4, having this change formapbox would be nice IMHO. cc:@gvwilson@ndrezn

BTW Let's start by adding the tests tomap first.
Thank you@BSd3v

@archmoj
Copy link
Contributor

Update: There is no need to add this feature tomapbox which is deprecated.
@BSd3v Would you please fetch upstream master and merge it to this branch.
Then please add a jasmine test to test the hover.
Thank you.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

@BSd3vBSd3v

Labels

communitycommunity contributionfeaturesomething newP2considered for next cycle

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@BSd3v@archmoj@gvwilson
[8]ページ先頭
©2009-2025 Movatter.jp