Image embeds have the wrong hit target

[vjeux]

You can click on the blank area at the right of the image and it still opens the image :(

[gmsecrieru]

I think that's mostly because we display the image as the background-image of an empty div within an a tag, which takes up all the available width. I think we would need to define the maximum width of the image container somehow (related to original image) or take another approach.

[liamdawson]

@gmsecrieru — The approach you've mentioned (background image on a full-width div) also prevents #1139 from happening, at least from my brief testing. Is there a reason the img tag isn't used as-is?

[gmsecrieru]

Not sure. @RocketChat/owners ?

[geekgonecrazy]

@gmsecrieru @liamdawson check out: #544

I think it might be a good chance to start a discussion about this.

[rodrigok]

I think I can improve this using the same way that Slack does.

[geekgonecrazy]

@rodrigok i'm not familiar with how slack handles this. How do they do it?

[stefanlegg]

Created a pr that solves this: #1167

[rodrigok]

@geekgonecrazy @stefanlegg please see here #1170