Skip to content
This repository was archived by the owner on Feb 15, 2019. It is now read-only.

feature: added scroll and tabWidth props#325

Open
JohnClema wants to merge 1 commit intoexpo:masterfrom
JohnClema:master
Open

feature: added scroll and tabWidth props#325
JohnClema wants to merge 1 commit intoexpo:masterfrom
JohnClema:master

Conversation

@JohnClema
Copy link

Adds the ability to have scrollable tabs and to change the width of tabs

@brentvatne
Copy link
Member

cc @satya164

};

type Props = {
scrollEnabled?: boolean,
Copy link
Contributor

@satya164 satya164 Dec 22, 2016

Choose a reason for hiding this comment

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

I think better to name it as tabBarScrollEnabled or something since you're passing it to TabNavigator. Any suggestions for a better name @brentvatne ?

Copy link
Member

Choose a reason for hiding this comment

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

sounds like a good name to me!

Copy link
Contributor

Choose a reason for hiding this comment

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

@JohnClema can update the PR to use this name instead?

Copy link
Author

Choose a reason for hiding this comment

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

@satya164 made the changes - let me know if you think there should be any more

@satya164
Copy link
Contributor

satya164 commented Jan 1, 2017

Looks good to me.

@JohnClema
Copy link
Author

JohnClema commented Jan 3, 2017

@satya164 @brentvatne will I need to put this into a separate branch for it to be merged?

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.

3 participants