436105 - Linkify URLs

Status: VERIFIED FIXED

(support.mozilla.org :: Forum, task)

Description

[Jason Barnabe (np)]

URLs posted with no formatting in forum topics should become links.

Comment 1

[Chris Ilias [:cilias]]

I remember we turned this off, because it caused problems with the screenshot in <http://support.mozilla.com/en-US/kb/Firefox+cannot+connect+securely+because+the+SSL+protocol+is+disabled>, which uses a URL in the alt text. I assume the switch is universal.

Comment 2

[Chris Ilias [:cilias]]

The feature is called "AutoLinks".
<http://support.mozilla.com/tiki-admin.php?page=textarea>

Comment 3

[David Tenser [:djst]]

Can we use different settings for forums? They would definitely be useful there.

Comment 4

[David Tenser [:djst]]

Nelson, any advice on my question above?

Comment 5

[Nelson Ko [:nkoth]]

I think it is better to try and fix autolinks so that it does not autolink patterns that appear in symtax for screenshots or other things...

Comment 6

[David Tenser [:djst]]

Is there a bug filed for that, either on SUMO or upstream?

Comment 7

[Nelson Ko [:nkoth]]

amette, can you check on whether this is something that is open/fixed upstream?

Comment 8

[johnson]

This script is not browser compatible.. doesnt work in IE which is used massive by many people...

Comment 9

[Chris Ilias [:cilias]]

Looking at the markup that caused us to turn AutoLinks off:

{SHOWFOR(browser=firefox2)}
If Firefox can't access a secure site, it will show the error, ''Firefox can't connect securely to www.example.com because the SSL protocol has been disabled.''
%%%
{img src="img/wiki_up/Secure_site_error.png" alt="A dialog with the text 'Firefox can't connect securely to www.securesite.com because the SSL protocol has been disabled'"}
{SHOWFOR}

The very text that is used for the alt attribute is right before the screenshot. This is also the only instance I know that the alt attribute is used for an image in the KB. I think we should just remove the alt attribute from that image, and re-enable AutoLinks.

Comment 10

[David Tenser [:djst]]

We want this functionality for the forum, but I'm not sure if it's a good idea to do it in the KB, since we might be using URL examples in articles that we don't want linkified.

I'm guessing that Nelson's comment 5 means that this is not possible though. Nelson?

Comment 11

[Chris Ilias [:cilias]]

Using the no-parse tag works for disabling specific instances.
E.g.
~np~ http://www.mozilla.com/ ~/np~

https://support-stage.mozilla.org/en-US/kb/Article+Sandbox

Comment 12

[David Tenser [:djst]]

In that case, can we test this on stage and see if it works well? I'm in favor of enabling this since it makes it a lot easier to quickly support users in the forum without having to fiddle with the wiki syntax for making links clickable.

Comment 13

[Chris Ilias [:cilias]]

As I understand it, 1.1 is frozen. I don't want to risk making things more difficult for QA, by making unnecessary admin changes. I'll set this for 1.2, and enable it on stage as soon as 1.1 is pushed.

Comment 14

[Chris Ilias [:cilias]]

AutoLinks is now enabled on stage.

Comment 15

[Stephen Donner [:stephend] Not actively reading bugmail]

(In reply to comment #14)
> AutoLinks is now enabled on stage.

Test post here: https://support-stage.mozilla.org/tiki-view_forum_thread.php?forumId=1&comments_parentId=349770; I'll keep looking around for potential regressions (before I verify), but it's looking good.

Comment 16

[Stephen Donner [:stephend] Not actively reading bugmail]

As you can see, I included:

<div class="posted-text">
<p><span class="img"><img alt="www.sense.com" src="img/wiki_up/Ssl2_disabled.png" border="0"  />
</span>
</p>
</div>

And didn't have a problem with the forum post; I also did the img src/alt markup Chris references in comment 9, and threw it up on:

https://support-stage.mozilla.org/en-US/kb/autolink?bl=n, without any problems.

I'm calling this verified FIXED, especially given the workaround/solution in comment 11.

Comment 17

[Chris Ilias [:cilias]]

AutoLinks is now enabled on prod.