Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add attribute to show page on iOS #3339

Merged
merged 7 commits into from May 6, 2019

Conversation

martijn00
Copy link
Contributor

@martijn00 martijn00 commented Mar 25, 2019

✨ What kind of change does this PR introduce? (Bug fix, feature, docs update...)

Feature

⤵️ What is the current behavior?

The way to show a viewcontroller page inside a UIPageViewController doesn't use the new presenter system.

🆕 What is the new behavior (if this is a feature change)?

Now you can easily do it with the attributes

💥 Does this PR introduce a breaking change?

Yes

🐛 Recommendations for testing

WIP

📝 Links to relevant issues/docs

Fixes #3230
Closes #1403

🤔 Checklist before submitting

@martijn00 martijn00 requested a review from a team March 25, 2019 16:03
Cheesebaron
Cheesebaron previously approved these changes Mar 26, 2019
MvvmCross/Platforms/Ios/Presenters/MvxIosViewPresenter.cs Outdated Show resolved Hide resolved
@Cheesebaron
Copy link
Member

Could we add a sample for this in the Playground?

@martijn00 martijn00 requested a review from Cheesebaron May 6, 2019 12:43
@martijn00 martijn00 added this to the 6.2.4 milestone May 6, 2019
@Cheesebaron
Copy link
Member

Ran the sample, looks pretty good. Nice and easy to use API. 👍

@Cheesebaron Cheesebaron added the p/ios iOS platform label May 6, 2019
@Cheesebaron Cheesebaron merged commit 8382e53 into MvvmCross:develop May 6, 2019
@martijn00 martijn00 modified the milestones: 6.2.4, 6.3.0 May 13, 2019
@Cheesebaron Cheesebaron added the t/breaking Breaking Change type label May 16, 2019
@martijn00 martijn00 deleted the pageviewcontroller branch May 20, 2019 15:16
@markuspalme markuspalme mentioned this pull request May 21, 2019
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p/ios iOS platform t/breaking Breaking Change type
3 participants