-
Notifications
You must be signed in to change notification settings - Fork 16.5k
fix(dashboard): resolve dropdown popup positioning #36963
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
base: master
Are you sure you want to change the base?
fix(dashboard): resolve dropdown popup positioning #36963
Conversation
|
CodeAnt AI is reviewing your PR. Thanks for using CodeAnt! 🎉We're free for open-source projects. if you're enjoying it, help us grow by sharing. Share on X · |
Nitpicks 🔍
|
|
CodeAnt AI finished reviewing your PR. |
✅ Deploy Preview for superset-docs-preview ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
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.
Code Review Agent Run #6ec00e
Actionable Suggestions - 1
-
superset-frontend/packages/superset-ui-core/src/components/Select/constants.ts - 1
- Deprecated API Usage · Line 36-40
Review Details
-
Files reviewed - 2 · Commit Range:
3f38a44..3f38a44- superset-frontend/packages/superset-ui-core/src/components/Select/Select.tsx
- superset-frontend/packages/superset-ui-core/src/components/Select/constants.ts
-
Files skipped - 0
-
Tools
- Eslint (Linter) - ✔︎ Successful
- Whispers (Secret Scanner) - ✔︎ Successful
- Detect-secrets (Secret Scanner) - ✔︎ Successful
Bito Usage Guide
Commands
Type the following command in the pull request comment and save the comment.
-
/review- Manually triggers a full AI review. -
/pause- Pauses automatic reviews on this pull request. -
/resume- Resumes automatic reviews. -
/resolve- Marks all Bito-posted review comments as resolved. -
/abort- Cancels all in-progress reviews.
Refer to the documentation for additional commands.
Configuration
This repository uses Superset You can customize the agent settings here or contact your Bito workspace admin at [email protected].
Documentation & Help
superset-frontend/packages/superset-ui-core/src/components/Select/constants.ts
Outdated
Show resolved
Hide resolved
6e73e3e to
a164e50
Compare
Code Review Agent Run #ba1917Actionable Suggestions - 0Review Details
Bito Usage GuideCommands Type the following command in the pull request comment and save the comment.
Refer to the documentation for additional commands. Configuration This repository uses Documentation & Help |
fd1b92d to
05bbab3
Compare
05bbab3 to
4d73f0e
Compare
Code Review Agent Run #653b80Actionable Suggestions - 0Review Details
Bito Usage GuideCommands Type the following command in the pull request comment and save the comment.
Refer to the documentation for additional commands. Configuration This repository uses Documentation & Help |
superset-frontend/packages/superset-ui-core/src/components/Select/constants.ts
Outdated
Show resolved
Hide resolved
|
For people who are going to QA this. The Select is used basically everywhere, so we might need to do a sanity check in most pages, including Modals and Popovers. |
superset-frontend/packages/superset-ui-core/src/components/Select/constants.ts
Outdated
Show resolved
Hide resolved
4d73f0e to
81a9b68
Compare
|
CodeAnt AI is running Incremental review Thanks for using CodeAnt! 🎉We're free for open-source projects. if you're enjoying it, help us grow by sharing. Share on X · |
superset-frontend/packages/superset-ui-core/src/components/Select/types.ts
Outdated
Show resolved
Hide resolved
81a9b68 to
109b61c
Compare
|
🎪 Showtime deployed environment on GHA for 109b61c • Environment: http://35.90.215.8:8080 (admin/admin) |
Code Review Agent Run #2e8994Actionable Suggestions - 0Review Details
Bito Usage GuideCommands Type the following command in the pull request comment and save the comment.
Refer to the documentation for additional commands. Configuration This repository uses Documentation & Help |
SUMMARY
When zooming in any page (150% to 200% zoom), the dropdown filter popup was positioning over the dropdown input creating a bad user experience when manipulating the dropdown component.
The fix simply aligns the dropdown popup always below the dropdown input.
This fix will affect all the places that are using this dropdown component.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
BEFORE:
The dropdown covers the input box, preventing any filtering or selection via typing. This breaks basic usability on smaller screens.
AFTER
short_video_after.mov
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION