Skip to content

Forward Port of #37594 to 13.0#38080

Closed
fw-bot wants to merge 1 commit intoodoo:13.0from
odoo-dev:13.0-12.0-dropdownoverflow-lpe-M6P2Z3DR-forwardport
Closed

Forward Port of #37594 to 13.0#38080
fw-bot wants to merge 1 commit intoodoo:13.0from
odoo-dev:13.0-12.0-dropdownoverflow-lpe-M6P2Z3DR-forwardport

Conversation

@fw-bot
Copy link
Contributor

@fw-bot fw-bot commented Oct 7, 2019

[FIX] web: dropdown boundary overflow

Open a search view modal ("search more" on a m2o)
Select a filter that will make records fall to zero
The modal will shrink
Trigger the opening of a dropdown (Filters), and make sure
that there are many many menu items there (many filters)

Before this commit, the dropdown was bigger than the modal
which had a scroll bar,
however, the dropdown itself was not scrollable properly
So, clicking in it to select a menu item will fail when scrolled
all the way down, because elements did not actually followed the scroll

After this commit, the dropdown follows the scroll, and it is possible
to use it even way down

We use a behavior of bootstrap 4 itself:
FIX: twbs/bootstrap#24976
ISSUE: twbs/bootstrap#24251

OPW 2076337

Description of the issue/feature this PR addresses:

Current behavior before PR:

Desired behavior after PR is merged:

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

Forward-Port-Of: #37594

Open a search view modal ("search more" on a m2o)
Select a filter that will make records fall to zero
The modal will shrink
Trigger the opening of a dropdown (Filters), and make sure
that there are many many menu items there (many filters)

Before this commit, the dropdown was bigger than the modal
which had a scroll bar,
however, the dropdown itself was not scrollable properly
So, clicking in it to select a menu item will fail when scrolled
all the way down, because elements did not actually followed the scroll

After this commit, the dropdown follows the scroll, and it is possible
to use it even way down

We use a behavior of bootstrap 4 itself:
FIX: twbs/bootstrap#24976
ISSUE: twbs/bootstrap#24251

OPW 2076337

X-original-commit: c3273b7
@fw-bot
Copy link
Contributor Author

fw-bot commented Oct 7, 2019

Ping @kebeclibre
This PR targets 13.0 and is the last of the forward-port chain containing:

To merge the full chain, say

@fw-bot r+

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@kebeclibre
Copy link
Contributor

@fw-bot r+

@robodoo robodoo added the r+ 👌 label Oct 7, 2019
@C3POdoo C3POdoo added the OE the report is linked to a support ticket (opw-...) label Oct 7, 2019
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Oct 7, 2019
@robodoo robodoo closed this in 5fe83fc Oct 7, 2019
@robodoo
Copy link
Contributor

robodoo commented Oct 7, 2019

Merged at 5fe83fc, thanks!

@fw-bot fw-bot deleted the 13.0-12.0-dropdownoverflow-lpe-M6P2Z3DR-forwardport branch October 21, 2019 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI 🤖 Robodoo has seen passing statuses OE the report is linked to a support ticket (opw-...)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants