-
Notifications
You must be signed in to change notification settings - Fork 30.6k
[FIX] website_slides: All Courses result dropdown below search #206909
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
Conversation
Scenario: go to homepage > Courses > one "View all" > type in search Result: dropdown with result is shown over the search input Issue: search form is in flex layout, but the dropdown is displayed in absolute so it is outside of flex flow and is positionned at the top-right of the form container. History: - before bootstrap 4 (odoo 15): the search form had a flex layout, but bootstrap dropdown used "top:100%" so were positionned at the bottom of it. - as of bootstrap 5 (odoo 16): the form kept a flex layout, but bootstrap dropdown now used "position: absolute" with unset top putting them below the search item in a block container, but over it in a flex layout. Fix: when the search bar is displayed, display it as block. opw-4735257
c798362 to
056b773
Compare
|
@odoo/rd-sm-learn what do you think about this? I have updated the target from 16.0 to 17.0 since 16.0 is unsupported The issue also doesn't happen since 19.0 since this has been refactored (eg. robodoo up to saas-18.4 |
|
Forward-porting to 'saas-18.4'. |
|
@flch-odoo what do you think about this? thanks |
flch-odoo
left a comment
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.
@robodoo r+
Scenario: go to homepage > Courses > one "View all" > type in search Result: dropdown with result is shown over the search input Issue: search form is in flex layout, but the dropdown is displayed in absolute so it is outside of flex flow and is positionned at the top-right of the form container. History: - before bootstrap 4 (odoo 15): the search form had a flex layout, but bootstrap dropdown used "top:100%" so were positionned at the bottom of it. - as of bootstrap 5 (odoo 16): the form kept a flex layout, but bootstrap dropdown now used "position: absolute" with unset top putting them below the search item in a block container, but over it in a flex layout. Fix: when the search bar is displayed, display it as block. opw-4735257 closes #206909 Signed-off-by: Florian Charlier (flch) <flch@odoo.com>
Scenario: go to homepage > Courses > one "View all" > type in search Result: dropdown with result is shown over the search input Issue: search form is in flex layout, but the dropdown is displayed in absolute so it is outside of flex flow and is positionned at the top-right of the form container. History: - before bootstrap 4 (odoo 15): the search form had a flex layout, but bootstrap dropdown used "top:100%" so were positionned at the bottom of it. - as of bootstrap 5 (odoo 16): the form kept a flex layout, but bootstrap dropdown now used "position: absolute" with unset top putting them below the search item in a block container, but over it in a flex layout. Fix: when the search bar is displayed, display it as block. opw-4735257 closes odoo#206909 Signed-off-by: Florian Charlier (flch) <flch@odoo.com>

Scenario: go to homepage > Courses > one "View all" > type in search
Result: dropdown with result is shown over the search input
Issue: search form is in flex layout, but the dropdown is displayed
in absolute so it is outside of flex flow and is positionned at the
top-right of the form container.
History:
before bootstrap 4 (odoo 15): the search form had a flex layout, but
bootstrap dropdown used "top:100%" so were positionned at the bottom
of it.
as of bootstrap 5 (odoo 16): the form kept a flex layout, but
bootstrap dropdown now used "position: absolute" with unset top
putting them below the search item in a block container, but over
it in a flex layout.
Fix: when the search bar is displayed, display it as block.
opw-4735257