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

[UWP] Remove Focus call when setting FormsTextBox Vertical Alignment #11350

Merged
merged 9 commits into from
Jul 12, 2020

Conversation

bmacombe
Copy link
Contributor

@bmacombe bmacombe commented Jul 8, 2020

Description of Change

Removes call to Focus added in #11140 that caused regression #11348

#8787 should be reopened as it is no longer resolved with this change.

Issues Resolved

API Changes

None

Platforms Affected

  • UWP

Behavioral/Visual Changes

Entry was getting focus when a page loaded and it should not be. This restores the original behavior.

Before/After Screenshots

Not applicable

Testing Procedure

Check issue pages 2172, 8503, 8222, and then Entry Gallery Focus page

PR Checklist

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

@bmacombe bmacombe changed the title Remove Focus call [UWP] Remove Focus call when setting FormsTextBox Vertical Alignment Jul 8, 2020
@samhouts samhouts added p/UWP 4.7.0 regression on 4.7.0 blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. m/high impact ⬛ t/bug 🐛 labels Jul 8, 2020
@samhouts samhouts requested review from jsuarezruiz and samhouts July 8, 2020 15:20
@bmacombe
Copy link
Contributor Author

bmacombe commented Jul 9, 2020

I looked at the failing UWP regression tests

Bugzilla36171 ran fine locally both with manual verification and running the test
Issue4915 the same

@PureWeen
Copy link
Contributor

PureWeen commented Jul 9, 2020

Does EditorTextDoesNotDisplayNonnumericInput run for you ok @bmacombe ?

@bmacombe
Copy link
Contributor Author

bmacombe commented Jul 9, 2020

@PureWeen Yeah, I ran both tests under 36171, just did them again for good measure.

image

Also I didn't look at the iOS test fails, I assume they aren't related

@PureWeen
Copy link
Contributor

PureWeen commented Jul 9, 2020

Right! you said 36171

Sounds good. We're still working out the kinks on the automated CI runs of UWP :-)

@bmacombe
Copy link
Contributor Author

bmacombe commented Jul 9, 2020

That will be great! I can get individual ones to run ok...on my desktop...but not on either a Surface Laptop 2 I had or 3 I have now, unless I switch out the SearchBar for Entry (first char in the search gets cut off with a SearchBar), then they work 😕 Except for some of the seemly random 'Currently selected window has been closed', timeouts, etc.

@samhouts samhouts added the Core label Jul 10, 2020
@bmacombe
Copy link
Contributor Author

@PureWeen Hope you take some time off this weekend! These test fixes come like clockwork!

@PureWeen PureWeen merged commit 5eaefce into xamarin:4.7.0 Jul 12, 2020
@samhouts samhouts modified the milestones: 4.8.0, 4.7.0 Jul 14, 2020
@bmacombe bmacombe deleted the GitHub-11348 branch July 15, 2020 22:20
@PureWeen
Copy link
Contributor

Hope you take some time off this weekend! These test fixes come like clockwork!

Can't sleep! Tests are failing....
Can't sleep! Tests are failing....
Can't sleep! Tests are failing....
Can't sleep! Tests are failing....
Can't sleep! Tests are failing....

:-p

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
4.7.0 regression on 4.7.0 a/shell 🐚 blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. ControlGallery Core i/regression p/UWP t/bug 🐛
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants