Comments (2)
LouisBarranqueiro
commented on June 16, 2024
2
Hey @tphan18 @human-a , I released a fix in v1.1.4. I started working to officially support React 16. For that, I have to upgrade all the test setup to run all the tests suite with react 16, 15, 14.
from reapop.
commented on June 16, 2024
I am also getting the following errors on react 16
Warning: Expected `onClick` listener to be a function, instead got a value of `string` type.
in div (created by Notification)
in Notification (created by Connect(Notification))
in Connect(Notification) (created by NotificationsContainer)
in CSSTransitionGroupChild (created by TransitionGroup)
in span (created by TransitionGroup)
in TransitionGroup (created by CSSTransitionGroup)
in CSSTransitionGroup (created by NotificationsContainer)
in div (created by NotificationsContainer)
in NotificationsContainer (created by NotificationsSystem)
in div (created by NotificationsSystem)
in NotificationsSystem (created by Connect(NotificationsSystem))
in Connect(NotificationsSystem) (at index.js:100)
in div (created by FocusTrap)
in FocusTrap (created by HotKeys)
in HotKeys (at index.js:99)
in Studio (created by Connect(Studio))
in Connect(Studio) (created by RouterContext)
in RouterContext (created by Router)
in Router (at index.js:52)
in Provider (at index.js:51)
Warning: Expected `onMouseEnter` listener to be a function, instead got a value of `string` type.
in div (created by Notification)
in Notification (created by Connect(Notification))
in Connect(Notification) (created by NotificationsContainer)
in CSSTransitionGroupChild (created by TransitionGroup)
in span (created by TransitionGroup)
in TransitionGroup (created by CSSTransitionGroup)
in CSSTransitionGroup (created by NotificationsContainer)
in div (created by NotificationsContainer)
in NotificationsContainer (created by NotificationsSystem)
in div (created by NotificationsSystem)
in NotificationsSystem (created by Connect(NotificationsSystem))
in Connect(NotificationsSystem) (at index.js:100)
in div (created by FocusTrap)
in FocusTrap (created by HotKeys)
in HotKeys (at index.js:99)
in Studio (created by Connect(Studio))
in Connect(Studio) (created by RouterContext)
in RouterContext (created by Router)
in Router (at index.js:52)
in Provider (at index.js:51)
Warning: Expected `onMouseLeave` listener to be a function, instead got a value of `string` type.
in div (created by Notification)
in Notification (created by Connect(Notification))
in Connect(Notification) (created by NotificationsContainer)
in CSSTransitionGroupChild (created by TransitionGroup)
in span (created by TransitionGroup)
in TransitionGroup (created by CSSTransitionGroup)
in CSSTransitionGroup (created by NotificationsContainer)
in div (created by NotificationsContainer)
in NotificationsContainer (created by NotificationsSystem)
in div (created by NotificationsSystem)
in NotificationsSystem (created by Connect(NotificationsSystem))
in Connect(NotificationsSystem) (at index.js:100)
in div (created by FocusTrap)
in FocusTrap (created by HotKeys)
in HotKeys (at index.js:99)
in Studio (created by Connect(Studio))
in Connect(Studio) (created by RouterContext)
in RouterContext (created by Router)
in Router (at index.js:52)
in Provider (at index.js:51)
PS.
Managed to fix it by changing line 233 to 236 inside Notification.js file to the following:
onClick: isDismissible && !closeButton ? this._remove : function(){},
onMouseEnter: timer ? this._pauseTimer : function(){},
onMouseLeave: timer ? this._resumeTimer : function(){}from reapop.
Related Issues (20)
- Updating notification throws state mutation error with redux-toolkit HOT 8
- notify.error("Message", STATUSES.error) not showing as error HOT 5
- Need instructions to use with Next Js HOT 4
- TypeError: Cannot read property 'filter' of undefined HOT 1
- How do I properly implement onClick for buttons? Noticed button onClick fires when notification is triggered. HOT 2
- Theme - notificationButtons style is not passed to the buttons container
- Undefined payload HOT 7
- Attaching Notification OnClick events to the specific notification HOT 3
- Wrong default class for NotificationIcon HOT 1
- [email protected] doesn't have "dist" folder in it HOT 3
- Mutation detected HOT 4
- Specify Message Tag HOT 2
- Cut a new release on npm HOT 7
- Failed to execute 'animate' on 'Element': Partial keyframes are not supported HOT 4
- IE11 not support HOT 3
- Hydration Mismatch in Next.js v13 HOT 3
- Is sending a notification in one component supposed to rerender parallel components? HOT 3
- `ReactNode` vs `string` for message HOT 4
- Close notification programatically HOT 7
- Promise support for notifications HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from reapop.