RedReader icon indicating copy to clipboard operation
RedReader copied to clipboard

Handle mailto scheme in links

Open B0pol opened this issue 5 years ago • 3 comments

https://www.reddit.com/r/spacex/comments/gxb7j1/wearethespacexsoftwareteamaskusanything/

At the bottom of the self-Text: […] send your resume to [[email protected]](mailto:[email protected])

It should be displayed like that: [email protected] but it detects twice the emaid: once between [ ] (and it adds mailto), and another time between ( )

screenshot of the problem

B0pol avatar Jul 31 '20 17:07 B0pol

Thanks for the report. It looks like that on the website too, though -- we don't actually parse markdown any more, we parse the already-formatted HTML :)

We could handle mailto links better in general though (and actually open the email app).

QuantumBadger avatar Aug 27 '20 20:08 QuantumBadger

It looks like that on the website too [...] we parse the already-formatted HTML

The bad parsing only happens on Old Reddit:

send your resume to [<a href="mailto:[email protected]">[email protected]</a>](mailto:<a href="mailto:[email protected]">[email protected]</a>)

Old Reddit renders with incorrectly parsed mailto link

The HTML output in New Reddit is fine:

send your resume to <a href="mailto:[email protected]" class="_3t5uN8xUmg0TOwRCOGQEcU" rel="noopener nofollow ugc" target="_blank">[email protected]</a>

New Reddit renders mailto link just fine

Could this be fixed by parsing New Reddit's HTML rather than Old Reddit's?

yawnoc avatar Jan 19 '21 09:01 yawnoc

Currently working on this.

Cguy7777 avatar Jul 08 '23 11:07 Cguy7777