Skip to content
This repository has been archived by the owner on Jun 9, 2022. It is now read-only.

proposal for reactjs #394

Closed
iamstarkov opened this issue May 4, 2015 · 10 comments
Closed

proposal for reactjs #394

iamstarkov opened this issue May 4, 2015 · 10 comments

Comments

@iamstarkov
Copy link

Can you write instruction how to integrate fastclick with reactjs?

@cmwelsh
Copy link

cmwelsh commented May 5, 2015

Here's how I integrate them:

var FastClick = require('fastclick');
FastClick.attach(document.body);

@iamstarkov
Copy link
Author

and does it work well with synthetic events in reactjs?

@cmwelsh
Copy link

cmwelsh commented May 5, 2015

It seems to be fully functional with typical React.js code:

<div onClick={ this.handleClick } />

@iamstarkov
Copy link
Author

I’m 80% sure this will not work. Have you seen first SO answer on this problem?

@iamstarkov
Copy link
Author

Have you tried?

@cmwelsh
Copy link

cmwelsh commented May 11, 2015

I use the above code on my application. I'm considering switching to selective-fastclick due to various issues on iOS.

@iamstarkov
Copy link
Author

Okay, maybe I don't understand something. Why you can use it simple onClick and this guy from SO have to write 80 line component to force fastclick work properly with reactjs?

@cmwelsh
Copy link

cmwelsh commented May 11, 2015

I don't know. facebook/react#2753 is a corresponding issue on the React project.

@iamstarkov
Copy link
Author

In Issue you linked people are talking about the same as I: fastclick is not working with react from a box

@iamstarkov
Copy link
Author

For anybody lurked for reactjs implementation, take a look at https://www.npmjs.com/package/react-tap-event-plugin

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

2 participants