Rather than hard-coding the images to use HTTP, or HTTPS (which requires #1642522: Using https version of profile image), I'd prefer a protocal-relative URL by default.

CommentFileSizeAuthor
#1 2298913-protocol-relative-image-url.patch606 bytesdave reid

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs review
StatusFileSize
new 2298913-protocol-relative-image-url.patch606 bytes
dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Looks like this depends on #2006142: Add support for image variants in profile images. getting committed first. :/

  • juampy committed 4791943 on 7.x-1.x authored by Dave Reid 
    Issue #2298913 by Dave Reid: Added Use protocol-relative image URLs.
juampynr’s picture

juampynr commented
Status: Needs review » Fixed

Committed. Thanks!

Status: Fixed » Closed (fixed)

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