brainglobe / brainglobe-registration

Registration to BrainGlobe atlases in napari using Elastix
BSD 3-Clause "New" or "Revised" License
13 stars 2 forks source link

Moving from tabs to a dropdown style widget #25

Closed IgorTatarnikov closed 9 months ago

IgorTatarnikov commented 9 months ago

Before submitting a pull request (PR), please read the contributing guide.

Please fill out as much of this template as you can, but if you have any problems or questions, just leave a comment and we will help out :)

Description

What is this PR

Why is this PR needed? This PR replaces tabs with collapsible compartments to guide the user through the workflow without overloading the screen with all possible options.

What does this PR do? Use the CollapsibleWidgetContainer from brainglobe-utils to organize the widgets displayed by the plugin.

How has this PR been tested?

All previous tests pass, plugin function has been examined visually.

Is this a breaking change?

No.

Does this PR require an update to the documentation?

No. If any features have changed, or have been added. Please explain how the documentation has been updated (and link to the associated PR). See here for details.

Checklist:

codecov[bot] commented 9 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (0f32e54) 82.35% compared to head (1885bd8) 82.08%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev #25 +/- ## ========================================== - Coverage 82.35% 82.08% -0.27% ========================================== Files 9 9 Lines 408 402 -6 ========================================== - Hits 336 330 -6 Misses 72 72 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

IgorTatarnikov commented 9 months ago

Sounds good! I've made issues to keep track of these suggestions.