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

[Android] fixes issue content set after an await is not visible #4861

Merged
merged 5 commits into from
Jan 29, 2019
Merged

[Android] fixes issue content set after an await is not visible #4861

merged 5 commits into from
Jan 29, 2019

Conversation

paymicro
Copy link
Contributor

@paymicro paymicro commented Dec 26, 2018

Description of Change

Added calling RequestLayout on parent Layout when adding child in current Layout.

Issues Resolved

API Changes

None

Platforms Affected

  • Android

Behavioral/Visual Changes

None

Before/After Screenshots

before
screenshot_2

after
screenshot_1

Testing Procedure

  • run UItest 1760

PR Checklist

  • Has automated tests
  • Rebased on top of the target branch at time of PR
  • Changes adhere to coding standard

Copy link
Contributor

@hartez hartez left a comment

Choose a reason for hiding this comment

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

Fix looks good, we just need to add a few things to the tests.

@samhouts samhouts added the blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. label Jan 2, 2019
@samhouts
Copy link
Member

build --uitests

Copy link
Member

@samhouts samhouts left a comment

Choose a reason for hiding this comment

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


Xamarin.Forms.Controls.Issues\Xamarin.Forms.Controls.Issues.Shared\Issue1760_1.cs(25,30): Error CS0103: The name 'Before' does not exist in the current context
--
  | Xamarin.Forms.Controls.Issues\Xamarin.Forms.Controls.Issues.Shared\Issue1760_1.cs(26,30): Error CS0103: The name 'After' does not exist in the current context
  | Xamarin.Forms.Controls.Issues\Xamarin.Forms.Controls.Issues.Shared\Issue1760_1.cs(29,30): Error CS0103: The name 'Before' does not exist in the current context
  | Xamarin.Forms.Controls.Issues\Xamarin.Forms.Controls.Issues.Shared\Issue1760_1.cs(30,30): Error CS0103: The name 'After' does not exist in the current context
  | Process 'msbuild.exe' exited with code '1'.

@samhouts
Copy link
Member

build --uitests

@samhouts samhouts dismissed their stale review January 19, 2019 00:27

fixed build, awaiting tests

@hartez
Copy link
Contributor

hartez commented Jan 29, 2019

These test failures don't look like they have anything to do with these changes (especially not the iOS ones).

@hartez hartez merged commit bf07331 into xamarin:3.5.0 Jan 29, 2019
@samhouts samhouts added this to the 3.5.0 milestone Feb 5, 2019
samhouts added a commit that referenced this pull request Feb 13, 2019
samhouts added a commit that referenced this pull request Feb 13, 2019
@samhouts samhouts added the approved Has two approvals, no pending reviews, and no changes requested label Jul 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Has two approvals, no pending reviews, and no changes requested blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. e/2 🕑 2 i/regression p/Android t/bug 🐛
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants