Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #9742 (closed)

Opened 7 years ago

Closed 6 years ago

Last modified 6 years ago

BUG Show text box in Measuring Tools window (ROI)

Reported by: pwalczysko Owned by: jburel
Priority: major Milestone: 5.1.0
Component: Insight Version: 4.4.9
Keywords: TAB Cc: ux@…, m.porter@…, jamoore, ajpatterson
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description

Downloaded OMERO. Insight 4.4.4 from OMERO client 4.4.4 downloads on the OME webpage.
Logged in as user-3. Opened "free edge 6.lif" and in full viewer and opened the Measuring Tool window. Applied various regions of interest as well as text onto the image. Ticked and untucked the box "Show Text" with no effect onto the text which was just put into the image.

Change History (11)

comment:1 Changed 7 years ago by jburel

  • Sprint changed from 2012-10-23 (1) to 2012-11-20 (2)

Moved from sprint 2012-10-23 (1)

comment:2 Changed 7 years ago by pwalczysko

At the time of the creation of the ticket did not understand the functionality fully.
The functionality is actually OK.
The confusion arose because of not being clear that the "Text" in "Show Text" is actually the region-commentary text.
Thought that text in "Show Text box means actually the Text as a Region (choose from the ROI tools just like circle, square, line).
Suggestion: Describe the "Show Text" tickbox as "Show region commentary" or "Region comments" or similar to make this more clear -
not sure though how to make the box description brief enough to be practicable.

comment:3 Changed 7 years ago by rleigh

Maybe rename the "Text" field to "Comment", and the "Show Text" field to "Show comment".

More as a general ROI model point, it might be worth having the type-agnostic "comment" field not share the "text" ROI type "text" field, since one is a generic ROI property, and one is a specific property of a specialised ROI type.

comment:4 Changed 7 years ago by jburel

  • Sprint changed from 2012-11-20 (2) to 2012-12-18 (3)

Moved from sprint 2012-11-20 (2)

comment:5 Changed 7 years ago by jburel

  • Cc ux@… m.porter@… added; omero-team@… removed

from #9506

http://qa.openmicroscopy.org.uk/qa/feedback/4598/

Comment: The 'show text' tick box in the measurement tool has some funny behaviours. If you have it un-ticked and you give an ROI some text, that text appears on the screen until the mouse pointer passes over the tick box. Also, if you save your ROIs then un-tick it and close the image you are prompted to save the ROIs again, but the next time you open that image the text will be displayed.

comment:6 Changed 7 years ago by jburel

  • Sprint 2012-12-18 (3) deleted

comment:7 Changed 6 years ago by jamoore

  • Keywords TAB added
  • Version set to 4.4.9

comment:8 Changed 6 years ago by jburel

  • Cc jamoore ajpatterson added

There is no way in the current model/DB to indicate if the text is visible or not.
The logic is the following if the text is not blank the text is visible. We will have to modify if we wish to save the value.
Other option is to allow to turn text on/off and not to prompt a dialog to save the ROI.

Last edited 6 years ago by jburel (previous) (diff)

comment:9 Changed 6 years ago by jburel

I have decided to adjust the code so the dialog does not show up with the user only show/hide comment on a ROI.
This can be easily modified if we decide to modify to change model.

comment:10 Changed 6 years ago by jburel

  • Resolution set to fixed
  • Status changed from new to closed

comment:11 Changed 6 years ago by jburel

  • Milestone changed from 5.x to 5.1.0
Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.80250 sec.)

We're Hiring!