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
Bug 1996792: Fix quick search modal missing icons #9856
Conversation
/kind bug |
@vikram-raj: This pull request references Bugzilla bug 1996792, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker. 3 validation(s) were run on this bug
No GitHub users were found matching the public email listed for the QA contact in Bugzilla (gamore@redhat.com), skipping review request. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
frontend/packages/console-shared/src/components/quick-search/QuickSearchList.tsx
Show resolved
Hide resolved
5 items should be visible on the left without the scrollbar, I think the height calculation here needs to be fixed as well. |
@vikram-raj: This pull request references Bugzilla bug 1996792, which is valid. 3 validation(s) were run on this bug
No GitHub users were found matching the public email listed for the QA contact in Bugzilla (gamore@redhat.com), skipping review request. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
353d777
to
6c54a1e
Compare
Fixed this. PTAL. |
@vikram-raj: This pull request references Bugzilla bug 1996792, which is valid. 3 validation(s) were run on this bug
No GitHub users were found matching the public email listed for the QA contact in Bugzilla (gamore@redhat.com), skipping review request. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/test frontend |
@vikram-raj: This pull request references Bugzilla bug 1996792, which is valid. 3 validation(s) were run on this bug
No GitHub users were found matching the public email listed for the QA contact in Bugzilla (gamore@redhat.com), skipping review request. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/lgtm |
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.
Since this was not the goal that was set out to do, I think we can keep it as-is. But there are definite ways we should improve this -- like using a flex container and have a min and max size. But that's not really needed to fix this issue.
@@ -44,7 +44,7 @@ const QuickSearchModalBody: React.FC<QuickSearchModalBodyProps> = ({ | |||
viewContainer, | |||
}) => { | |||
const DEFAULT_HEIGHT_WITH_NO_ITEMS = 60; | |||
const DEFAULT_HEIGHT_WITH_ITEMS = 400; | |||
const DEFAULT_HEIGHT_WITH_ITEMS = 483; |
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.
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andrewballantyne, karthikjeeyar, vikram-raj The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@vikram-raj: All pull requests linked via external trackers have merged: Bugzilla bug 1996792 has been moved to the MODIFIED state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Fixes:
https://issues.redhat.com/browse/ODC-6248
Analysis / Root cause:
The quick search results icons are missing.
Solution Description:
Screen shots / Gifs for design review:
Before:
After:
Browser conformance: