-
Notifications
You must be signed in to change notification settings - Fork 36
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
Comments
Thanks for raising the issue. I will look into it asap. |
Hi, what is the status of this issue? :) |
added date control and radioButtons WIP
React v16 versioning issue is fixed : d934006 |
Please let me know if you face issue. |
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. |
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 |
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 |
Invisible popover issue fixed on my fork by using reactstrap (not react-bootstrap) for Bootstrap 4 popovers. You will need to specify a |
@YassienW would you do a PR of your fork to merge support for bs4? |
@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 Hi How did you builded/compiled this project ? I have this error when I try
|
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. |
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 Yes, i should probably specify that. I think we should move any discussion related to my project to its repository to avoid spamming here. |
Hi i would like to inform you that popup mode is not working with react 16.
The text was updated successfully, but these errors were encountered: