Skip to content

fix(search-modal): move overscroll-behavior to dialog#1455

Merged
LeoMcA merged 2 commits intomainfrom
1451-search-modal-scrollable
Apr 13, 2026
Merged

fix(search-modal): move overscroll-behavior to dialog#1455
LeoMcA merged 2 commits intomainfrom
1451-search-modal-scrollable

Conversation

@caugner
Copy link
Copy Markdown
Contributor

@caugner caugner commented Apr 7, 2026

Description

Updates the search-modal component, moving overscroll-behavior from ul to the enclosing dialog.

Motivation

Ensure the search modal is scrollable in current stable versions of Chrome, Firefox, and Safari.

Additional details

Related issues and pull requests

Fixes #1451.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

d18b9a8 was deployed to: https://fred-pr1455.review.mdn.allizom.net/

@caugner caugner marked this pull request as ready for review April 7, 2026 11:05
@caugner caugner requested a review from a team as a code owner April 7, 2026 11:05
@caugner caugner requested a review from LeoMcA April 7, 2026 11:05
@LeoMcA LeoMcA enabled auto-merge (squash) April 13, 2026 08:49
@LeoMcA LeoMcA merged commit c2d9952 into main Apr 13, 2026
16 of 17 checks passed
@LeoMcA LeoMcA deleted the 1451-search-modal-scrollable branch April 13, 2026 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Search results are not scrollable due to changes in overscroll-behavior: contain

4 participants