fzaninotto / DependencyWheel

A package dependency visualization using d3.js. Currently supports Composer for PHP.
http://www.redotheweb.com/DependencyWheel/
MIT License
263 stars 154 forks source link

Add click action to highlight instead of only mouse over #7

Closed grillorafael closed 8 years ago

grillorafael commented 8 years ago

When the dependency wheel is too big it's hard to keep the mouse over the element.

It would be good to add this in text click/hover as well.

What do you think?

fzaninotto commented 8 years ago

If it's hard to hover, it's also hard to click, no? Would you have an example of "too big" wheel?

grillorafael commented 8 years ago

In that case the label click would do the job. I'll add an image here in a bit. Just woke up hahaha

On Monday, September 14, 2015, Francois Zaninotto notifications@github.com wrote:

If it's hard to hover, it's also hard to click, no? Would you have an example of "too big" wheel?

— Reply to this email directly or view it on GitHub https://github.com/fzaninotto/DependencyWheel/issues/7#issuecomment-140046024 .

Rafael Grillo Abreu +55 21 99461-1339 LinkedIn https://www.linkedin.com/profile/view?id=89764578 Developer

grillorafael commented 8 years ago

screencapture-localhost-3000-1442159440798

grillorafael commented 8 years ago

Those tiny elements are very hard to hover.

In fact this are two things instead of just one.

If we bind to click element/label we can add scroll to the box where the dependency wheel is in and then we can scroll without losing focus of what we want.

And it will be easier to focus elements by clicking on their labels.

fzaninotto commented 8 years ago

good idea, the labels should be hoverable, too.

fzaninotto commented 8 years ago

Fixed by #9

grillorafael commented 8 years ago

Good! Thanks @katrotz @fzaninotto