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

[UWP] Fix 11946 Entry Clear Button Visibility initial value not honored #11953

Merged
merged 1 commit into from
Sep 7, 2020

Conversation

bmacombe
Copy link
Contributor

@bmacombe bmacombe commented Aug 28, 2020

Description of Change

Clear Button Visibility set before the control is loaded, in code or XAML wasn't being honored.

This change adds updating the visibility based on the ClearButtonVisible state during OnApplyTemplate since that is the first time the clear button could be removed.

Issues Resolved

API Changes

None

Platforms Affected

  • Core/XAML (all platforms)
  • iOS
  • Android
  • UWP

Behavioral/Visual Changes

None

Before/After Screenshots

Not applicable

Testing Procedure

Open issue page 8836 and follow the instructions

PR Checklist

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

@bmacombe bmacombe changed the base branch from main to 4.8.0 August 28, 2020 18:49
@bmacombe
Copy link
Contributor Author

rebased to Target 4.8

@samhouts samhouts added the retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. label Aug 31, 2020
@samhouts samhouts added retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. and removed retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. labels Aug 31, 2020
@bmacombe
Copy link
Contributor Author

bmacombe commented Sep 1, 2020

I looked at the test failure, I don't think it's related.

Info: .NET Core SDK/runtime 2.2 and 3.0 are now End of Life(EOL) and have been removed from all hosted agents. If you're using these SDK/runtimes on hosted agents, kindly upgrade to newer versions which are not EOL, or else use UseDotNet task to install the required version.
[command]"C:\Program Files\dotnet\dotnet.exe" cake --target=cg-uwp-run-tests-ci --verbosity=diagnostic --UWP_TEST_LIBRARY=D:\a\r1\a/build/win_build/UITests/Xamarin.Forms.Core.Windows.UITests.dll
Could not execute because the specified command or file was not found.

@samhouts samhouts removed the retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. label Sep 1, 2020
@rmarinho rmarinho merged commit e2c0d37 into xamarin:4.8.0 Sep 7, 2020
@bmacombe bmacombe deleted the GitHub-11946_ClearButton branch September 7, 2020 13:25
@samhouts samhouts added this to the 4.8.0 milestone Sep 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] ClearButtonVisibility.Never does not take effect on UWP (2)
4 participants