Skip to content
This repository has been archived by the owner on Dec 28, 2023. It is now read-only.

Refactoring CircularShell Renderer #293

Closed
wants to merge 1 commit into from

Conversation

myroot
Copy link
Collaborator

@myroot myroot commented May 6, 2020

Description of Change

Refactoring CircularShellRenderer to use XF.ShellRenderer

  • Need to update XF.ShellRenderer to extend

Bugs Fixed

None

API Changes

None

Behavioral Changes

None

@rookiejava rookiejava marked this pull request as draft May 7, 2020 11:25
@rookiejava rookiejava changed the title [WIP] Refactoring CircularShell Renderer Refactoring CircularShell Renderer May 7, 2020

protected override NavigationDrawer CreateNavigationDrawer(EvasObject parent)
{
return _drawer = base.CreateNavigationDrawer(EvasObject parent);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

            return _drawer = base.CreateNavigationDrawer(parent);

RegisterPropertyHandler(CircularShell.FlyoutForegroundColorProperty, UpdateFlyoutForegroundColor);
}

protected override NavigationDrawer CreateNavigationDrawer(EvasObject parent)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should use alias as below to avoid build error.

using NavigationDrawer = Xamarin.Forms.Platform.Tizen.Watch.NavigationDrawer;
using NavigationView = Xamarin.Forms.Platform.Tizen.Watch.NavigationView;

@rookiejava rookiejava changed the base branch from master to 1.5.0 May 17, 2020 23:52
@rookiejava
Copy link
Contributor

This change is already merged into 1.5.0 through c8641dd.

@rookiejava rookiejava closed this May 17, 2020
@rookiejava rookiejava added this to Done in 1.5.0 Jul 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
No open projects
1.5.0
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants