rega-cev / phylogeotool

4 stars 3 forks source link

Enhance 'Question Mark' #18

Closed Ewout1988 closed 7 years ago

Ewout1988 commented 7 years ago

Users don't understand that the question mark on the bottom has a hover over effect. It would be better if there was a click action connected to it that would show the same tooltip

@plibin Can you post a part of code that we can put in a click listener that would cause the tooltip to appear?

plibin commented 7 years ago

the clickable component is a WImage wImage;

wImage.mouseClicked().addListener(this, new Signal.Listener() { public void trigger() { //code dat moet worden uitgevoerd } }); wImage.mouseWentOver().addListener(this, new Signal.Listener() { public void trigger() { //code dat moet worden uitgevoerd } });

GuyBaele commented 7 years ago

I does take too long for the popup to appear. Can this time be reduced somehow? Clicking on it may indeed also provide some sort of action.

plibin commented 7 years ago

I’m afraid that mouseHover speed is browser-dependent. Prolly there exists some javascript to make this faster?

Ewout1988 commented 7 years ago

When people click on the 'question mark', we now show a dialog like this:

screen shot 2017-04-26 at 13 21 05
ktheyss commented 7 years ago

Something like this is sufficient for me "This number indicates the current level of the tree shown. Use the webbrowser back/forward buttons to navigate to previous levels and back."

Ewout1988 commented 7 years ago

closeicons-dialog.png could not be found -> fix

GuyBaele commented 7 years ago

@Ewout1988 What does this last message mean?

Ewout1988 commented 7 years ago

There was a problem when users opened the 'Question Mark' popup dialog window, that they could not see the closing cross in the right upper corner of the dialog window due to an image that was missing in the .war This should be fixed in the meantime. On top of that, I think I should remove the message "Use the home button (to the right of the indicated level) to go back to the beginning" (like @ktheyss mentioned before).

screen shot 2017-04-27 at 12 39 08
GuyBaele commented 7 years ago

I agree, remove that message and we're good.

Ewout1988 commented 7 years ago

Please check and close if good