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

Remove unnecessary parens - () #172

Merged
merged 2 commits into from
Dec 1, 2017
Merged

Remove unnecessary parens - () #172

merged 2 commits into from
Dec 1, 2017

Conversation

jbampton
Copy link
Contributor

No description provided.

@chriddyp
Copy link
Member

chriddyp commented Dec 1, 2017

🎉 thanks!

@chriddyp chriddyp merged commit 12f9b50 into plotly:master Dec 1, 2017
@jbampton jbampton deleted the patch-5 branch December 1, 2017 00:29
HammadTheOne pushed a commit to HammadTheOne/dash that referenced this pull request May 28, 2021
HammadTheOne pushed a commit to HammadTheOne/dash that referenced this pull request May 28, 2021
* Propagate to ControlledTable on navKeys

* Test if arrow keys can navigate and save cell value

* Add keyboard navigation tests for focused inputs

* Update changelog

* Add onMouseUp functionality for arrow key navigation

* Calculate derivedHandlers for onMouseUp

* Add test for caret selection on focused input

* Rebuild bundle

* Jump to next cell on enter key

* Fix focused input test by doubleclicking instead of enter

* Remove .only from copy paste test

* Remove artifact from it's tomb of copy_paste_test
HammadTheOne pushed a commit that referenced this pull request Jul 23, 2021
* Propagate to ControlledTable on navKeys

* Test if arrow keys can navigate and save cell value

* Add keyboard navigation tests for focused inputs

* Update changelog

* Add onMouseUp functionality for arrow key navigation

* Calculate derivedHandlers for onMouseUp

* Add test for caret selection on focused input

* Rebuild bundle

* Jump to next cell on enter key

* Fix focused input test by doubleclicking instead of enter

* Remove .only from copy paste test

* Remove artifact from it's tomb of copy_paste_test
HammadTheOne pushed a commit that referenced this pull request Jul 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants