Skip to content
This repository has been archived by the owner on Jul 16, 2019. It is now read-only.

iOS 10 Support & Fix Transparency #56

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

Conversation

sfrdx
Copy link

@sfrdx sfrdx commented Jul 21, 2016

  • Fixes the iOS 10 permission crash in the Example project
  • Fixes the iOS 10 crash caused by trying accessing an empty array
  • Sets the transparency to work let Apple's default transparency
  • Added new customization options to better provide support similar to Apple toolbar / navigation bars.
  • Removed the default tint / background colors to match Apple's default of nil.
  • Fixed an additional issue where the album view's header did not have a background so the text would just overlap the top of whatever was under it while scrolling.

Was tested on iOS 9.3 iPhone 6 & iPod touch, iOS 10 beta 3 iPhone 6s, and iOS 10 public beta 1 iPad Air

edgesForExtendedLayout was forcing the table/collection view to be lower than the navigation bar preventing transparency from ever working, and the background image / shadow image navigation bar code was creating an issue when the edge layout was removed you could see the content on top of the status bar.
added the backgroundColor for the toolbar and the barTintColor for the navigation bar, since backgroundColor and barTintColor behave slightly differently for how the color of the bar is affected
The apple default is nil and the transparency can be seen much better. The white tint / background actually affects how much the transparency can show through.
…round

The current clear color creates an issue where the text is seen on top of the image collection with no seperation. It would be better to at least match the picker color background so the headers are legible as you scroll.
@sfrdx
Copy link
Author

sfrdx commented Aug 6, 2016

Fixes #54

patelhiren pushed a commit to patelhiren/GMImagePicker that referenced this pull request Aug 8, 2016
Merging changes from
guillermomuntaner#56
Which are not merged with the main branch yet.
@maarek
Copy link

maarek commented Sep 12, 2016

Can we get this merged?

@ndbroadbent
Copy link

ndbroadbent commented Oct 11, 2016

For anyone else who needs the fix right now:

pod 'GMImagePicker', git: 'https://github.com/TechSmith/GMImagePicker.git', branch: 'iOS10-Support'

Thanks @sarafrederixon!

@AnumQ
Copy link

AnumQ commented Oct 19, 2016

@guillermomuntaner can you merge this?

Copy link

@AnumQ AnumQ left a comment

Choose a reason for hiding this comment

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

I am getting an error at this line:
_navigationController.toolbar.tintColor = _toolbarTintColor; (in GMImagePickerController.m)

I am not entire sure if it is caused by this change, however I would like to request a change =)

@sfrdx
Copy link
Author

sfrdx commented Oct 20, 2016

@AnumQ what error are you getting?
Are you getting it in this branch?
What version of iOS are you running on?
Do you mean this line here? https://github.com/guillermomuntaner/GMImagePicker/pull/56/files#diff-ba450c653d6226d8f605ae81b636f289L88
And does it work if you run the example project?

I didn't change this line so curious on what is happening for you.

@AnumQ
Copy link

AnumQ commented Oct 24, 2016

This bug occurred last week and it was gone when I checked again on Friday.
I have been testing on ios 10.

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

Successfully merging this pull request may close these issues.

4 participants