search

monarch-initiative / phenogrid

The phenogrid widget
13 stars 14 forks source link

Remove "collapse" HPO treeview, leave it expanded once built #147

Closed yuanzhou closed 9 years ago

yuanzhou commented 9 years ago

I realized that there's no need to show the text of "collapse xxx", we should just display the classification hierarchy once it's expanded. Because once the treeview is displayed, it's cached, we benefit nothing from collapsing the treeview in the tooltip. And once it's collapses, if we move back to the same phenotype, we still need to click to expand the cached treeview again. Why not just let the treeview stay there once it's expanded?

capture

capture1

yuanzhou commented 9 years ago

FYI @davism84, I've removed the collapse HPO related things from phenogrid.js and tooltiprender.js.

yuanzhou commented 9 years ago

FYI @davism84 I've also changed the clickable icon to clickable text (styled it with underline and red color to make it eye-catchy and look like an link)

capture

Once you have it expanded, move mouse over again will show the treeview directly:

capture

harryhoch commented 9 years ago

@yuanzhou , looks good. thanks