Skip to content
This repository has been archived by the owner on Jul 29, 2019. It is now read-only.

Added mouseover and mouseout events to the time bar. This currently wor... #773

Closed
wants to merge 5 commits into from

Conversation

andyflatt
Copy link

...ks with RangeItems only.

@josdejong
Copy link
Contributor

Thanks @andyflatt!

A few remarks:

  • It would be great if you could implement it for each of the item types.
  • Also, it will be better performance wise to only attach a mouseover and mouseout event listener to the ItemSet rather than to each individual item (event delegation).
  • Can you please only commit the changes in the source, and not in the generated files in /dist?

@andyflatt
Copy link
Author

Ok, I made these changes for a demo last week so I was aware that they are not complete. Ill try and find time to correct them in the next fortnight.

@andyflatt
Copy link
Author

I have implemented this support for box and point

Because of the difficulties of event delegation on mouseover, I can not see how I can use event delegation from the itemset. Do you have any suggestions?

For the last commit I did not commit the generated files.

@josdejong josdejong closed this May 22, 2015
@andyflatt
Copy link
Author

@josdejong You closed the issue without merging in the changes. I implemented what you suggested and asked some questions. How come you do not want this hover support?

@josdejong josdejong reopened this May 26, 2015
@josdejong
Copy link
Contributor

Maybe this happened because I deleted the develop branch and replaced it with the (released) v4 branch. I've reopened this issue. I think I overlooked your open PR, sorry. We should get this merged into the code again.

@andyflatt
Copy link
Author

Does this code need any more work before it can be merged?

@josdejong
Copy link
Contributor

@andyflatt I think your latest version was good (except for no need to commit changes in /dist). Would be great if you could redo your PR, sorry for the inconvenience.

@The-Dub
Copy link

The-Dub commented Sep 10, 2015

Hi,
I was trying to find if there was a mouseover event in the vis.timeline library.
Did you do implemented that functionality ?

Great library btw, thanks !

@AlexDM0
Copy link
Contributor

AlexDM0 commented Sep 10, 2015

Hi,

What do you need these event for? We have not had a usecase that requires these events.

Cheers

On 10 sep. 2015, at 18:03, The-Dub notifications@github.com wrote:

Hi,
I was trying to find if there was a mouseover event in the vis.timeline library.
Did you do implemented that functionality ?

Great library btw, thanks !


Reply to this email directly or view it on GitHub.

@The-Dub
Copy link

The-Dub commented Sep 10, 2015

I created an interactive timeline with images, and tooltips attached to those images are created on-the-fly. I needed to grab the date (or the item #) to do some matching behind to show the right information.

I worked my way around with mouseover inside the timeline triggering a click event, and then grabbing the info from that (simplest way I could find).

So not a big deal, if it's a unique request, don't bother implementing it, but I think it could be use for interactivity outside the timeline.

thanks for the answer ! ;)

@andyflatt
Copy link
Author

I remember seeing this feature on the wishlist on the vis website. This merge request allows for 'mouseover' and 'mouseout' events in the timeline. I implemented this because I was using the timeline in conjunction with a the map and wanted to highlight things on the map on mouse over.

Do I need to redo the PR? What should I do differently on a new PR? Is it to just not include the dist folder?

@The-Dub Please pull the develop branch of https://github.com/andyflatt/vis to see if it satisfies you requirement.

@The-Dub
Copy link

The-Dub commented Sep 11, 2015

I'll take a look at that today.
Thanks !

@andyflatt
Copy link
Author

Created #1367 for the new pull request

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants