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

picker.defaultAssetCollection not working on iPad with form sheet presentation style #160

Closed
armandzwan opened this issue Oct 7, 2015 · 7 comments
Labels

Comments

@armandzwan
Copy link

picker.defaultAssetCollection = PHAssetCollectionSubtypeSmartAlbumUserLibrary;

Is not working for me, I'm getting a black popup (with empty navbar)

Tried different types, but only PHAssetCollectionSubtypeAlbumRegular gives me a working picker. But then the user has to tap an album first. This is not what I want.

What am I doing wrong?

screen shot 2015-10-07 at 10 57 34

@1and2papa
Copy link
Owner

Hi @armandzwan, the navigation bar is much thicker than an normal iPad app in your screenshot. It seems that your app was developed as an iPhone only app and run it in the 2x zoom view of iPad? It might be the reason as the picker use iOS 8's size class which Apple suggested for building universal apps.

Please try the demo app see whether it works fine. Thanks.

@armandzwan
Copy link
Author

No it is not. It is an iPad App with just a big Navigation Bar.

@armandzwan
Copy link
Author

Hi,

It is still not working. Can you reopen this?

@1and2papa
Copy link
Owner

Hi @armandzwan, confirmed that this issue only happens on iPad with form sheet presentation style. Will fix it.

@1and2papa 1and2papa reopened this Feb 24, 2016
@1and2papa 1and2papa added the bug label Feb 24, 2016
@1and2papa 1and2papa changed the title picker.defaultAssetCollection not working picker.defaultAssetCollection not working on iPad with form sheet presentation style Feb 24, 2016
@armandzwan
Copy link
Author

Thanks!

1and2papa added a commit that referenced this issue Feb 24, 2016
@1and2papa
Copy link
Owner

@armandzwan Please update to v3.3.1.

@armandzwan
Copy link
Author

Sorry, forgot to inform you it is fixed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants