Merge branch '3.5.0'

[UITest] Add Preserve to issue #1931 uitest

Null/disposed check for spannableString to fix ObjectDisposedException. (#4878) Fixes #4864

Add instructions to test 33450

fixes #2246

[UITests] Fix #1685 UITest

[ios] clear perfect value add ui test (#4968)

- fixes #4957

Merge branch '3.5.0'

[Build] Fix method accessibility

[iOS, Android] Implement Snap alignment for CollectionView (#4414)

* Implement snap alignment for iOS CollectionView

* Add missing End/MandatorySingle implementation for Android

* Implement MandatorySingle snapping on iOS

* Fix issues with Android MandatorySingle skipping items;

* Fix rebase issue

fix merge

fix merge

Disable suggestions for all Pickers (#4824) fixes #4629

* Disable suggestions for all Pickers

* Removed redundant disabling of spellcheck

[iOS] Fix unsubscribing the `PropertyChanged` event of the ViewCell (#4728)

* [Controls] Add repo for comment related with #4186

* [iOS] Fix unsubscribing PropertyChanged when disposing ViewTableCell

* [Controls] Fix test

Changes that VS Mac and Dev16 automatically make when you open SLN (#4969)

* vsmac changes these when you open it

* dev 16 changes

Merge branch '3.5.0'

[Android] Fix brief display of incorrect keyboard (#4384)

* Automate test for 39072

* Add Focus test category

* Add test instructions

* Use named flags value instead of magic number

* Rearrange keyboard hide/show handling when focus changes; fixes #3647

Clean up usings

Add null checks in case Controls are unloaded before the handler executes

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

Co-Authored-By: hartez <hartez@users.noreply.github.com>
* Update Xamarin.Forms.Platform.Android/ViewRenderer.cs

Co-Authored-By: hartez <hartez@users.noreply.github.com>
* Fix bug where programmatic focus of SearchView does not display keyboard;
Add test for programmatic focus of SearchBar, Entry, Editor;

* Prevent object disposed exception if control is disposed before posted ShowKeyboard is run

Merge branch 'easy-tests' into 3.5.0

Automate test for Issue 1700

Automate test for Issue 1685

Automate test for Issue 1590

Automate test for Issue 1557

Auotmate test for Issue 1538

[tests] UWP requires file extensions (#4944)

fix #4788 (#4959) fixes #4788

[macOS] BoxView binding to BackgroundColor broken (#4958)

- fixes #4338

[tests] UWP requires file extensions (#4945)

[UITests] Ignore test failure on iOS9, use manual review

[Android] fixes Right-to-Left Hamburger icon in MasterDetailPage (#4716) fixes #2818

Map with ItemsSource and ItemTemplate (#4269) fixes #1708

[Enhancement] Entry: Read-only Entry and Editor (#1972)

- fixes #1678
- fixes #4834

* InputView BindableProperty IsReadOnly

* Android Editor and Entry setup

* MacOS Editor and Entry setup

* Tizen Editor and Entry setup

* UAP Editor and Entry setup

* WPF Editor and Entry setup

* iOS Editor and Entry setup

* Issue1678 - TestContentPage added

* Tests added

* Tizen Editor and Entry fix

* UI Test fix

* Android fix for Entry and Editor.

* Unit Tests fix.

* Android Renderer first try.

* MacOS Entry renderer small fix.

* UpdateEditable fix.

* Android Entry and Editor reduced number of calls to BP.

* fix TestAttributes conflicts

* Remove keyboard

* [macOS] relinquish first responder

* [Android] remove call to UpdateCursorSelection

* [UWP] fix tabs

[Android] Fix for ListViewRenderer causing ObjectDisposedException (#4931)

* [Android] Fix for ListViewRenderer causing ObjectDisposedException

* Probably cleaner to use _refresh for IsDisposed check

[UITests] Disable test #1931 on iOS12 for now

hide obsolete API from editor (#4938)

- fixes #4772

Only post DisableTimer to the queue if it's not on the UI thread (#4866)

Fix incorrect bug description

[iOS] CollectionView EmptyView (#4870)

* Implement EmptyView for iOS

* Allow for changes to EmptyView on iOS; rename confusing variables;

[Android] If drawable doesn't exist don't dispose of it (#4904)

[Android] Ensure the page background can be un-set (#4916)

added case in GTK listviewrendered for change of SelectedItem property (#4757)

Material Design Renderers for the Material Visual (#4733)

* Use the auto-generated constants

* [Visual] Work on the Material Frame

* Improving the code for the sample

* Added a controller to help with frames that have additional padding
 - Android MaterialCardView does not use the default padding to determine where the content starts, rather it uses the content padding of the view because there is a border that does not affect the content

* [Visual] Added a few extra checks on Android to reduce unnecessary interop

* [Visual] A few more frame changes and some button tweaks

* [Visual] use the themers on iOS and save default properties

* [Visual] Add placeholders for themer and cache defaults

* [Visual] Added the material slider for iOS
 - Android does not have a custom control, so uses the existing renderer

* [enhancements] Move from duplicate LoadImageAsync code to GetNativeImageAsync

* [visual] Updated the controls to use the new iOS bits

* [visual] some fixes for material components

* [visual] Added hacks for material alerts

* [visual] removing the alert changes for the main branch

* [visual] Update the MaterialComponents NuGet

* [visual] Rework the theming/customization of Material controls on iOS

* [visual] fix the places where the user colors were being changed

* [visual] Improve the ColorStateList management for Android

* [visual] Re-implemented the Android ProgressBar as a fast, material renderer

* Material Entry

* [visual] Add Android ActivityIndicator

* filter out material layouts for 8.1

* remove folder

* fix __ANDROID_28__

* MaterialContextThemeWrapper

* [visual] remove the `IFrameController` interface

* [visual] reverting the changes to the Frame layout

* [visual] reverting whitespace

* [visual] make sure to raise both property changed

* formatting changes

* fixing colors on android to match with ios themes

* Update Xamarin.Forms.Platform.Android/Material/MaterialButtonRenderer.cs

Co-Authored-By: mattleibow <mattleibow@live.com>
* fix sizing of entry with infinite width size request

* update to release 28 of support

* Update Xamarin.Forms.Platform.Android/Resources/values/styles.xml

Co-Authored-By: PureWeen <shane94@hotmail.com>
* Update Xamarin.Forms.Platform.Android/Resources/values/styles.xml

Co-Authored-By: PureWeen <shane94@hotmail.com>
* PR Comment changes

Use PushAsync for 1305 UI test so the toolbar shows up

[iOS] Implement item move on iOS CollectionView (#4863)

* Implement item move on iOS CollectionView

* Remove private modifier

Merge branch '3.5.0'

Add missing preserve attribute for UI test 1267

Fix test 852

Merge branch '3.5.0' of github.com:xamarin/Xamarin.Forms into 3.5.0

Automate UI test 1267

Automate UI test 1236

Automate UI test 1219

Add manual test instructions for 1075, match button text with actual color

Automate UI test 1028

Add manual instructions to UI test 1026

Add manual test run instructions for UI test 1025

Clean up old comments

Add instructions for test 764; remove automated test that did nothing;

Min viable FontImageSource (#4747)

* min viable fontImageSource

* [UWP] Min viable FontImageSource (#4817)

* [UWP] font image source

* address comment

* [UWP] performance optimization of the rendering of icons font
- the minimum Dpi of the icon is 300

* char -> string; CR feedback

Make test button visible on UWP

Remove nonsensical comments

Add instructions to manual UI test

Implement Android single selection in CollectionView (#4651)

* Implement Android single selection

* Fix naming conflicts with SelectionChangedEventArgs

* Split files by class; make SelectionChangedEventArgs constructors internal;
Optimizations for SelectionChangedEventArgs;
Fire selection changed event/command from property changed handler;

* Update Xamarin.Forms.Core/Items/SelectableItemsView.cs

Co-Authored-By: StephaneDelcroix <stephane@delcroix.org>
* Set default value for SelectedItemProperty

* Remove unnecessary null check

* Fix order of OnElementChanged and Setup/Teardown Element

* Fix project file weirdness

[XamlC] Test generic parameters to evaluate equality of types (#4062)

Make ListView CachingStrategy property visible to Intellisense (#4846)

[iOS] Added RowAnimationsEnabled platform specific to control row animations (#4709)

* added platform specific

* changed method signatures

* addressed changes

[Shell] fix queryParam navigation (#4843)

when navigating to pages, not shell-section, handle the queryParams

- fixes #4837

[Tizen] Supports Picker.TitleColor, FontSize, FontFamily and FontAttributes (#4839)

[Flex] fix Padding on the FlexLayout (#4823)

As the flex engine doesn't handle x,y offset on the root layout,
handle that at the XF.FlexLayout level.

- fixes #2663

[*] Drop MenuItem.IsEnabledPropertyName (#4659)

* [*] Drop MenuItem.IsEnabledPropertyName

While doing one of the big refactoring of 2006, we missed an
opportunity to use a propertyKey for IsEnabled, ending up in some
less-than-optimal  design choices, like IMenuController.IsEnabledPropertyName.

Properly using a public r-o BP removes the need for IsEnabledPropertyName.

* other plats

* use the key

Remove unnecessary OK button check that won't pass on older Android versions

XmlnsPrefix d: (#4781)

be ready for when we'll merge #4743

[X] Design time properties (#4743)

An alternate xaml resource file provider can request the XamlLoader to
_not_ ignore normally ignored properties in prebuilt XF design xmlns, as
in the following snippet:

```xaml
<ContentPage
    xmlns="http://xamarin.com/schemas/2014/forms""
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml""
    xmlns:d="http://xamarin.com/schemas/2014/forms/design""
    xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006""
    mc:Ignorable="d"">
  <Label d:Text="Bar" Text="Foo" x:Name="label" />
</ContentPage>
```

The `d:` should be the prefix used by default for this, but any other
prefix will do too.

The `d:Text` property maps to the exact same property as `Text`, as the
XmllnsDefinitionAttributes are identical (that's convenient from a Intelisense
point of view), but, when (and only when) instructed by a provided Xaml resource
loader, the `d:Text` will override the `Text` property.

This works with virtually all properties defined on built-in Xamarin.Forms
controls, but it doesn't mean it's a sane idea to try to assign design value
to all existing properties.

The APi ofr setting the ResourceLoader had to change, and instead of taking
pre-defined arguments, it accepts and returns query and response types. This
is slightly less convenient to invoke through reflection, but way more easy
to extend in the future.

[Android] set ClipBounds to use ViewCompat.SetClipBounds (#4819)

[Xaml] require escaping curly braces (#4723)

This is a breaking change as we weren't throwing errors on that before.

- fixes #2756

[XamlC] instantiate generic parameter on base types (#4776)

* [XamlC] Instantiate generic parameters in base type generic arguments

* Add unit test for #4047

- fixes #4760
- closes #4047

[C] WeakEventManager minor changes (#4774)

As pointed in https://github.com/xamarin/Xamarin.Forms/issues/4750#issuecomment-447673540,
internal WeakEventManager usage can be simplified.

This also updates the code to ValueTuple, throw expressions, ...

[Android] Check if Drawable is null in Draw override (#4783)

- fixes #4782

[X] throw XPE on missing element .ctor (#4786)

* [X] throw XPE on missing element .ctor

In case of runtime parse, wrap the MissingMethodException in a
XamlParseException, to get lineInfo. In case of XamlC, detect missing
constructor and fail early.

- fixes #4751

* fix for ns1.0

[XamlC] Always enable implicit casting, boxing and unboxing (#4238)

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