Skip to content

[modularization] Re-export react-native-tableview-simple and put our custom cells into a module#2895

Merged
rye merged 3 commits into
masterfrom
reorg-take6-part5.12
Aug 27, 2018
Merged

[modularization] Re-export react-native-tableview-simple and put our custom cells into a module#2895
rye merged 3 commits into
masterfrom
reorg-take6-part5.12

Conversation

@hawkrives

Copy link
Copy Markdown
Member

Two things:

  1. Moves the "components/cells" custom TableView cells into a module, @frogpond/tableview.
  2. Re-exports the TableView, Section, and Cell components from react-native-tableview-simple

I'm aiming to get to where we can design with TableView and style appropriately for Android/iOS, and I think that @frogpond/tableview is where we should expose that to our app.

Also, it'll let us have one place to set defaults for the app, like setting the Section's background color.

Neither of those things have yet been implemented.

Part of the great #1537 redo.

@hawkrives hawkrives self-assigned this Aug 26, 2018
@hawkrives hawkrives requested review from drewvolz and rye August 26, 2018 21:42
drewvolz
drewvolz previously approved these changes Aug 26, 2018
rye
rye previously approved these changes Aug 26, 2018
@rye rye dismissed stale reviews from drewvolz and themself via a18d3b4 August 26, 2018 22:39
@rye rye force-pushed the reorg-take6-part5.12 branch from 6b37561 to a18d3b4 Compare August 26, 2018 22:39
@rye

rye commented Aug 26, 2018

Copy link
Copy Markdown
Member

This could use a sanity check post-rebase. I had a merge conflict.

@hawkrives hawkrives left a comment

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.

LGTM

@hawkrives

Copy link
Copy Markdown
Member Author

Except for the fact that it needs more commits

@rye

rye commented Aug 26, 2018

Copy link
Copy Markdown
Member

Okay, I'll just merge the appropriate branch, then.

@rye rye force-pushed the reorg-take6-part5.12 branch from 1eccc57 to 296962d Compare August 27, 2018 00:38

@rye rye left a comment

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.

Flow passes locally, and this PR is now atomic.

@rye rye merged commit 00490d8 into master Aug 27, 2018
@rye rye deleted the reorg-take6-part5.12 branch August 27, 2018 01:31
@rye rye added this to the v2.6 milestone Sep 11, 2018
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