Skip to content

Use config file to set unselected annot opacity - #543

Merged
naglepuff merged 2 commits into
masterfrom
mln/unfocused-annotation-opacity-setting
Aug 11, 2026
Merged

Use config file to set unselected annot opacity#543
naglepuff merged 2 commits into
masterfrom
mln/unfocused-annotation-opacity-setting

Conversation

@naglepuff

Copy link
Copy Markdown
Contributor

Fix #541

Previously, non-selected annotation elements had their opacity reduced by multiplying by 0.25. This reduction was hard-coded behavior in the girder_large_image_annotation image viewer widget.

Accompanying changes to the girder_large_image_annotation plugin allow passing in a custom opacity multiplier.

The changes in this PR read a key/value pair with the key unselectedOpacityMultiplier from the .histomicsui_config.yaml item and pass that value to the image viewer widget.

@naglepuff
naglepuff force-pushed the mln/unfocused-annotation-opacity-setting branch from 8e2919d to 4900f05 Compare August 10, 2026 21:30
@naglepuff
naglepuff force-pushed the mln/unfocused-annotation-opacity-setting branch from 4900f05 to 7572019 Compare August 10, 2026 21:59
@naglepuff
naglepuff marked this pull request as ready for review August 11, 2026 14:00
@naglepuff
naglepuff requested a review from manthey August 11, 2026 14:00
@manthey

manthey commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Testing now -- can you add this to the docs here: https://github.com/DigitalSlideArchive/HistomicsUI/blob/master/docs/config_options.rst

@manthey manthey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can add the new config setting to the docs as part of this PR or as a follow on PR.

@naglepuff

Copy link
Copy Markdown
Contributor Author

We can add the new config setting to the docs as part of this PR or as a follow on PR.

Added in d70893c. Let me know if you'd like and changes @manthey

@manthey

manthey commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

We can add the new config setting to the docs as part of this PR or as a follow on PR.

Added in d70893c. Let me know if you'd like and changes @manthey

LGTM. Go ahead and merge.

@naglepuff
naglepuff merged commit 349321d into master Aug 11, 2026
9 checks passed
@naglepuff
naglepuff deleted the mln/unfocused-annotation-opacity-setting branch August 11, 2026 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve visibility of non-selected instances

2 participants