Merge branch '3.5.0'

Merge branch '3.4.0' into 3.5.0

[Controls] Remove default timeout test #2482

[Android] Avoid acting on a disposed reference (#4780)

[Tizen] Supports SwitchCell.OnColor (#4785)

[C, iOS, AND, UWP] Color for Picker.Title (#4701)

* Implemented Core, iOS and Android

* Implemented UWP

* Processed code review feedback

* More code review feedback
And implemented reset scenarios for all platforms

* Also apply color to popup dialog on Android

- fixes #4669

[XamlC] Resolve generic parameters of a method with generic return type (#4244)

Revert "Properly detect SDK style projects (#4699)"

This reverts commit 36b5345596180a62c732836b7ff20484b76861af.

[IOS] Add launch storyboard (#4724)

* [IOS} Add launch storyboard

* [Mac] Fix csproj

* [Controls] Update MinimumOSVersion iOS

* [Controls] Remove missing file reference

* Update Xamarin.Forms.ControlGallery.MacOS.csproj

[UItests] Fix UITest 41424 (#4745)

* [UITests] Delay for showing picker

* [UITests] Another try to wait for picker opened

Merge branch '3.5.0'

Merge branch '3.4.0' into 3.5.0

[iOS] Update cell size estimates after empty data source (#4688) fixes #4600

* Create gallery to test issue

* Prevent DetermineCellSize loop; allow autolayout to size first cell after
initially empty data source; update estimates after initially empty data source;
fixes #4600

* Automated test

* Fix typo in comments

* Using Ignore to avoid running test because UI tests seems to ignore Explicit

* Change `_previousCount` to `bool _wasEmpty`

update to preview9 and uncomment no longer broken interface (#4727)

[Android] fixes pop-up dialog in Pickers when you call Focus() (#4725)

* [Android] fixes pop-up dialog in Pickers when you call Focus()

* [Android] Extract the popup-on-focus logic
 - the will be useful when other, non-EditText triggers are used
 - this can be extended later if more features are needed
 - no specific types in base types

Add NUnitTestAdapter to xamarin ui test projects so VSIX isn't required for running UI Tests (#4719)

* add NUnitTestAdapter to xamarin ui test projects

* NUnitTestAdapter not needed on ios project

* NUnitTestAdapter not needed on macOS project

delay ui test a bit so elements can vanish first (#4726)

remove armeabi and set HttpClient Implementation on iOS to NSUrlSession (#4717)

* remove armeabi and set  Http Client Implementation to NSUrlSession

* remove armeabi

[Xaml] throw on unexpected empty elements (#4582)

new exception being thrown:
>Xamarin.Forms.Xaml.XamlParseException : Position 7:4. Unexpected empty element '<Grid.ColumnDefinitions/>'

also reduce the number of places we resolve lineInfo for exceptions
purposes

- fixes #4553

Properly detect SDK style projects (#4699)

The `$(MSBuildSDKsPath)` is actually *always* set whenever there are MSBuild SDKs
*installed*, not necessarily when the current project is an SDK-style project itself.

This means that we're potentially adding duplicate items and otherwise disrupting
normal operation of the non-SDK style projects with this import.

So use the well-known properties that are set by the .NET [Sdk.props](https://github.com/dotnet/sdk/blob/master/src/Tasks/Microsoft.NET.Build.Tasks/targets/Microsoft.NET.Sdk.props#L22-L27) file instead,
which is the first imported target even before nuget-provided .props files.

[Android] ItemContentView should use all available space (#4569) fixes #4406

* [Android] ItemContentView should use all available space; fixes #4406
Fix inconsistent/inaccurate class naming for cells/views.

* Update Xamarin.Forms.Platform.Android/CollectionView/SizedItemContentView.cs

Add missing space.

Co-Authored-By: hartez <hartez@users.noreply.github.com>
* Make ReuseId values match class names

Merge branch '3.5.0'

Root the XFBuildTasksLocation path

Merge branch '3.5.0'

revert package downgrade

Updated README to reflect correct name (#4687)

Corrected typo to reflect the correct Android name.

Merge branch '3.5.0'

fix merge mistake

Merge branch '3.5.0'

Merge 3.4.0 to 3.5.0

[build] add a Directory.Build.props (#4672)

Copying <PropertyGroup>s on (almost) every .csproj, and get the
relative paths just right is tedious. Starting msbuild 15 (vs2017)
we can use a Directory.Build.props (and Directory.Build.targets)
so I see no reason why we shouldn't.

see https://docs.microsoft.com/en-us/visualstudio/msbuild/customize-your-build?view=vs-2017

[tasks] Build.Tasks targets net46 (#4670)

* [tasks] Build.Tasks targets net46

Build tasks in VS MUST target net46 and no higher

- fixes #4667

* fix nuspec

[XamlC] Compile the {DataTemplate} markup (#4641)

* [XamlC] Compile the {DataTemplate} markup

- fixes #4592

* [C] throw XPE on missing mandatory member

TypeName for {Type} and {DataTemplate} markup extensions is mandatory.
Throw a XamlParseException, with context, when that happens.

* fix style

* (ab)use of nameof

Update issue templates

Change SearchBar base class to InputView (#3248) fixes #2902

* Changed SearchView base class from View to InputView

* Minor code cleanup

[iOS] Change default background color of Cell to UIColor.Clear (#4002) fixes #3450

* fix https://github.com/xamarin/Xamarin.Forms/issues/3450

* Added possibility to configure cell background color for iOS

* Update Xamarin.Forms.Platform.iOS/Cells/CellRenderer.cs

* Fixed merge suggestion issue

* Update Xamarin.Forms.Core/PlatformConfiguration/iOSSpecific/Cell.cs

Co-Authored-By: AndreiMisiukevich <andrei.misiukevich@gmail.com>
* Update Xamarin.Forms.Core/PlatformConfiguration/iOSSpecific/Cell.cs

Co-Authored-By: AndreiMisiukevich <andrei.misiukevich@gmail.com>
* Update Xamarin.Forms.Core/PlatformConfiguration/iOSSpecific/Cell.cs

Co-Authored-By: AndreiMisiukevich <andrei.misiukevich@gmail.com>

[Build] Don't build Tizen and Design projects on iPhone of iPhoneSimulator (#4646)

* [Build] Don't build Tizen and Design projects on iPhone of iPhoneSimulator

* [Build] Update build

bump gitinfo

GH 4444; Fix local nuget build script (#4668)

Added the style from .NET Foundation plus variances (#4575)

[Android] fixes crash when switching Bottom Navigation Tabs too fast or click on More menu (#4617) fixes #4382

[Shell] expose CurrentShell (#4626) fixes #4625

* [Shell] expose CurrentShell

- fixes #4625

* Update Xamarin.Forms.Core/Shell/Shell.cs

Co-Authored-By: hartez <hartez@users.noreply.github.com>

[test] add passing test for #4572 (#4656)

- closes #4572

Update issue templates

[Android] Fix ImageSource being set to null and fix ImageCell so it loads images (#4601) fixes #4597 fixes #4584

* [Android] fix imageCell and null ImageSource

* [iOS] fixed ui tests for iOS

* [Android] fix FR UI Tests

* suggested formatting fixes

[Android] fixes Picker list showing up incorrectly when focus is set on other controls (#4344) fixes #4318 fixes #4187 fixes #4494

* [Android] fixes Picker list shows up, when focus is set on other controls

* - fixes Picker outside the cell, when you tap on the adjacent EditBox
- address comments

* [Android] added automated UItest

* [Android] added automated test in the carousel

* correct namespace UITest

* fix uitest

* fix uitest #2

* [Controls] Add test for #4494

* minor formating

Merge branch '3.5.0'

Merge branch '3.4.0' into 3.5.0

remove nupkgs (#4633)

* remove nupkgs

* add nupkg to gitignore

* add pdb to gitignore

[Android] Java.Lang.RuntimeException: Animators may only be run on Lo… (#4177)

* [Android] Java.Lang.RuntimeException: Animators may only be run on Looper threads on Battery Saver Mode

https://github.com/xamarin/Xamarin.Forms/issues/4176

* rewrite code to BeginInvokeOnMainThread
- fixes #4176

[Enhancement] Add drag started and completed events/commands to Slider (#4423) fixes #1450

* Implement DragStarted and DragCompleted events/commands in Slider.

* Add Tizen support.

* Use explicit interface implementation.

* Fix unit tests.

* Update Xamarin.Forms.Core/ISliderController.cs

Co-Authored-By: velocitysystems <matthew@velocitysystems.co.nz>
* Fix whitespace.

* Add missing namespace import

[Xaml] expose XmlnsPrefixAttribute (#4562)

Expose the XmlnsPrefixAttribute, that can be used by external tools to
support object tree serialixation to Xaml.

- fixes #3191

[Android] Crash at Dependency service call possible fix (#4157)

* https://github.com/xamarin/Xamarin.Forms/issues/4097 possible fix

* renamed lock object

* updated DS and sample

* merge fix

- fixes #4097

Ignore context change to null on UAP listview (#4412) fixes #4373

[XamlC] test for null in Binding paths (#4521) fixes #4102

* [XamlC] test for null in Binding paths

Instead of relying on a NRE behing thrown while trying to evaluate a
compiled binding getter, detect the null value and fail faster, without
throwing. This require changing the type of the getter so they return a
(TProperty, bool) value tuple indicating the success.

- fixes #4102

* depend on ValueTuple for ns1.0

[UWP] Fixes throws NullReferenceException on empty NavigationPage (#3325) fixes #2102

[Shell][iOS] do not try to smartly unblend bgcolor (#4593)

Unblending bg color was a smart idea. But only works in a controlled
subset of cases. To avoid unexpected blends, we will do the dumb stuff,
as we always do.

[tasks] avoid passing dupe assemblies to Cecil (#4624) fixes #4620

* [tasks] avoid passing dupe assemblies to Cecil

avoid feeding Mono.Cecil with dupes. Removes a negligeable handful
of milliseconds (100ms on average, hard to measure) from XamlC. Even
less from XamlG.

- fixes #4620

* add extra check

[Enhancement] Added GroupHeaderStyle property for iOS specific ListView class (#4616)

* Added group header style property for ios specific list view class

* removed redundant test line

[build] do not fail on multiple core rebuild (#4621) fixes #4619

- fixes #4619

remove whitespaces, makes git happier

[Shell] Allow queryParams navigation (#4615)

* [Shell] Allow queryParams navigation

Allow uri navigation with queryParams. queryParams can be applied
to:
- the ShellContent (if inherited)
- the Content (or inflated ContentTemplate) of the ShellContent
- the BindingContext of the Content

those 3 can inherit an interface and handle the queryParams, or
the types can be decorated with some [QueryProperty] attributes.
Attributes are handled on the 3 typesL ShellContent, actual content,
or bindingcontext.

- fixes #4596

* syntax errors

* Address formatting comments

* [Core] add check when setting BindingContext on page to self

* [Core] fix navigation for non templated item

[Shell] allow missing trailing slash on uri (#4595)

[C] Expose Application.LogWarningsListener (#4539) fixes #1517 fixes #3873

* [C] Expose Application.LogWarningsListener

The fix proposed in #2911 didn't worked with Release build of XF, aka
the majority of the builds used.
This fix allows the developer to set any logger he might like, but the
plan is to add a default one in the template, that uses Debug.WriteLine,
so it will Log if the user application is build with Debug, and not if
XF binaries are.

- fixes #1517
- fixes #3873

* Update Xamarin.Forms.Core/BindableObject.cs

* Update Xamarin.Forms.Core/ListView.cs

[UWP] Rename ConvertExtensions to ColorExtensions and change access to public (#4558) fixes #1522 closes #1535

* Rename ConvertExtensions to ColorExtensions and change access to public

- fixes #1522
- closes #1535

* remove unused convertextensions

[iOS] Fixed keyboard type for cells in TableView (#4377) fixes #1386

* fix https://github.com/xamarin/Xamarin.Forms/issues/1386

* Added test case

* Added test case

* fixed build

[Tizen]Update style for Phone Idiom in ImageCell renderer (#4610)

Add custom tabs to MonoAndroid90 nuspec to improve essentials experience (#4605)

Merge branch '3.5.0'

Merge branch '3.4.0' into 3.5.0

[Core] remove code setting NavProxy to null (#4604)

[iOS] Recover from popping page after the page has already been popped by back button (#2873)

* Cleanup navigation task on Pop exception

* Bugzilla 59172 + recovery repro tests

* Trap for Pop race

* [Core] add check on OnPopToRootAsync

* Added test scenario description
- fixes #4129

[iOS] ImageButton border radius issue (#4518)

* fix https://github.com/xamarin/Xamarin.Forms/issues/4511

* Added test-case
- fixes #4511

[build] add a stamp file to shamehat (#4591) fixes #4563

In addition to #4579 that check, in the tool, that the extra
method isn't present, this protects the task itself from being ran
twice.

Both fixes are required for extra performance and safety.

This also removes the last reference to the original name of the tool.
But I'll still remember it fondly, and use it in internal conversations.

[build] prevent ShameHat to run twice (#4579)

- fixes #4563

apply the increment to all ctors

Merge branch '3.4.0' into 3.5.0

[Shell] The glorious Shell (#4522)

* [Shell] The glorious Shell

* set the flag for the test

* remove the FastRenderer flag

* [Android] If we were disposed earlier Element can be null and Control was disposed

* [Controls] Seems part of #2520 was solved so we don't need this

* [iOS] Fix NRE on the ScrollviewTracker

* add missing constructor for RenderWith

* [Core] propagate visual/flow through shell

* use interlock increment

* hide IDataTemplateController

* [Android] Fix TableViewModelRenderer dispose

* [Core] propagate visual/flow direction through shell hierarchy

* [Android-Gallery] removed some left in settings changes that effect UI Tests

* [iOS,Droid,Core] Code formatting

* [iOS,Core] Last minute style refactoring

[UWP] add missing xbf files to nuspec (#4588)

* [UWP] add missing xbf files to nuspec

* [UWP] fix target for ItemsListView

Set ImageButton property of Android.ImageButtonRenderer public (#4534)

* Set ImageButton property public
Set ImageButton property public.
* Rename ImageButton to Element
the Element set property remains private.

Fix initial load of pins (#4304)

 [iOS] Clear the delegate from ContextActionCell  (#4324) fixes #4314

* [Controls] Add repo for issue #4341

* [iOS] Clear the global closer delegate from ContextActionCell when disposing

* [Controls] Rename to correct issue number

* [Controls] Try fix opening context menu on iOS10

* [Controls] Add more testing to #4314

* [iOS] Fix Globalcloser when no cell exists

[Core]Call invalidate measure after the FormattedText is set (#4448) fixes #4026

* [Controls] Add repo for for issue #4026

* [Controls]Add instructions for issue #4026

* [Core] Hook FormattedTextChanging

* [Core] Invalidate measure after the renderer updates the Control's text

* [iOS] Make sure to clear perfectSize on Font property changing

* [iOS] Fix nre when accessing old or new element

removing a helper introduced in #2782

GH2691: Make XmlnsDefinitionAttribute public (#2782)

* Make XmlnsDefinitionAttribute public. Change XamlC, XamlG, and XamlParser to search for all referenced/available assemblies for attribute and referenced types. Add XAML unit tests and control gallery entry.

* change ns for xmlnsdefattribute

- fixes #2691

[Android] total fix of issue #3575 (#4567)

* total fix of issue #3575

we need to check _childView with null also

* Update MasterDetailContainer.cs

Modify Android and iOS adapters to handle null items sources (#4536) fixes #4366

* Modify Android and iOS adapters to handle null items sources; fixes #4366

* Correct behavior for EmptySource indexer on iOS

update support and sync up nuspecs (#4571)

Fix edge case in unsubscribing weak events (#4538)

When an event has two subscriptions to the same handler,
unsubscribing cleared all the handlers, not only one.

 Implemented colored refresh indicator for ListView pull-to-refresh (#2961)

* Implemented colored spinner for ListView pull-to-refresh

* Added back default spinner color

* Renamed away from "spinner" and did some default value magic

Color.Default reinstated, but also added the default black color in the iOS `UpdateRefreshControlColor` method. Seeing this also happens for the seperator color this is something that is necessary. If not, the spinner is invisible.

* Implemented default color on iOS and right property name on Android

For iOS, when the new `RefreshControlColor` has the default color, the `TintColor` for iOS is set to null, causing the color to be the default one.

Forgot to refactor the new name in the Android renderer, whoops

* Reinstated coloring on listview gallry page

* Implemented colored spinner for ListView pull-to-refresh

* Added back default spinner color

* Renamed away from "spinner" and did some default value magic

Color.Default reinstated, but also added the default black color in the iOS `UpdateRefreshControlColor` method. Seeing this also happens for the seperator color this is something that is necessary. If not, the spinner is invisible.

* Implemented default color on iOS and right property name on Android

For iOS, when the new `RefreshControlColor` has the default color, the `TintColor` for iOS is set to null, causing the color to be the default one.

Forgot to refactor the new name in the Android renderer, whoops

* Reinstated coloring on listview gallry page

* Added documentation for new property

* Fixed merge boo-boos

[Android] Handle accessibility importance when using modal pages (#4404) fixes #3622

* [Controls] Add repo for issue #3622

* [Android] Handle accessibility importance when using modal pages

* [Controls] Use default styles

* Update Xamarin.Forms.Platform.Android/AppCompat/Platform.cs

Co-Authored-By: rmarinho <me@ruimarinho.net>

[C] Seal our attributes (#4559)

Follow CA1813 guidance. And seal our attributes.

- fixes #1788

[binding] return defaultValueCreator on failure (#4541)

if the FallbackValue isn't set, returns the value created by the
defaultValueCreator instead of the DefaultValue when a binding can not
be resolved.

- fixes #2752

Fix Xaml Native Views Gallery on Android when Linker is enabled

[UWP] Prevent Layouts with no background color from being input transparent (#4508)

* [UWP] Prevent Layouts with no background color from being input transparent; fixes #4360;

* Add comments

* Fix build for UITests projects

[UWP] fixes control render by ViewToRendererConverter in off screen (#4532) fixes #4463 fixes #4116

Revert "[Design] Design assemblies targets v4.5 (#4560)  fixes #2216"

This reverts commit 6fa2c1708e1508a016d4a4332f12a0f0d832d8bb.
This reverts commit 11fd3ee9e55475f9017eb35c2b26018082122e94.

[Design] missing one

adapt CODEOWNERS

[Design] Design assemblies targets v4.5 (#4560)  fixes #2216