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
FolderPicker: Prevent dropdown menu from disappearing off screen #32603
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jackw
added
type/bug
area/panel/dashboard-list
area/panel/alertlist
area/dashboard/folders
area/frontend
add to changelog
old backport v7.5.x
Mark PR for automatic backport to v7.5.x
labels
Apr 1, 2021
jackw
requested review from
torkelo and
kaydelaney
and removed request for
a team
April 1, 2021 14:38
torkelo
reviewed
Apr 2, 2021
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes |
jackw
force-pushed
the
jackw/folder-picker-positioning
branch
from
April 2, 2021 08:02
47ba11b
to
0d2bcae
Compare
jackw
changed the title
FolderPicker: Prevent dropdown menu from disappearing off screen or inside modal
FolderPicker: Prevent dropdown menu from disappearing off screen
Apr 2, 2021
…r for dashlist & alertlist
jackw
force-pushed
the
jackw/folder-picker-positioning
branch
from
April 7, 2021 07:13
0d2bcae
to
62a3701
Compare
torkelo
approved these changes
Apr 7, 2021
grafanabot
pushed a commit
that referenced
this pull request
Apr 7, 2021
) * fix(folderpicker): remove position fixed so menu works in panel editor for dashlist & alertlist * test(folderpicker): update snapshot (cherry picked from commit 8b38b70)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
add to changelog
area/dashboard/folders
area/frontend
area/panel/alertlist
area/panel/dashboard-list
old backport v7.5.x
Mark PR for automatic backport to v7.5.x
type/bug
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR does / why we need it:
This PR fixes odd positioning issue with the FolderPicker component that causes it to appear offscreen.
I also followed the same implementation in this earlier PR to fix issues around select menus not being able to expand beyond the modal container.
Which issue(s) this PR fixes:
Fixes #32474
Special notes for your reviewer:
I tested the following uses of
FolderPicker
and couldn't find any visual issues with these changes: