Skip to content
This repository has been archived by the owner on Jul 11, 2019. It is now read-only.

More user-friendly means of posting images #105

Open
hackergrrl opened this issue May 1, 2015 · 9 comments
Open

More user-friendly means of posting images #105

hackergrrl opened this issue May 1, 2015 · 9 comments

Comments

@hackergrrl
Copy link
Contributor

Right now users can only post images using the awkward markdown syntax (e.g.):

![Sorry guys](http://my.setale.me/image/052u2w2q3u3w/Schermata 2015-04-30 alle 19.09.21.png)

What's worse is that the above actually doesn't work.

This issue is to brainstorm and ideally decide on an alternative (or, that the current functionality is fine).

@hackergrrl
Copy link
Contributor Author

Idea: how about auto-converting image URLs to inlined images? It means straying from pure markdown, but I'm not sure how sold folks are on keeping that as a strict requirement.

@tunnckoCore
Copy link
Contributor

using the awkward markdown syntax

omg, lol. markdown is standard around the community.

@simi
Copy link

simi commented May 1, 2015

markdown is ok, but it is anoying to type

![Sorry guys](http://my.setale.me/image/052u2w2q3u3w/Schermata 2015-04-30 alle 19.09.21.png)

instead just copy - paste image url

@misspokeycat
Copy link
Contributor

Perhaps we can handle image URLs similar to how Slack manages them, by leaving the image URL in the message but displaying the image below? We can still use markdown for inline messages.

@tunnckoCore
Copy link
Contributor

displaying the image in chat would be wrong way, imho

@ungoldman
Copy link
Member

@tunnckoCore Markdown is well known to developers but I don't think we necessarily want to make something just for developers. Let's try to be friendly about it.

I think @pokemonmegaman has a point -- we can just allow users to toggle whether or not they want images, videos, etc to be embedded. IRC Cloud does this like so:

screen shot 2015-05-01 at 10 41 29 am

I think there's room for allowing both embedded images and processing markdown syntax.

@tunnckoCore
Copy link
Contributor

@ngoldman sgtm, good point. 👍

@SkonTeam
Copy link

So i tried adding image preview from message containing links here's what i got :

image

any feedback is welcome 👍

@ungoldman
Copy link
Member

looks good @SkonTeam! I think it's best to keep the original link visible, and maybe removed the rounded corners too. feel free to open a PR when you think it's ready.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants