Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

change the children's key to be unique to able html mouse events #46

Open
wants to merge 25 commits into
base: coverall
Choose a base branch
from
Open

Conversation

hartuvofir
Copy link

The File i changed is lib/highlighter.js .
The problem was that the item key (count), was not unique between the items (i debug the html) -
and react cant capture click/mouse events in this case.

I changed it to be based by the Address(string), which is unique.

helior and others added 25 commits February 28, 2016 23:26
Add Coveralls.io integration
No longer rely upon React.__spread which is breaking on 15.0. Also use React.createElement directly rather than React.DOM.span.
Bump react peer dependency to allow v14 and v15
Use React.createElement to allow usage with React 15.0.
Fixes deprecation warnings in React 15.5.0
Specifically create-react-class and prop-types"
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling cac0495 on bookmd:master into ** on helior:master**.

@hartuvofir hartuvofir changed the base branch from master to coverall August 20, 2017 08:19
@janmyler
Copy link
Contributor

@hartuvofir I'd suggest you remove all the IntelliJ related files from the PR and keep only the relevant changes.

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.

8 participants