-
Notifications
You must be signed in to change notification settings - Fork 36
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
MGMT-17473: Add scrollbar to OpenshiftVersionDropdown component #2542
MGMT-17473: Add scrollbar to OpenshiftVersionDropdown component #2542
Conversation
@ammont82: This pull request references MGMT-17473 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the bug to target the "4.16.0" version, but no target version was set. 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 openshift-eng/jira-lifecycle-plugin repository. |
197b298
to
e1483af
Compare
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.
The scrollbar works well.
However, I just noticed a small layout change. When the additional versions modal opens, the main version dropdown stays open too. But as you click on the text input to select the custom version, the main version dropdown closes on the background.
IMO, it should either stay open until the modal is closed, or it should close when the modal is open.
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: ammont82, celdrake The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
We are using the deprecated version of the Dropdown component. (See dropdown/react-deprecated vs dropdown.) The newer version supports a flip behavior - if the dropdown doesn't have enough space bellow the toggle, it tries to render it above. Maybe this is a good opportunity to migrate? |
Also, I noticed that when we limit the height of the dropdown, the "Show all available versions" button is hidden at the bottom. I find that a bit unfortunate. |
@jgyselov is not so easy migrate to new Dropdown version. It implies a lot of changes and we need to take a look carefully. I'll open a task to do in the future. |
New changes are detected. LGTM label has been removed. |
Related to https://issues.redhat.com/browse/MGMT-17473
Adding scrollbar to OpenshiftVersionDropdown component avoid problems about changing window layout.