-
Notifications
You must be signed in to change notification settings - Fork 1
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
Set fixed height for the scroll of the bib. #115
Comments
Meeting is scheduled with the team for a feature tech'refinement discussion. |
This should be removed from the AC. It is already implemented. |
This should at a minimum include the top as well. We may want to look at left and right padding as well. This portion of the change may need to be in auro-dropdown though. Since we would want consistent behavior between at minimum auro-select and auro-combobox. |
Post conversation it was agreed that this work is not priority and the thing to be done is to update the demo to illustrate how a user can simply use CSS to restrict height and auro-menu will naturally listen to this. |
Work was addressed in this PR: |
Is your feature request related to a problem? Please describe.
See the following issue for background on how this was discovered and the design work that was done.
There is also this issue that came in after this was created that we should also consider from a UX point of view and perhaps use the same solution for both use cases where we have a long list of options with auro-select.
Describe the solution you'd like
The design should be discussed as to how this solves the original issue that was submitted. A technical specification needs to be discussed with the team and exit criteria defined.
AC:
Note for `finement.
The flight search team implemented the work around Geoff suggested for the currency converter.
https://www.alaskaair.com/search/results?A=1&O=sea&D=las&OD=2023-08-25&OT=Anytime&DD=2023-08-25&DT=Anytime&UPG=none
Scroll down to the bottom of the page and click on the currency converter in the fine print.
The text was updated successfully, but these errors were encountered: