Skip to content

Android bootstrap, NavigatorIOS -> Navigator throughout#4

Merged
AndrewHamm merged 13 commits intomasterfrom
android-bootstrap
Nov 2, 2015
Merged

Android bootstrap, NavigatorIOS -> Navigator throughout#4
AndrewHamm merged 13 commits intomasterfrom
android-bootstrap

Conversation

@gregziegan
Copy link
Copy Markdown
Member

all of these changes do not break any iOS functionality (well, we don't have tests so my metric is via manual navigation).

I can break apart some of the index.ios.js code.

Also, this new system means that there may no longer be any need for screens. Each tab now contains a navigator and the navigation bar is optional/customizable with each tab.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Fine for now, in the future I will probably get rid of/change this since everything is not necessarily a show

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.

I was under the impression that show was the blanket term for either movie or TV show.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Then this would make sense. Maybe "Reel" instead of show ?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@BillyBarbaro it may be ambiguous but no more ambiguous/confusing than labeling a movie as a show. Plus, in the design mocks this page doesn't even have a header so it would be just for internal naming. And also then it would allow us to have more specific names for attributes like movies and shows which may contain different information. For example shows may list info about seasons and episodes while movies will not.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

also just saw Greg's comment about lolore so we have already kind of decided to call stuff movies by choosing to use lolomo.... hmmmm something to think about

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@thebritican I think I see what you mean now

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Ya i did like 2-3 hours of reading and coding while watching 30 Rock. So I was a bit distracted but I definitely think I'm closer to the right solution.

AndrewHamm added a commit that referenced this pull request Nov 2, 2015
Android bootstrap, NavigatorIOS -> Navigator throughout
@AndrewHamm AndrewHamm merged commit 2f349e5 into master Nov 2, 2015
@AndrewHamm AndrewHamm deleted the android-bootstrap branch November 2, 2015 17:28
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.

3 participants