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

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
Milestone

Comments

@slicerbot
Copy link
Collaborator

slicerbot commented Mar 13, 2020

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
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
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
Labels
None yet
Development

No branches or pull requests

2 participants