search

dboun / MultiPatientImageProcessing

Bulk processing of medical images
1 stars 1 forks source link

MitkDrawingTool is buggy #3

Closed dboun closed 5 years ago

dboun commented 5 years ago

Each time you click paint a thickness slider gets added. This should happen once.

Hide label (eye icon) doesn't work.

In the right click on labels, some options are not needed or even crash.

The tool's theme is light, but our app is dark. For instance, the 'number of labels' text is dark and everything else white on white. There is probably a way to enable a general dark theme somehow, MITK Workbench has it.

ashishsingh18 commented 5 years ago

multiple slider issue is now fixed with #10

dboun commented 5 years ago

The paint tool is fixed. The other 3 remain.

ashishsingh18 commented 5 years ago

@dboun We need to disable right click on labels.

dboun commented 5 years ago

Is there a way to do this without messing with mitk sources? Because I'm not sure they provide a method for that

Maybe, if there isn't a function, we can put a view on top of it that lets only left clicks pass under it

ashishsingh18 commented 5 years ago

Not sure if this is possible, otherwise we need to subclass and handle.

ashishsingh18 commented 5 years ago

@dboun Can we open separate issues for each and close this one?

dboun commented 5 years ago

Also we probably want to allow the remove label, rename label from the menu (which work)

I will separate them in a bit!

dboun commented 5 years ago

Closing this issue to split it into separate ones