yuzhenmi
commented
10 years ago Hmm..
Tooltip -Background is already 97% opaque, either your browser isn't supporting it or it's too hard to notice. -Added 5px padding. -Do you want me to move the entire tool tip up by 20 pixels? Like 20 pixels above the mouse position? The little triangle will not be pointing at the right place then... Note that our green tooltip is also aligned vertically with the DOM element, when popping from the left or right. -I really don't feel the need to draw a dot and connect the tooltip to the dot. The mouse position should be an obvious indicator of where the tooltip is pointing. Also, since the mouse position is not necessarily positioned strategically on the view object, I don't see any point in indicating that position with a dot (it would just be a dot on some random place on the view object). It would also take A LOT more work and extra code maintenance to draw that line than to add the little triangle too (the triangle is pure HTML/CSS, but it might not suffice for the line and circle)! -Actually, the white circles is not my doing. SUBA3 is offline right now (for the second time I've seen). It's been offline for hours. It's also crashing some of the other features on ePlant because there seems to be too many Ajax requests waiting for SUBA3's response. I think the colored circles are important for indicating localization. The stroke color only indicates whether mouse is hovering over it.
GOF label -Adding background to the label is technically undoable in Cytoscape.js. However, they do support outlining the label, which I think is even better. Check it out! If you like it then I'll see if I can duplicate it for other views...
Crashing -Again, due to SUBA3 being offline and Ajax going crazy. In fact, after you try to load a gene, no Ajax request will work because it is freezing on all the SUBA3 requests. I can't even get a response from SUBA3's website! http://suba.plantenergy.uwa.edu.au/
Sticky popup -This is caused by a modification I made yesterday. I tried to prevent Cytoscape from picking up mouse events when mouse is over the popup, but it seems to be backfiring by causing the popup to become glitchy. I recovered it so it should no longer be a problem.
Gene list dialog -Clicking on "Get Data" now removes the gene list dialog.
jamiewaese
CYTOSCAPE TOOL TIPS Love those Cytoscape tool tips! Can you move them up ~20 pixels? They shouldn't overlap the circle we're looking at. Also add some padding around the text and a 97% opaque background.
Can you draw a line from the middle of the box to mouseX, mouseY, terminating with a little dot? This will connect the popup box to the line.
By the way, I like the white circles with color on the strokes instead. Much cleaner. Nice!
CYTOSCAPE G.O.F. LABEL I can't read the center Gene Of Focus label. The green isn't contrasty enough. How about putting a green rectangle behind black text, sort of like what the title bar in the annotation popups look like? In fact, I was also going to suggest the same thing on the Chromosome View. You still can't easily see which is the GOF, even with the larger font and the underline. There isn't enough contrast between the GOF and the unselected labels. So... the green rectangle background might do it. Think of it like a highlighter pen. It would be nice to have the same convention across views too.
Hey, did you know there's a crash if you're in Cytoscape view and you try to get data for another gene?
CHROMOSOME VIEW ANNOTATION POPUPS Incidentally, I like the 4-way cursor icon when you're moving the annotation popups around the screen. But the move function is a bit sticky, no? Every once in a while it doesn't register the mouseRelease event, and the box follows my mouse around the screen. You have to click to release it again.
CHROMOSOME VIEW GENE LIST BOXES And finally, speaking of sticky boxes... on the Chromosome View, the gene list boxes that appear next to the chromosomes have a habit of sticking around too long. They should be told to go away if the user clicks 'get data' in the annotation popup.