BrownBiomechanics / SlicerAutoscoperM

This 3D Slicer extension enables users to perform image registration.
https://autoscoperm.slicer.org
MIT License
1 stars 4 forks source link

ENH: Collapse 'Advanced' Pre-Processing menu by default #146

Closed sbelsk closed 5 days ago

sbelsk commented 1 week ago

Resolves https://github.com/BrownBiomechanics/SlicerAutoscoperM/issues/145.

The 'Advanced Options' menu in the Pre-Processing module is still toggleable, but is now rendered as collapsed by default on module launch.

sbelsk commented 5 days ago

Noting that this change will be superseded by the UI overhaul in https://github.com/BrownBiomechanics/SlicerAutoscoperM/pull/141, @amymmorton would you like this change to merge first anyway, or should we discard this PR and instead implement the request from https://github.com/BrownBiomechanics/SlicerAutoscoperM/issues/145 in https://github.com/BrownBiomechanics/SlicerAutoscoperM/pull/141 (e.g., by rendering the "Default Subdirectories" group as collapsed by default)?

amymmorton commented 5 days ago

Great point- we can discard (sorry!) in favor of 141

On Wed, Nov 20, 2024 at 10:44 AM Shelly Belsky @.***> wrote:

Noting that this change will be superseded by the UI overhaul in #141 https://github.com/BrownBiomechanics/SlicerAutoscoperM/pull/141, @amymmorton https://github.com/amymmorton would you like this change to merge first anyway, or should we discard this PR and instead implement the request from #145 https://github.com/BrownBiomechanics/SlicerAutoscoperM/issues/145 in

141 https://github.com/BrownBiomechanics/SlicerAutoscoperM/pull/141

(e.g., by rendering the "Default Subdirectories" group as collapsed by default)?

— Reply to this email directly, view it on GitHub https://github.com/BrownBiomechanics/SlicerAutoscoperM/pull/146#issuecomment-2488931623, or unsubscribe https://github.com/notifications/unsubscribe-auth/APUUPFWX7MZQNFFNA3EEUSL2BSU5NAVCNFSM6AAAAABRW4MDS6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOBYHEZTCNRSGM . You are receiving this because you were mentioned.Message ID: @.***>

--

Amy Morton MSc Senior Research Engineer

*Brown University Healt*h

Orthopaedic Research, 1 Hoppin Street, Coro West, 4th Providence, RI 02903-4923

linkedin https://www.linkedin.com/in/amy-morton-35374340/

sbelsk commented 5 days ago

Closing in favor of #141.