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

Jump to bottom

Slice offset slider widget offset (ctkSliderWidget) #3346

Closed
slicerbot opened this issue Mar 13, 2020 · 0 comments
Closed

Slice offset slider widget offset (ctkSliderWidget) #3346

slicerbot opened this issue Mar 13, 2020 · 0 comments
Assignees
@lassoan
Milestone
Slicer 4.13

Comments

@slicerbot
Copy link
Collaborator

This issue was created automatically from an original Mantis Issue. Further discussion may take place here.
@lassoan lassoan added this to the Backlog milestone Mar 16, 2020
@lassoan lassoan removed the backlog label Mar 16, 2020
lassoan added a commit to lassoan/Slicer that referenced this issue Aug 8, 2020
@lassoan
BUG: Make slice offset slider range span between slice centerpoints
2204a32 
Slice slider offset covered are between the two edges of the image (side of the voxels). This was a problem because when the slider was manually moved all the way to one side, then the slice position was not centered anymore but was set at the boundary between two slices.

Changed range to start at the first voxel center and end at the last voxel center. This way the slice position set by the slider remains centered on the slice.

Fixes Slicer#3346.
@lassoan lassoan modified the milestones: Backlog, Slicer 4.11.0 Aug 8, 2020
@lassoan lassoan self-assigned this Aug 8, 2020
@lassoan lassoan closed this as completed Aug 8, 2020
lassoan added a commit that referenced this issue Aug 8, 2020
@lassoan
BUG: Make slice offset slider range span between slice centerpoints
600f722 
Slice slider offset covered are between the two edges of the image (side of the voxels). This was a problem because when the slider was manually moved all the way to one side, then the slice position was not centered anymore but was set at the boundary between two slices.

Changed range to start at the first voxel center and end at the last voxel center. This way the slice position set by the slider remains centered on the slice.

Fixes #3346.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lassoan lassoan

Labels
None yet
Milestone
Slicer 4.13
Development

No branches or pull requests
2 participants
@lassoan @slicerbot