Skip to content

docs(docs-infra): retain the results in the search dialog while typing #61935

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hawkgs
Copy link
Member

@hawkgs hawkgs commented Jun 6, 2025

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • angular.dev application / infrastructure changes

What is the new behavior?

Due to resource() not retaining its value while there is a request in progress, the results list flickers. The new searchResults signal retains that previous value until the request is completed.

@hawkgs hawkgs requested a review from JeanMeche June 6, 2025 14:39
@hawkgs hawkgs added area: docs-infra Angular.dev application and infrastructure adev: preview labels Jun 6, 2025
@ngbot ngbot bot added this to the Backlog milestone Jun 6, 2025
@angular-robot angular-robot bot added the area: docs Related to the documentation label Jun 6, 2025
Due to `resource()` not retaining its value while there is a request in progress, the results list flickers. The new `searchResults` signal retains that previous value until the request is completed.
@hawkgs hawkgs force-pushed the docs/retain-search-dialog-value branch from 8771e94 to 1482014 Compare June 6, 2025 14:44
Copy link

github-actions bot commented Jun 6, 2025

Deployed adev-preview for 1482014 to: https://ng-dev-previews-fw--pr-angular-angular-61935-adev-prev-xycmj2yl.web.app

Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
adev: preview area: docs Related to the documentation area: docs-infra Angular.dev application and infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy