react-x-editable icon indicating copy to clipboard operation
react-x-editable copied to clipboard

React 16 - Popup mode is not working

Open uxieme opened this issue 7 years ago • 14 comments

Hi i would like to inform you that popup mode is not working with react 16.

bundle.js:8275 Uncaught Error: _processChildContext is not available in React 16+. This likely means you have multiple copies of React and are attempting to nest a React 15 tree inside a React 16 tree using unstable_renderSubtreeIntoContainer, which isn't supported. Try to make sure you have only one copy of React (and ideally, switch to ReactDOM.createPortal).

uxieme avatar Dec 07 '17 10:12 uxieme

Thanks for raising the issue. I will look into it asap.

ni3galave avatar Dec 08 '17 04:12 ni3galave

Hi, what is the status of this issue? :)

uxieme avatar Dec 12 '17 17:12 uxieme

React v16 versioning issue is fixed : https://github.com/ni3galave/react-x-editable/commit/d934006b0109ad9262a3d4f7b94103c6e699625e

ni3galave avatar Mar 13 '18 09:03 ni3galave

Added support for React 16+.

Please let me know if you face issue.

ni3galave avatar Jun 10 '18 10:06 ni3galave

Hi, POPUP example doesn't work on the Demo page (also doesn't work locally on my project). It seems that this issue is still active. Is there any chance for you to fix it?

PS: In my project, I don't see any error or warnings in the console...

Thanks.

elvir92 avatar Jul 15 '18 03:07 elvir92

Popup issue is working on my local, how to call this function inside another tag ?

...working this editable tag but ,i want to call this function on inputStyle tag

cpratiksha avatar Jul 26 '18 07:07 cpratiksha

After doing a little digging, i discovered that the popups on the example website (and in my project) aren't working properly because Bootstrap 4 is being used instead of Bootstrap 3. If you inspect the html the popups are added to the DOM but they simply don't show. This library depends on react-bootstrap, which sadly doesn't have Bootstrap 4 support yet. I'll be trying to get it working with Bootstrap 4 since i already have a project using react-bootstrap & Bootstrap 4 without any issues

YassienW avatar Jan 28 '19 15:01 YassienW

Invisible popover issue fixed on my fork by using reactstrap (not react-bootstrap) for Bootstrap 4 popovers. You will need to specify a bootstrap4={true} prop on the editable

YassienW avatar Jan 29 '19 15:01 YassienW

@YassienW would you do a PR of your fork to merge support for bs4?

fcpauldiaz avatar Apr 13 '19 17:04 fcpauldiaz

@fcpauldiaz I don't think the PR will get merged, the project looks abandoned to me. I did start my own similar project though but it doesn't have popover mode yet

YassienW avatar Apr 13 '19 18:04 YassienW

@YassienW Hi

How did you builded/compiled this project ?

I have this error when I try

ERROR in ./libs/js/Editable.jsx
Module parse failed: /react-x-editable/libs/js/Editable.jsx Unexpected token (62:18)
You may need an appropriate loader to handle this file type.
|     }
|   }
|   setInitialValue = (newProps) => {
|     const { dataType, options, value } = newProps;
|     if(dataType == "select" || dataType == "checklist"){

fcpauldiaz avatar Apr 17 '19 19:04 fcpauldiaz

I just ran the scripts in package.json, they work fine for me.

I did implement popover mode in my own project, if you would like to try it out. Feedback would be appreciated.

YassienW avatar Apr 18 '19 20:04 YassienW

Nice work @YassienW. I noticed that it uses font-awesome but I didn't found any dependency to font-awesome, it should be loaded in the parent project using it ? (https://github.com/YassienW/react-bootstrap-editable/blob/master/src/Editable.js#L40)

P.S. I would like to be a contributor

fcpauldiaz avatar Apr 19 '19 19:04 fcpauldiaz

@fcpauldiaz Yes, i should probably specify that. I think we should move any discussion related to my project to its repository to avoid spamming here.

YassienW avatar Apr 19 '19 19:04 YassienW