Skip to content

PR for #29 - Implement common page/navigation model#30

Merged
tom-made merged 15 commits intomasterfrom
reshuffle
May 21, 2018
Merged

PR for #29 - Implement common page/navigation model#30
tom-made merged 15 commits intomasterfrom
reshuffle

Conversation

@jamesmcroft
Copy link
Copy Markdown
Member

Change Title

Added common logic for application pages across Windows and Android.

Changes include a base page class for Windows Page and Android Support Fragment, a base navigation frame for Windows Frame and Android Activity, and a common navigation service to handle the navigation across both platforms.

Associated Issues

#29 - [Feature] Implement common page/navigation model

@jamesmcroft jamesmcroft requested a review from tom-made May 21, 2018 15:43
Copy link
Copy Markdown
Contributor

@tom-made tom-made left a comment

Choose a reason for hiding this comment

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

Just a few minor changes required.

@@ -0,0 +1,37 @@
#if WINDOWS_UWP
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No documentation here

Comment thread CODE_OF_CONDUCT.md
@@ -1,46 +0,0 @@
# Contributor Covenant Code of Conduct
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We probably still need this file

Copy link
Copy Markdown
Contributor

@tom-made tom-made left a comment

Choose a reason for hiding this comment

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

Thanks, approved!

@tom-made tom-made merged commit 8306489 into master May 21, 2018
@tom-made tom-made deleted the reshuffle branch May 21, 2018 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants