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

Adjust position and z-index of region control dropdown #391

Merged
merged 1 commit into from
Jun 20, 2024

Conversation

gabrielle6249
Copy link
Contributor

This changes the z-index and position of the second region control's drop down button so it doesn't cover the first region control's drop down menu, an issue raised here

@gabrielle6249 gabrielle6249 marked this pull request as ready for review June 13, 2024 14:13
Copy link

@marySalvi marySalvi left a comment

Choose a reason for hiding this comment

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

Looks good to me.
I checked to make sure this did not affect standard HistomicsUI analysis since this applies to the region-dropdown class and not specific to slicer_cli_web, it appears to work across the board.
I will leave official review/approval to @manthey in case this affects other analysis.

@gabrielle6249 gabrielle6249 changed the title Adjusting z-index of region control dropdown Adjust position and z-index of region control dropdown Jun 17, 2024
@manthey manthey merged commit d02237b into master Jun 20, 2024
9 checks passed
@manthey manthey deleted the region-control-z-level branch June 20, 2024 14:45
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.

3 participants