Posting this on d.o:

<ul>
<li>https://medium.com/@dan_abramov/smart-and-dumb-components-7ca2f9a7c7d0#.q8xmja91n
<li>https://medium.com/@learnreact/container-components-c0e67432e005#.cb6jw5nxd
<li>http://jaketrent.com/post/smart-dumb-components-react/
</ul>

should linkify all three URLs. But the first is not. You'd think it's because of the @, but you'd be wrong, because the second URL is linkified just fine.

Discovered at #2702061-67: Unify & simplify render & theme system: component-based rendering (enables pattern library, style guides, interface previews, client-side re-rendering). Attached screenshot of previewing a new node with just the above sample content proves that it's easily reproducible, and not a special edge case in that particular (complex) comment.

CommentFileSizeAuthor
Screen Shot 2016-07-13 at 19.33.50.png29.71 KBWim Leers
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Wim Leers created an issue. See original summary.

Wim Leers’s picture

Wim Leers
Location Ghent 🇧🇪🇪🇺
CreditAttribution: Wim Leers at Acquia commented
Issue summary: View changes
Wim Leers’s picture

Wim Leers
Location Ghent 🇧🇪🇪🇺
CreditAttribution: Wim Leers at Acquia commented

I tried reproducing this in D8 core, but could not.

apaderno’s picture

apaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
CreditAttribution: apaderno as a volunteer commented
Component: Project/Git problem » Other
TR’s picture

TR CreditAttribution: TR commented
Project: Drupal.org site moderators » Drupal core
Version: » 7.x-dev
Component: Other » filter.module

Yes, this fails on any Drupal site running Drupal 7 using "Filtered HTML" or "Full HTML". Works on any Drupal site running Drupal 8. (But if you remove the underscore in @dan_abramov it works on a D7 site).

So it's not a webmaster issue for drupal.org, this is a problem with how the core filter module regexp parses links.

Moving to the core queue under the filter.module component.