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

Warning: Received false for non-boolean attribute cursor. If this is expected, cast the value to a string. #43

Open
Archakov06 opened this issue Oct 19, 2017 · 14 comments

Comments

@Archakov06
Copy link

Warning: Received false for non-boolean attribute cursor. If this is expected, cast the value to a string.
in input (created by EnhancedSwitch)
in div (created by EnhancedSwitch)
in label (created by EnhancedSwitch)
in EnhancedSwitch (created by Checkbox)
in Checkbox (at App.js:132)
in div (at App.js:126)
in div (at App.js:125)
in div (at App.js:87)
in div (at App.js:86)
in div (at App.js:85)
in div (at App.js:84)
in div (at App.js:61)
in div (at App.js:59)
in App (created by Connect(App))
in Connect(App) (created by Route)
in Route (at index.js:17)
in div (at index.js:16)
in Router (created by ConnectedRouter)
in ConnectedRouter (at index.js:15)
in Provider (at index.js:14)

<Checkbox
                        checkboxClass="icheckbox_square-blue"
                        label="С фотографией"
                      />
                      <Checkbox
                        checkboxClass="icheckbox_square-blue"
                        label="Бесплатно"
                      />
@ghost
Copy link

ghost commented Nov 8, 2017

is there any work around for this problem?

@matyashovandrey
Copy link

No one to correct an error?

@matyashovandrey
Copy link

have any ideas how to fix this?

@DiegoNovati
Copy link

DiegoNovati commented Dec 21, 2017

I have temporary fixed it by opening the 'EnhancedSwitch' under node_modules and commenting the lines where the 'cursor' property is used.
This of course is only a temporary solution.

@danbopes
Copy link

danbopes commented Jan 4, 2018

Can we please get this fixed?

@xilanhthanki
Copy link

Same issue here. Someone have any suggestions?

@alfeltrin
Copy link

As suggested by @DiegoNovati open the file "node_modules / react-icheck / lib / EnhancedSwitch.js" and comment the following lines:
126: cursor: _propTypes2['default'].bool,
183: cursor: false,

@vndevil
Copy link

vndevil commented Jan 17, 2018

@alfeltrin you're right, but this is only a temporary solution.

rglodzinski pushed a commit to rglodzinski/react-icheck that referenced this issue Mar 2, 2018
@wickedev
Copy link

This project is not maintain anymore?

@johanleroch
Copy link

Seriously... It seems to be no longer maintained. I fork it and I fix it, don't have the time to wait

@iqqmuT
Copy link

iqqmuT commented Sep 25, 2018

@johanleroch Are you planning to publish your fixed fork with different npm name?

@johanleroch
Copy link

Hi @iqqmuT , sorry, I do not plan to do it, I have to much to do on my other projects, but you can fork my fix and deploy with different name on npm if you want.

@berzniz
Copy link

berzniz commented Oct 4, 2018

A workaround is to do this:

import EnhancedSwitch from 'react-icheck/lib/EnhancedSwitch'
import { Checkbox } from 'react-icheck'

// Workaround for https://github.com/luqin/react-icheck/issues/43
EnhancedSwitch.propTypes = {
  ...EnhancedSwitch.propTypes,
  cursor: PropTypes.string
}

const Component = () => <Checkbox cursor="pointer" ...rest_of_props... />

@victorpopkov
Copy link

If someone is still interested, I have recently made a complete rework of this project to continue the development: https://github.com/victorpopkov/react-ui-icheck

npm install react-ui-icheck --save

I can't guarantee that it has the full backwards compatibility, but in most cases, it should be compatible.

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

No branches or pull requests