Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MT textbox #1988

Merged
merged 4 commits into from Jan 24, 2014
Merged

MT textbox #1988

merged 4 commits into from Jan 24, 2014

Conversation

jburel
Copy link
Member

@jburel jburel commented Jan 16, 2014

Do not show a dialog when the user only shows or hides comment on a shape
To avoid confusion between a text shape and a comment on a shape, I have renamed the text on a shape to
comment.
See https://trac.openmicroscopy.org.uk/ome/ticket/9742

To test:

  • Open an image
  • Open the measurement tool (MT). Make the text in toolbar is "Show Comment"
  • Draw few shapes, add text to some of them. Save
  • Select a shape, Go to inspector Tab, Make sure the name of the column is "Comment"
  • Close MT and Viewer
  • Open Viewer/MT
  • Text should be displayed on shape.
  • Un-select "Show Comment". The text on shape should no longer be displayed. Do not perform other actions.
  • Close MT. Close Viewer. A Dialog indicating if you want to save ROI should NOT pop up.
  • Open Viewer, open MT. Move a shape
  • Close MT. Close Viewer. A Dialog indicating if you want to save ROI should pop up.

This is TAB, so PR should be rebased to 5.0.0

@pwalczysko
Copy link
Member

Works as expected in the sense of the above workflow. Checked the ticket as well and it solves the ticket okay.

The other point (possibly for another PR) which was apparent when testing this was the fact that you cannot see what you write as you are writing the comment (the text is (nearly) invisible, because there is no background in the text box). On the other hand, once you finished writing, the whole box gets a very wide background, which obscures the middle of the shape, thus interfering with/hiding the biological feature which is inside the ROI in the first place. - this was remarked on by M Porter on some ticket already, just was reminded now how true that is.
I see that this is the idea behind the "Show comment" button in the first place, but still, I think there could be some improvements (came up also as an important feature during the OMERO.workshop - people want to mark their ROIs and see the markings & the ROIs & the image at the same time).

The other remark is - note that when the ROI was moved as instructed in the workflow, an unpleasant smudge was left "leaking" from the comment box (see screenshot), again obscuring wide area (see screenshot).

screen shot 2014-01-17 at 12 22 26

@jburel
Copy link
Member Author

jburel commented Jan 17, 2014

@pwalczysko: Thanks for the report.
I will prefer to do that in another PR. The MT needs to be totally rewritten.
Could you create a ticket?

@pwalczysko
Copy link
Member

@joshmoore
Copy link
Member

👍

joshmoore added a commit that referenced this pull request Jan 24, 2014
@joshmoore joshmoore merged commit 00e2d12 into ome:develop Jan 24, 2014
@jburel jburel mentioned this pull request Jan 24, 2014
@jburel
Copy link
Member Author

jburel commented Jan 24, 2014

--rebased-to #2023

@jburel jburel deleted the mt-textbox branch May 2, 2014 12:41
@sbesson sbesson modified the milestones: 5.1.0, 5.1.0-m1 Oct 14, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants