Skip to content

feat: implemented hide on scroll strategy w/ tests#1782

Open
NikGurev wants to merge 2 commits into
feat/DS-1920from
feat/hide-on-scroll-overlay
Open

feat: implemented hide on scroll strategy w/ tests#1782
NikGurev wants to merge 2 commits into
feat/DS-1920from
feat/hide-on-scroll-overlay

Conversation

@NikGurev

Copy link
Copy Markdown
Contributor

Summary

Applied ability of using new scroll strategy in overlay related components.

Existing behavior is unchanged. For popover - it used hide on scroll behavior, so applied new strategy by default

  • scroll strategy token in each component still defaults to reposition
  • When applied a new strategy, component listens to bounds intersection (hide$) and hides/closes depending on implementation. So strategy simply emits , others react .
  • Covered new feature with tests for all related components
  • Provided example for popover to visually check realization.

@NikGurev NikGurev self-assigned this Jul 24, 2026
@NikGurev NikGurev added enhancement New feature or request 20.x labels Jul 24, 2026
@github-actions

Copy link
Copy Markdown

Visit the preview URL for this PR (updated for commit 78e5b70):

https://koobiq-next--prs-1782-q0uf1xlm.web.app

(expires Mon, 27 Jul 2026 16:03:04 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

20.x enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant