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

(iOS) Added preferences for privacy on background only and overriding LaunchImage #42

Open
wants to merge 26 commits into
base: master
Choose a base branch
from

Conversation

MobilityTiago
Copy link

@MobilityTiago MobilityTiago commented Feb 24, 2017

Description

In iOS activating privacy in WillResign method generates a behavior that isn't desirable in current project I'm working on (i.e. Shows privacy screen in Touch ID prompt), as such a preference "PrivacyOnBackground" is now checked, if it is true then privacy screen is only activated on background.
Another preference "PrivacyOverrideLaunchImage" has been added so that even if a launch image is set, if this preference is true it overrides to the "Default" image.

Related Issue

#33

Motivation and Context

On our app that uses this plugin, when the Touch Id Prompt was launched the privacy screen blocked the app view.
The app has a launch image, but it has been defined in our app that the privacy screen must be different

How Has This Been Tested?

Manual testing to check if intended behavior was added, manual testing to check if previous behavior was kept without changes to current users of the plugin.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • [X ] New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • [X ] My code follows the code style of this project.
  • [X ] My change requires a change to the documentation.
  • [X ] I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@devgeeks
Copy link
Owner

I would have liked the fix for the Touch ID issue, but wish it hadn't been wrapped up with another feature. Maybe you could find a way to make this two different PRs?

MobilityTiago and others added 9 commits December 15, 2017 17:23
Added screen sizes for 2x and 3x density using system like splash screen (launch screen)
Added iPhoneX to device struct
Removed '-' from calculated privacy screen images names;
Support for iPhoneX screen size;
Removed support for Launch Screen Storyboard;
Update PrivacyScreenPlugin.m
Update PrivacyScreenPlugin.h
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.

3 participants