Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

[Bug] Shell.SearchHandler shows items with full screen height on Android #13403

Closed
mkanyo opened this issue Jan 14, 2021 · 0 comments · Fixed by #13658
Closed

[Bug] Shell.SearchHandler shows items with full screen height on Android #13403

mkanyo opened this issue Jan 14, 2021 · 0 comments · Fixed by #13658
Assignees
Labels
5.0.0 Regression on 5.0.0 a/shell 🐚 i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often i/regression p/Android t/bug 🐛
Projects

Comments

@mkanyo
Copy link

mkanyo commented Jan 14, 2021

Description

Running the latest version of Xamarin Forms 5.0.0.1874 the Shell.SearchHandler shows search result items with full screen height on Android.

Steps to Reproduce

The issue can currently be observed by running the Xamarin Form Sample Xaminals.

This is the repo:
https://github.com/xamarin/xamarin-forms-samples/tree/master/UserInterface/Xaminals

In the MoneysPage.xaml
The SearchHandler is defined like this:

    <Shell.SearchHandler>
        <controls:MonkeySearchHandler Placeholder="Enter search term"
                                      ShowsResults="true"
                                      DisplayMemberName="Name" />
    </Shell.SearchHandler>

The custom SearchHandler is like so:

using System.Linq;
using System.Threading.Tasks;
using Xamarin.Forms;
using Xaminals.Data;
using Xaminals.Models;

namespace Xaminals.Controls
{
    public class MonkeySearchHandler : SearchHandler
    {
        protected override void OnQueryChanged(string oldValue, string newValue)
        {
            base.OnQueryChanged(oldValue, newValue);

            if (string.IsNullOrWhiteSpace(newValue))
            {
                ItemsSource = null;
            }
            else
            {
                ItemsSource = MonkeyData.Monkeys
                    .Where(monkey => monkey.Name.ToLower().Contains(newValue.ToLower()))
                    .ToList<Animal>();
            }
        }

        protected override async void OnItemSelected(object item)
        {
            base.OnItemSelected(item);
            await Task.Delay(1000);

            // Note: strings will be URL encoded for navigation (e.g. "Blue Monkey" becomes "Blue%20Monkey"). Therefore, decode at the receiver.
            // This works because route names are unique in this application.
            await Shell.Current.GoToAsync($"monkeydetails?name={((Animal)item).Name}");
            // The full route is shown below.
            // await Shell.Current.GoToAsync($"//animals/monkeys/monkeydetails?name={((Animal)item).Name}");
        }
    }
}

Expected Behavior

The search items should render under each other with reasonable size.

Actual Behavior

Each item gets full screen height and the list therefore becomes huge with full screen items and a lengthy scrollbar.

Basic Information

  • Version with issue: 5.0.0.1874
  • Last known good version: 5.0.0.1791-pre5
  • Platform Target Frameworks:
    • Android: 10.0,

Screenshots

Screenshot:
https://www.screencast.com/users/MiklosKanyo/folders/Default/media/83b3ee95-98ae-49ec-9933-57853968e787

Video:
https://www.screencast.com/users/MiklosKanyo/folders/Default/media/5adeb4e5-45d4-42e4-a13b-86cc1e510442

Show/Hide Visual Studio info
Microsoft Visual Studio Professional 2019
Version 16.8.3
VisualStudio.16.Release/16.8.3+30804.86
Microsoft .NET Framework
Version 4.8.04084

Installed Version: Professional

Visual C++ 2019   00435-60000-00000-AA382
Microsoft Visual C++ 2019

ASP.NET and Web Tools 2019   16.8.554.20160
ASP.NET and Web Tools 2019

ASP.NET Core Razor Language Services   16.1.0.2052803+84e121f1403378489b842e1797df2f3f5a49ac3c
Provides languages services for ASP.NET Core Razor.

ASP.NET Web Frameworks and Tools 2019   16.8.554.20160
For additional information, visit https://www.asp.net/

Azure App Service Tools v3.0.0   16.8.554.20160
Azure App Service Tools v3.0.0

Azure Functions and Web Jobs Tools   16.8.554.20160
Azure Functions and Web Jobs Tools

C# Tools   3.8.0-5.20604.10+9ed4b774d20940880de8df1ca8b07508aa01c8cd
C# components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.

Common Azure Tools   1.10
Provides common services for use by Azure Mobile Services and Microsoft Azure Tools.

Extensibility Message Bus   1.2.6 (master@34d6af2)
Provides common messaging-based MEF services for loosely coupled Visual Studio extension components communication and integration.

ILSpy.AddIn   1.0
Integration of the ILSpy Decompiler into Visual Studio.

IntelliCode Extension   1.0
IntelliCode Visual Studio Extension Detailed Info

Microsoft Azure Tools   2.9
Microsoft Azure Tools for Microsoft Visual Studio 2019 - v2.9.30924.1

Microsoft Continuous Delivery Tools for Visual Studio   0.4
Simplifying the configuration of Azure DevOps pipelines from within the Visual Studio IDE.

Microsoft JVM Debugger   1.0
Provides support for connecting the Visual Studio debugger to JDWP compatible Java Virtual Machines

Microsoft Library Manager   2.1.113+g422d40002e.RR
Install client-side libraries easily to any web project

Microsoft MI-Based Debugger   1.0
Provides support for connecting Visual Studio to MI compatible debuggers

Microsoft Visual C++ Wizards   1.0
Microsoft Visual C++ Wizards

Microsoft Visual Studio Tools for Containers   1.1
Develop, run, validate your ASP.NET Core applications in the target environment. F5 your application directly into a container with debugging, or CTRL + F5 to edit & refresh your app without having to rebuild the container.

Microsoft Visual Studio VC Package   1.0
Microsoft Visual Studio VC Package

Mono Debugging for Visual Studio   16.8.43 (00471f8)
Support for debugging Mono processes with Visual Studio.

Node.js Tools   1.5.20902.1 Commit Hash:b474efcb6f92db52a8f8e2e6a8cb9648476885cc
Adds support for developing and debugging Node.js apps in Visual Studio

NuGet Package Manager   5.8.0
NuGet Package Manager in Visual Studio. For more information about NuGet, visit https://docs.nuget.org/

ProjectServicesPackage Extension   1.0
ProjectServicesPackage Visual Studio Extension Detailed Info

SQL Server Data Tools   16.0.62010.06180
Microsoft SQL Server Data Tools

StylerPackage Extension   1.0
StylerPackage Visual Stuido Extension Detailed Info

TypeScript Tools   16.0.21016.2001
TypeScript Tools for Microsoft Visual Studio

Visual Basic Tools   3.8.0-5.20604.10+9ed4b774d20940880de8df1ca8b07508aa01c8cd
Visual Basic components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.

Visual F# Tools   16.8.0-beta.20507.4+da6be68280c89131cdba2045525b80890401defd
Microsoft Visual F# Tools

Visual Studio Code Debug Adapter Host Package   1.0
Interop layer for hosting Visual Studio Code debug adapters in Visual Studio

Visual Studio Container Tools Extensions   1.0
View, manage, and diagnose containers within Visual Studio.

Visual Studio Tools for Containers   1.0
Visual Studio Tools for Containers

VisualStudio.DeviceLog   1.0
Information about my package

VisualStudio.Foo   1.0
Information about my package

VisualStudio.Mac   1.0
Mac Extension for Visual Studio

Xamarin   16.8.000.260 (d16-8@fd405a2)
Visual Studio extension to enable development for Xamarin.iOS and Xamarin.Android.

Xamarin Designer   16.8.0.507 (remotes/origin/d16-8@e87b24884)
Visual Studio extension to enable Xamarin Designer tools in Visual Studio.

Xamarin Templates   16.8.112 (86385a3)
Templates for building iOS, Android, and Windows apps with Xamarin and Xamarin.Forms.

Xamarin.Android SDK   11.1.0.17 (d16-8/c0e2b8e)
Xamarin.Android Reference Assemblies and MSBuild support.
    Mono: be2226b
    Java.Interop: xamarin/java.interop/d16-8@79d9533
    ProGuard: Guardsquare/proguard/proguard6.2.2@ebe9000
    SQLite: xamarin/sqlite/3.32.1@1a3276b
    Xamarin.Android Tools: xamarin/xamarin-android-tools/d16-8@2fb1cbc


Xamarin.iOS and Xamarin.Mac SDK   14.6.0.15 (87a1b18d8)
Xamarin.iOS and Xamarin.Mac Reference Assemblies and MSBuild support.

Reproduction Link

https://github.com/xamarin/xamarin-forms-samples/tree/master/UserInterface/Xaminals

Workaround

Rolling back to version 5.0.0.1791-pre5

@mkanyo mkanyo added s/unverified New report that has yet to be verified t/bug 🐛 labels Jan 14, 2021
@samhouts samhouts added this to New in Triage Jan 14, 2021
@mkanyo mkanyo changed the title [Bug] Shell.SearchHandler shows items without height on Android [Bug] Shell.SearchHandler shows items with full screen height on Android Jan 14, 2021
@PureWeen PureWeen added 5.0.0 Regression on 5.0.0 i/regression labels Jan 20, 2021
@PureWeen PureWeen added this to To do in vNext+1 (5.0.0) via automation Jan 20, 2021
@PureWeen PureWeen self-assigned this Jan 20, 2021
@PureWeen PureWeen moved this from New to Ready For Work in Triage Jan 20, 2021
@Redth Redth moved this from Ready For Work to Needs Estimate in Triage Jan 21, 2021
@hartez hartez added this to To Do in 5.0.0 SR 3 Feb 2, 2021
@hartez hartez moved this from To Fix to To Verify in 5.0.0 SR 3 Feb 2, 2021
@hartez hartez removed this from To do in vNext+1 (5.0.0) Feb 2, 2021
@hartez hartez removed the s/unverified New report that has yet to be verified label Feb 2, 2021
@hartez hartez removed this from Needs Estimate in Triage Feb 2, 2021
@hartez hartez added the i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often label Feb 2, 2021
@hartez hartez moved this from To Verify to To Fix in 5.0.0 SR 3 Feb 2, 2021
@PureWeen PureWeen moved this from To Fix to Issue In Progress in 5.0.0 SR 3 Feb 4, 2021
5.0.0 SR 3 automation moved this from Issue In Progress to Done Feb 9, 2021
rmarinho pushed a commit that referenced this issue Feb 9, 2021
…fixes #13403

* Force Shell TitleView to height of container

* - fix collapse on scroll and scroll

* - fix android

* - uiitests

* - fix header positioning

* Fix search handler results to size to view

* Update ShellViewRenderer.cs
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
5.0.0 Regression on 5.0.0 a/shell 🐚 i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often i/regression p/Android t/bug 🐛
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

4 participants