Skip to content
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

[Shell] Dynamically setting SearchHandler Query property does not update text in the search box #14497

Open
kklose23 opened this issue Apr 10, 2023 · 7 comments
Labels
area-controls-entry Entry area-controls-shell Shell Navigation, Routes, Tabs, Flyout platform/android 🤖 platform/iOS 🍎 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Milestone

Comments

@kklose23
Copy link

Description

Dynamically setting the SearchHandler Query property does not update text in the search box. The search box needs to be re-rendered for the text to show up appropriately.

Steps to Reproduce

  1. Follow these steps https://learn.microsoft.com/en-us/dotnet/maui/fundamentals/shell/search?view=net-maui-7.0
  2. Dynamically set the Query property on a page where the search handler search box is visible
  3. The search box text will not change

Link to public reproduction project repository

N/A

Version with bug

7.0 (current)

Last version that worked well

Unknown/Other

Affected platforms

iOS, Android

Affected platform versions

iOS 16.2, Android 13

Did you find any workaround?

After setting the Query property, do the following:
SearchBoxVisibility = SearchBoxVisibility.Hidden;
SearchBoxVisibility = SearchBoxVisibility.Expanded;

Relevant log output

No response

@kklose23 kklose23 added the t/bug Something isn't working label Apr 10, 2023
@Eilon Eilon added the area-controls-shell Shell Navigation, Routes, Tabs, Flyout label Apr 10, 2023
@rachelkang rachelkang added the s/needs-repro Attach a solution or code which reproduces the issue label Apr 11, 2023
@ghost
Copy link

ghost commented Apr 11, 2023

Hi @kklose23. We have added the "s/needs-repro" label to this issue, which indicates that we require steps and sample code to reproduce the issue before we can take further action. Please try to create a minimal sample project/solution or code samples which reproduce the issue, ideally as a GitHub repo that we can clone. See more details about creating repros here: https://github.com/dotnet/maui/blob/main/.github/repro.md

This issue will be closed automatically in 7 days if we do not hear back from you by then - please feel free to re-open it if you come back to this issue after that time.

@ghost ghost added the legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor label Apr 11, 2023
@jfversluis
Copy link
Member

While you did add the "follow the steps from this doc" we still would like a repro. It might be that you forgot a step in following that doc or we are missing something in our docs that does not make it work. In either case, if you provide a simple reproduction sample that should make that clear really fast. Thanks!

@kklose23
Copy link
Author

@jfversluis Just wrote up a repro here: https://github.com/kklose23/shell-search-query-issue

Clicking the Get Monkeys button should also change the text within the search handler.

@ghost ghost added s/needs-attention Issue has more information and needs another look and removed s/needs-repro Attach a solution or code which reproduces the issue labels Apr 11, 2023
@jsuarezruiz jsuarezruiz removed the s/needs-attention Issue has more information and needs another look label Apr 12, 2023
@jsuarezruiz jsuarezruiz added this to the Backlog milestone Apr 12, 2023
@ghost
Copy link

ghost commented Apr 12, 2023

We've added this issue to our backlog, and we will work to address it as time and resources allow. If you have any additional information or questions about this issue, please leave a comment. For additional info about issue management, please read our Triage Process.

@XamlTest XamlTest added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Jun 27, 2023
@XamlTest
Copy link

Verified this on Visual Studio Enterprise 17.7.0 Preview 2.0. This issue repro on Android 13.0-API33 and iOS 16.4, not repro on Windows 11 with below Project:
MonkeyFinder.zip

@kklose23
Copy link
Author

Note: There is no workaround for iOS, only Android

@Eilon Eilon removed the legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor label May 10, 2024
@Thoronion
Copy link

Bug is still present on Android 14 using MAUI 8.0.92.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-controls-entry Entry area-controls-shell Shell Navigation, Routes, Tabs, Flyout platform/android 🤖 platform/iOS 🍎 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

8 participants