Propose new functionalities for Decidim software
#DecidimRoadmap Designing Decidim together
Allow attachements in comments
Is your feature request related to a problem? A clear and concise description of what the problem is.
We've had this request by a couple clients. It's also in the Decidim Awesome roadmap.
Now that comments have be re-written in rails it's possible to enhance the comment function.
Describe the solution you'd like
As an admin I want to be able to enable attachement in comments using a config flag on the components config
As a user adding a comment when attachements are enabled I have an upload button bellow the text field. Once the comment is published the attachement displays under the text as link inside an attachement box. (See attachements)
Describe alternatives you've considered
No
Additional context
Comment were re-written in rails : https://meta.decidim.org/processes/roadmap/f/122/proposals/14369 https://github.com/decidim/decidim/pull/6498
Does this issue could impact on users private data?
No
Funded by
Might be funded by CNRHR
List of Endorsements
Report inappropriate content
Is this content inappropriate?
Comment details
You are seeing a single comment
View all comments
Conversation with Antti Hukkanen
Virgile: Haven't heard this request but I think it would be warmly welcomed.
I'd prefer to have something similar to GitHub issue attachments/images, so you could possibly add more than 1. But that requires having this possibility in the editor as Carol mentioned. After ActiveStorage is introduced to Decidim, it should become easier to add this feature (Trix not necessary but ActiveStorage would make it easier to upload and store files dynamically).
Only thing I'm concerned about is accessibility. Accessibility requirements require every image and attachment having an alternative text/title defined for them. The UI should ask the user to give a title for the attachment after its upload. GitHub issue comments default to the original file name of the uploaded file.
Furilo also raises a very good point about introducing Trix. It should be heavily tested before introduction so we don't find similar pitfalls with it as with quill.js.
By the way the (at) mentions seem to constantly break sending comments. I get HTTP 422.
I tried to send this message with the names (Virgile, Carol and Furtilo) replaced with (at) mentions and it won't accept my comment. Everything is fine after removing the mentions.
Loading comments ...