Skip to content

top tabs support - ios#2454

Merged
yogevbd merged 4 commits into
v2from
v2-topTabs
Jan 2, 2018
Merged

top tabs support - ios#2454
yogevbd merged 4 commits into
v2from
v2-topTabs

Conversation

@yogevbd
Copy link
Copy Markdown
Collaborator

@yogevbd yogevbd commented Jan 1, 2018

No description provided.

Comment thread lib/ios/RNNNavigationStackManager.m Outdated
self.fromVC = vc;

if (self.toVC.isAnimated) {
if ([self.toVC isKindOfClass:[RNNRootViewController class]] && self.toVC.isAnimated) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

no longer needed, all VC's conform to protocol

"scale" : "3x"
},
{
"idiom" : "ios-marketing",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove this

DanielZlotin
DanielZlotin previously approved these changes Jan 2, 2018
@yogevbd yogevbd merged commit 06aa587 into v2 Jan 2, 2018
@yogevbd yogevbd deleted the v2-topTabs branch January 2, 2018 14:47
chilinh added a commit to chilinh/react-native-navigation that referenced this pull request Jan 5, 2018
* r_v2: (34 commits)
  Fix typo in README.md
  top tabs support - ios (wix#2454)
  homogenous root view container protocol (wix#2455)
  Fixing stuff (wix#2456)
  Support changing TopTabs font size (wix#2449)
  Implement selected and unselected toTab color (wix#2429)
  added drawUnder (wix#2427)
  Fix failing android e2e (wix#2424)
  Load typface when parsing options (wix#2416)
  V2 Current tab by container ID (wix#2264)
  remove package.lock from npmignore
  V2 rn49 (wix#2411)
  Top tab font family (wix#2410)
  added missing options (wix#2408)
  V2 async commands (wix#2407)
  Tab bar items (wix#2345)
  Added Jenkinsfile
  Added new jenkins file for jenkins pipleine
  [docs] Linkify parameter types (wix#2403)
  [docs] Change heading depth (wix#2401)
  ...
@ManAnRuck
Copy link
Copy Markdown

ManAnRuck commented Sep 5, 2018

@yogevbd nested topTabs doesn't work on iOS, but on android. do you know if it is possible to make it working?

#3769

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants