Skip to content

allows you to show a lightbox directly after dismissing on ios#1466

Merged
guyca merged 2 commits into
wix:masterfrom
nicorichard:fix/showLightboxAfterDismiss
May 16, 2018
Merged

allows you to show a lightbox directly after dismissing on ios#1466
guyca merged 2 commits into
wix:masterfrom
nicorichard:fix/showLightboxAfterDismiss

Conversation

@nicorichard
Copy link
Copy Markdown
Contributor

Fixes #462

@ericlewis
Copy link
Copy Markdown
Contributor

+1, this works great!

@chrisward
Copy link
Copy Markdown

Hey guys. Is there any reason this pull request hasn't been approved yet? There seems to be zero negative feedback, it's five months old and it fixes an issue that a number of us are experiencing.

@leoskyrocker
Copy link
Copy Markdown

@guyca Could you take a look if you've got time?

@neatnick
Copy link
Copy Markdown

Works great! I'd love to see this merged in.

@aeke
Copy link
Copy Markdown

aeke commented May 13, 2018

+1 Thanks a lot! My problem solved.

@guyca guyca merged commit 4518797 into wix:master May 16, 2018
adampash pushed a commit to adampash/react-native-navigation that referenced this pull request May 18, 2018
chilinh added a commit to chilinh/react-native-navigation that referenced this pull request May 26, 2018
* r_master: (31 commits)
  Fix syntax error: "screen" is read-only (wix#3237)
  Revert "- Fixes freeze of passProps on iOS (wix#3220)" (wix#3222)
  - Fixes freeze of passProps on iOS (wix#3220)
  allows you to show a lightbox directly after dismissing on ios (wix#1466)
  Enable RTL support on LeftButton (wix#3154)
  Trigger build
  Prevent navBar header background transition in modals, fixes black background in modals navBar
  Link to root of movieapp repo (wix#3112)
  Test [skip ci]
  Revert "Support a “modal” prop for tabbed iOS apps (wix#1578)"
  Revert "fix build issue"
  fix build issue
  fixed a bug modal viewController was not deallocated when to call dismissAllModals (wix#2843)
  Support a “modal” prop for tabbed iOS apps (wix#1578)
  Add iPad landscape launch image (wix#2473)
  Now passing an activity to clearHostOnActivityDestroy (wix#3133)
  Change CI var env to JENKINS_CI
  Avoid NPR in SET (wix#3115)
  Update README.md
  Update README.md
  ...
reimertz pushed a commit to mavencook/react-native-navigation that referenced this pull request Sep 14, 2018
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.

8 participants