Screenshot annotator improvements

[lassoan]

• [x] Aspect ratios of icons in the annotator were not correct
• [x] Add title to the annotator window (now it is just "Slicer")
• [x] There are cropping issues when using two monitors
• [ ] Click position is sometimes not computed correctly
• [x] Need to be able to add text to click markers
• [x] Need to highlight the widgets as the mouse moves on the screen
• [x] At the same position there are several widgets (button or a parent frame, etc.), therefore we need a way to select which one we want the annotation to be anchored to
• [x] Runtime error:

  Traceback (most recent call last):
  File "/Users/pieper/slicer/latest/TutorialMaker/TutorialMaker/Lib/TutorialGUI.py", line 100, in eventFilter
  self.mouse_press_event(event)
  File "/Users/pieper/slicer/latest/TutorialMaker/TutorialMaker/Lib/TutorialGUI.py", line 441, in mouse_press_event
  self.calculate_annotation()
  File "/Users/pieper/slicer/latest/TutorialMaker/TutorialMaker/Lib/TutorialGUI.py", line 480, in calculate_annotation
  x_i, y_i = wdgts_child["position"]
  TypeError: 'NoneType' object is not subscriptable

[DouSam]

@lassoan I think not only the ratios of the icons but the images too. When the screenshot is taken, its resolution is the Slicer resolution. So, on my computer will take an ultrawide screenshot, and in my notebook a 4k screenshot. I think all the screenshots should be sized to 1080p for example.

[vmms]

@lassoan @LuskasHusty @enriquehdez98 I can the latest version on an iMac computer that was not used for development and found that when taking the screenshot the entire screen is taken into account, not just the Slicer window. For example, if Slicer is not maximized you can see that the desktop is captured in the background.

But if it is maximized you can still see that the Apple taskbar is captured.

This causes problems regarding the position of the widgets, since since they are being stored in the JSON, the relative position is captured with respect to the Slicer window and not to the position of the screen.

Even in the second image where Slicer is maximized, the annotations are out of focus due to the taskbar.

[LuskasHusty]

@vmms That is a issue with the screenshot capture tool then. I'ill look into it, until then please confirm that you are using the latest version.

[vmms]

The versions I used are:

• Slicer 5.7.0
• TutorialMaker 31-01-2024 Then, on another Mac with an extended screen the same problem happens, it even combines the two monitors

[DouSam]

The versions I used are:

• Slicer 5.7.0
• TutorialMaker 31-01-2024 Then, on another Mac with an extended screen the same problem happens, it even combines the two monitors

@vmms clone the extension again. One of Lucas's last commits has solved this problem!

[vmms]

@lassoan The point: Need to be able to add text to click markers. Arrows have text, but do they need text in all annotations?

[lassoan]

Most of the time arrows will have text, but if we only want to highlight one thing in the image then it is possible that there is no text.

[vmms]

I modified the code so that when there is no text in the input field the box is not painted, but when there is text it is painted along with a box.

[DouSam]

Good. Dont forget to make a commit with these changes. You can create a new branch using the updated version of the repository.

[vmms]

Now when you change the color of the arrow, the color of the accompanying text box also changes. The default color was orange. Also, I increased the limit of characters that can be written, and added a function to make the box larger and fit the text.