-
Notifications
You must be signed in to change notification settings - Fork 160
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
Get rid of frontend dependencies #161
Comments
I do fully agree. It is a lot of work, but it's worth the effort. About anonymous users, next release 2.6.2 will explain how to customize the app so that it doesn't allow anonymous users to post comments. It's in branch issue-140. |
That's great you've been thinking about it and are planning on it! The new "only users can post" enhancement is great too. I might play around with a VanillaJS implementation so will let you know if I get anywhere. |
@danirus @getup8 To add to this discussion, I think at this moment, the JS plugin is too much tied up with the |
Hello, thanks for this excellent package. Wondering if there's an ETA for this vanilla javascript release. Have a deadline coming up and we'd love to use django-comments-xtd with the ajax javascript plugin. Despite experience with jQuery and bootstrap, our team isn't familiar with react and we weren't able to get it working. We are also using bootstrap 3 and while we were able to modify a few of the templates to use the old bootstrap 3 classes, it looks like we'll need to modify the js plugin as well to continue using bootstrap 3. Some packages like crispy-forms have a 'bootstrap_version' setting where users can set the value to 'bootstrap 3', 4 or 5, and the package includes those different versions of the templates. That would be another excellent feature (not sure how much work it is!) It might be possible to include the previous versions of the templates and choose which ones to display based on the setting. Thanks so much. |
Hi @jhancia, there is no estimation. I started working on it as part of other major changes in the app (branch issue-210, which uses a new model |
Same here, I want to express my gratitude to the team for this awesome package. |
So I was looking on how to use this project with Tailwind (it is just a template file so it shouldn't be difficult) but as others wrote the issue is the JS framework. I see various ticket for version3 that should remove the React plugin but they are very old information. How we can help to move on the version 3.0? |
The work to have a ReactJS-free and Bootstrap-free django-comments-xtd was moved to the comment's ink organization:
I didn't work on the project since almost two years. I do not consider it ready to use because the templating is over-complicated and caching comments is too messy. |
Yeah I saw that the project doesn't get updates since 2 years, so we can consider the 3.0 milestone abandoned and that this project is like in maintenance mode on 2.9.x branch with react. |
About django-comments-ink, postponed rather than abandoned describes better the current situation. About django-comments-xtd, it will stay with the ReactJS plugin and Bootstrap. |
So I did: With those changes Remarkable and Django libraries are used only if available. |
Reading the JS code the bootstrap templating is defined in the jsx file so to switch to Tailwind is required to change the code and compile it: django-comments-xtd/django_comments_xtd/static/django_comments_xtd/js/src/comment.jsx Line 441 in 8c56a42
|
As I commented in PR #420, none of the two mentioned PRs (#417 and #420) remove the dependencies referred to in this issue, which are the Bootstrap UI library and ReactJS. |
First off, thanks for creating such a great app, it has a ton of functionality and seems well built.
Just wanted to give some feedback after going through the tutorial, feel free to do with it what you will :)
It would be amazing if this could have zero dependencies (besides django-contrib-comments of course). But needing to use React, jQuery and Bootstrap from the get go is a lot to ask if you're using another CSS library (or none at all) and VanillaJS or another JS library. Maybe consider a VanillaJS implementation with some basic styling as the default version instead? And then have plugins / extensions? I know that'd be a lot of work..
Also, a setting that just didn't allow commenting / interacting with comments for anon users would be nice. Doesn't seem to exist unless I misread.
The text was updated successfully, but these errors were encountered: