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

[GH-12429] Fixed extra height on Shell Flyout and fix issues with flyout items not updating layout #12967

Merged
merged 15 commits into from
Dec 18, 2020

Conversation

pictos
Copy link
Contributor

@pictos pictos commented Nov 25, 2020

Description of Change

  • removed the hard-coded value of 44 at the platform level for iOS and just applied it to the default template. On Android/UWP the FlyoutItems just size to the content you specify if you use a default template which is more expected behavior. This also lets you hide flyout items through visibility properties.
  • In order to make the resizing work on iOS I had to rework where the Views get placed on the Flyout which fixed a Tap Gesture issue
  • Fixed Android to correctly resize flyoutitems if the size of the content changes
  • Fixed UWP to correctly calculate the width, bind to the correct Automation property, resize based on content, set a default height on the UWP nav items to 44 at the xplat level.

Issues Resolved

API Changes

None

Platforms Affected

  • iOS
  • UWP
  • Android

Behavioral/Visual Changes

On iOS if you use a custom datatemplate the height will size to whatever the height of the content you specify is. This allows for a better developer experience because the developer isn't just forced to use a min height that's impossible to overcome. Android already worked like this so this also makes the platforms consistent.

Before/After Screenshots

Before:
Screen Shot 2020-11-24 at 22 14 44

After:
Screen Shot 2020-11-24 at 22 28 33

Testing Procedure

PR Checklist

  • Targets the correct branch
  • Tests are passing (or failures are unrelated)

@PureWeen PureWeen added the DO-NOT-MERGE-!!! 🛑 This is in progress and needs to be updated before it can be merged. label Nov 25, 2020
@PureWeen PureWeen removed the DO-NOT-MERGE-!!! 🛑 This is in progress and needs to be updated before it can be merged. label Dec 2, 2020
@PureWeen PureWeen added this to To do in vNext+1 (5.0.0) via automation Dec 7, 2020
@PureWeen PureWeen added this to the 5.0.0 milestone Dec 7, 2020
# Conflicts:
#	Xamarin.Forms.Controls.Issues/Xamarin.Forms.Controls.Issues.Shared/Xamarin.Forms.Controls.Issues.Shared.projitems
#	Xamarin.Forms.Platform.Android/Renderers/ContainerView.cs
- Fix Android Measuring
@PureWeen PureWeen changed the title [GH-12429] Fixed extra height on Shell Flyout [GH-12429] Fixed extra height on Shell Flyout and fix issues with flyout items not updating layout Dec 16, 2020
@PureWeen
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@PureWeen PureWeen moved this from To do to In Review in vNext+1 (5.0.0) Dec 16, 2020
@PureWeen
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@PureWeen
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

# Conflicts:
#	Xamarin.Forms.Platform.Android/Renderers/ShellViewRenderer.cs
@PureWeen
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rmarinho rmarinho merged commit acb4af5 into xamarin:5.0.0 Dec 18, 2020
vNext+1 (5.0.0) automation moved this from In Review to Done Dec 18, 2020
@pictos pictos deleted the fix-12429 branch December 9, 2022 00:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
5 participants