Skip to content
This repository has been archived by the owner on Mar 12, 2020. It is now read-only.

Add tap and route to other entity handling. #23

Closed
5 tasks done
alloy opened this issue May 3, 2016 · 4 comments
Closed
5 tasks done

Add tap and route to other entity handling. #23

alloy opened this issue May 3, 2016 · 4 comments
Assignees
Milestone

Comments

@alloy
Copy link
Contributor

alloy commented May 3, 2016

  • Add a native module that can use Eigen’s ARSwitchBoard to open a new VC and add it to the stack. (I have a rough version here which is used here.)
  • Add tap handling to articles.
  • Add tap handling to artworks.
  • Add tap handling to shows.
  • Add tap handling to artists.
@alloy alloy added this to the v1 milestone May 3, 2016
@alloy alloy self-assigned this May 3, 2016
@orta
Copy link
Contributor

orta commented May 4, 2016

This line - should use the ARTopMenuVC sharedVC - pushViewController to ensure UINavigation/UISplitVCs get shown as modals instead of pushes ( which will assert )

@alloy alloy assigned sarahscott and unassigned alloy May 13, 2016
@alloy
Copy link
Contributor Author

alloy commented May 13, 2016

@sarahscott If you get to this, maybe make it so that you do the tap handling and then console.log() the Artsy URL that represents the tapped entity.

@alloy
Copy link
Contributor Author

alloy commented May 18, 2016

@sarahscott It seems like the TouchableWithoutFeedback component I used isn’t working as expected, I need to tap an area twice before it’s triggered.

@alloy
Copy link
Contributor Author

alloy commented May 18, 2016

I wonder if that issue is related to facebook/react-native#1352

@alloy alloy closed this as completed May 23, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants