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

Fix ResponsiveNavigation resize handler leak #27

Merged
merged 2 commits into from
Apr 18, 2017

Conversation

Coobaha
Copy link

@Coobaha Coobaha commented Apr 15, 2017

No description provided.

@crisu83
Copy link
Contributor

crisu83 commented Apr 18, 2017

Looks good, thanks for this.

@crisu83 crisu83 merged commit e097895 into digiaonline:develop Apr 18, 2017
crisu83 pushed a commit that referenced this pull request Jun 21, 2017
* Fix ResponsiveNavigation resize event leak

* Use bind instead of class properties
@crisu83 crisu83 mentioned this pull request Jun 21, 2017
crisu83 added a commit that referenced this pull request Jun 21, 2017
* Update nodejs versions in Travis configuration (to latest, 6, 5 and 4)

* Fix ResponsiveNavigation resize handler leak (#27)

* Fix ResponsiveNavigation resize event leak

* Use bind instead of class properties

* Add accordion component (#30)

* Add accordion component

* Add missing isActive prop on AccordionContent

* Add tests for isActive prop on AccordionContent

* Add yarn.lock

* Change to use prop-types library

* Fix warning in progress bar test

* Fix linting errors

* Add precommit hook
@crisu83 crisu83 mentioned this pull request Jul 5, 2017
crisu83 added a commit that referenced this pull request Jul 5, 2017
* Update nodejs versions in Travis configuration (to latest, 6, 5 and 4)

* Fix ResponsiveNavigation resize handler leak (#27)

* Fix ResponsiveNavigation resize event leak

* Use bind instead of class properties

* Add accordion component (#30)

* Add accordion component

* Add missing isActive prop on AccordionContent

* Add tests for isActive prop on AccordionContent

* Add yarn.lock

* Change to use prop-types library

* Fix warning in progress bar test

* Fix linting errors

* Add precommit hook

* Bump version

* Update Travis CI badge link

* Convert remaining PropTypes imports to use prop-types library (#34)

* Bump version

* Bump version
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants