Skip to content

fix: regression - search results not visible until scroll#23321

Merged
alextran1502 merged 1 commit intomainfrom
fix/search_results_visible
Oct 28, 2025
Merged

fix: regression - search results not visible until scroll#23321
alextran1502 merged 1 commit intomainfrom
fix/search_results_visible

Conversation

@midzelis
Copy link
Collaborator

Regression from #23194 - improper reactivity

@github-actions
Copy link
Contributor

github-actions bot commented Oct 28, 2025

Preview environment has been removed.

const w = {
let scrollTop = $state(0);
let slidingWindow = $derived.by(() => {
const top = (scrollTop || 0) - slidingWindowOffset;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

|| 0 seems unnecessary

@alextran1502 alextran1502 merged commit 8d25f81 into main Oct 28, 2025
67 of 69 checks passed
@alextran1502 alextran1502 deleted the fix/search_results_visible branch October 28, 2025 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants