Skip to content

Further work on TopTabs and some refactoring#2362

Merged
guyca merged 4 commits into
v2from
TopTabsAndroid3
Dec 18, 2017
Merged

Further work on TopTabs and some refactoring#2362
guyca merged 4 commits into
v2from
TopTabsAndroid3

Conversation

@guyca
Copy link
Copy Markdown
Collaborator

@guyca guyca commented Dec 17, 2017

  • OptionsPresenter receives views as params
  • Remove TopBar and style logic from ViewControllers, these are the OptionsPresenters concern
  • StackAnimator isn't coupled to LinearLayout
  • Convert Runnable to lambda
  • ContainerLayout now implements ReactContainer interface
  • Enable desugaring in playground app
  • Cover top tabs with unit tests

* OptionsPresenter receives views as params
* Remove TopBar and style logic from ViewControllers, these are the OptionsPresenters concern
* StackAnimator isn't coupled to LinearLayout
* Convert Runnable to lambda
* ContainerLayout now implements ReactContainer interface
* Enable desugaring in playground app
@guyca guyca added the v2 label Dec 18, 2017
@guyca guyca requested review from Cool04ek and DanielZlotin and removed request for DanielZlotin December 18, 2017 08:25
@guyca guyca assigned Cool04ek and unassigned DanielZlotin Dec 18, 2017
@guyca guyca merged commit 1383915 into v2 Dec 18, 2017
@guyca guyca deleted the TopTabsAndroid3 branch December 18, 2017 10:32
chilinh added a commit to chilinh/react-native-navigation that referenced this pull request Dec 21, 2017
* r_v2: (75 commits)
  Topbar (wix#2382)
  V2 background Image options (wix#2375)
  async await for push and showModal (wix#2270)
  Temporarily disable failing e2e test (wix#2377)
  navigation bars e2e with testIDs (wix#2365)
  V2 drawUnder for topBar and bottomTabs (wix#2376)
  xcpretty was swallowing test errors
  back to normal
  back to normal
  try fix travis
  Setup TopTabs with ViewPager (wix#2373)
  try fix travis
  try fix travis
  Further work on TopTabs and some refactoring (wix#2362)
  try fix travis
  try fix travis
  try fix travis
  try fix travis
  try fix travis
  another try
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants