View Issue Details

IDProjectCategoryView StatusLast Update
0001281Slicer4Core: GUIpublic2011-09-15 19:51
Reporterkikinis Assigned Topieper  
PrioritynormalSeverityfeatureReproducibilityalways
Status closedResolutionfixed 
Product VersionSlicer 4.0 gamma RC2 
Target VersionSlicer 4.0.0Fixed in VersionSlicer 4.0.0 
Summary0001281: cosmetics for the editor tool
Description

Suggested change in the arrangement of the editor tool presentation. One big advantage of this would be that the effects have the whole width of the module available, when selected. Mockup attached. BTW: do we need the default tool? It does not seem to have any function from a user point of view.

TagsNo tags attached.

Activities

2011-07-18 04:24

 

pieper

pieper

2011-07-18 05:57

administrator   ~0002607

Sure, that layout makes a lot of sense.

Regarding the default tool, it is there mainly as a way to escape/cancel the current tool.

kikinis

kikinis

2011-07-18 06:23

developer   ~0002608

The default when opening that page should be the drawing or painting tool.
The new shape of the mouse pointer feels strange to me. It is not clear when it is turned on and when it is turned off.
The erase tool sets the label value to 0, but when I go back to an effect I have to reselect the label I was working with.

2011-08-15 05:19

 

kikinis

kikinis

2011-08-15 05:19

developer   ~0002698

Please use the eye instead of a check box to show if advanced is visible or not

pieper

pieper

2011-08-18 12:09

administrator   ~0002735

Actually I think a collabsable group box is more appropriate than the eye icon for the Advanced tab (it has the triangle that rotates to show the status).

kikinis

kikinis

2011-08-18 12:24

developer   ~0002736

agreed. That is what Isaiah did for TractographyDisplay , 0001296 and what Julien suggested:"In order to be consistent with other modules, I would advice against the eye or the checkbox, but to use a ctkCollapsibleGroupBox (or ctkCollapsibleButton). "

pieper

pieper

2011-09-05 15:13

administrator   ~0002915

Addressed in revision 17970.

Fixed the erase - it now works as in slicer3 where it toggles the paint color without exiting the current effect.

Was not able to make paint tool active when starting - we may add this later, but I don't think it is too critical.

Issue History

Date Modified Username Field Change
2011-07-18 04:24 kikinis New Issue
2011-07-18 04:24 kikinis File Added: Screen shot 2011-07-18 at 7.53.43 AM.png
2011-07-18 05:57 pieper Note Added: 0002607
2011-07-18 06:23 kikinis Note Added: 0002608
2011-07-18 14:53 finetjul Status new => assigned
2011-07-18 14:53 finetjul Assigned To => pieper
2011-08-15 05:19 kikinis File Added: Screen Shot 2011-08-15 at 8.31.09 AM.png
2011-08-15 05:19 kikinis Note Added: 0002698
2011-08-18 12:09 pieper Note Added: 0002735
2011-08-18 12:24 kikinis Note Added: 0002736
2011-08-25 09:27 finetjul Target Version => Slicer 4.0 RSNA
2011-09-05 15:13 pieper Note Added: 0002915
2011-09-05 15:13 pieper Status assigned => resolved
2011-09-06 04:54 kikinis Status resolved => closed
2011-09-06 04:54 kikinis Resolution open => fixed
2011-09-15 19:51 finetjul Fixed in Version => Slicer 4.0 RSNA