Problem

Currently there is no way to add images to a comment on a node. In some cases this is necessary to have meaningful discussions on certain topics/events.

Solution

It should be possible for all LU to add images to a comment on a node. We already have the comment file uploads module which we should leverage. If we allow image file extensions and improve the display of uploaded images we can already make this happen.

Acceptance criteria:

  • User should be able to insert image in comments on content (only node)
  • Should work next to the comment file upload module
  • Behaviour should be similar to the image on a post
  • Image should be presented in Lightbox (same as what is done on post currently)
  • Image can be added to the comment (same interaction as post)
  • Images can also be added to replies on a comment (same as comment)
  • (Discuss: performance loading impact as well)
  • Determine how the image on comments will look like on streams as well

PR Here:
https://github.com/goalgorilla/open_social/pull/1231

Image file uploads on comments

CommentFileSizeAuthor
#5 comment_detail page.png345.29 KBjaapjan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jaapjan created an issue. See original summary.

jaapjan credited Kingdutch.

jaapjan credited xinyuma.

jaapjan’s picture

Issue summary: View changes
FileSize
345.29 KB
jaapjan’s picture

Status: Needs review » Fixed

Will be released in 5.0.

  • ronaldtebrake authored 654c6a7 on 8.x-5.x
    Issue #3044340 by ronaldtebrake, Kingdutch, xinyuma, jaapjan: insert an...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

tepelena’s picture

Just updated to the latest release of OS and the option to add image when commenting is not there.

aangel’s picture

@tepelena there is a module to turn on ("social_comment_upload"). Then you should see "Add attachment" below the textfield and a "Choose file" below that.

It's not working for me quite yet but it worked on a stock install of OS 7.1; I think I have an Ajax problem on my install.