[Shell][Tizen] Add the MaterialShellRenderer on Tizen #6196
Conversation
/AzurePipelines run |
No pipelines are associated with this pull request. |
/AzurePipelines run |
No pipelines are associated with this pull request. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think hard copy the duplicate source code from existing renderer (Xamarin.Forms.Platform.Tizen
) to material renderer(Xamarin.Forms.Material.Tizen
) is better for the sake of maintenance. And the screenshot should also be updated.
@@ -9,7 +9,7 @@ | |||
<PackageReference Include="Tizen.NET" Version="4.0.0"> | |||
<ExcludeAssets>Runtime</ExcludeAssets> | |||
</PackageReference> | |||
<PackageReference Include="Tizen.NET.MaterialComponents" Version="0.9.5-preview" /> | |||
<PackageReference Include="Tizen.NET.MaterialComponents" Version="0.9.6-preview" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use 0.9.7-preview
which is the latest version.
it would be better #6204 precede this PR. |
/azp run |
No pipelines are associated with this pull request. |
@yourina do you mind rebase? thanks |
rebase please. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test done.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Build failed:
Xamarin.Forms.Platform.Tizen\Shell\NavigationView.cs(102,11): Error CS0618: 'MenuItem.Icon' is obsolete: 'Icon is obsolete as of 4.0.0. Please use IconImageSource instead.'
--
| Xamarin.Forms.Platform.Tizen\Shell\NavigationView.cs(104,15): Error CS0618: 'MenuItem.Icon' is obsolete: 'Icon is obsolete as of 4.0.0. Please use IconImageSource instead.'
Fixed it. Thank you. |
Build still failing https://dev.azure.com/xamarin/public/_build/results?buildId=5692&view=results |
067adde
to
f80a57e
Compare
Rebased. |
* [Shell][Tizen] Add the MaterialShellRenderer on Tizen * Fixed the build fail * Update namespace
* [Shell][Tizen] Add the MaterialShellRenderer on Tizen * Fixed the build fail * Update namespace
Description of Change
This PR is for adding
MaterialShellRenderer
on Tizen.Issues Resolved
None
API Changes
None
Platforms Affected
Behavioral/Visual Changes
None
Before/After Screenshots
Testing Procedure
Use
StoreShell
PR Checklist