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

React 16 - Popup mode is not working #4

Open
uxieme opened this issue Dec 7, 2017 · 14 comments
Open

React 16 - Popup mode is not working #4

uxieme opened this issue Dec 7, 2017 · 14 comments
Assignees

Comments

@uxieme
Copy link

uxieme commented Dec 7, 2017

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).

@ni3galave ni3galave self-assigned this Dec 7, 2017
@ni3galave
Copy link
Owner

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

@uxieme
Copy link
Author

uxieme commented Dec 12, 2017

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

ni3galave pushed a commit that referenced this issue Jan 11, 2018
added date control and radioButtons WIP
@ni3galave
Copy link
Owner

React v16 versioning issue is fixed : d934006

@ni3galave
Copy link
Owner

Added support for React 16+.

Please let me know if you face issue.

@elvir92
Copy link

elvir92 commented Jul 15, 2018

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.

@cpratiksha
Copy link

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

@YassienW
Copy link

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
Copy link

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

@fcpauldiaz
Copy link

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

@YassienW
Copy link

@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

@fcpauldiaz
Copy link

@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"){

@YassienW
Copy link

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.

@fcpauldiaz
Copy link

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

@YassienW
Copy link

@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.

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

6 participants