Auto-fill URL field on Insert/edit link popup

[tillkruess]

I'd like to see the following behavior: When I select a plain URL or email address in the editor and click on "Insert/edit link" the URL field is already populated with either the URL or with a "mailto:<email-address>". Any thoughts? I'd more than happy to write a patch.

[SergeyBiryukov]

Related: #18149

[nacin]

Sounds good to me.

[tillkruess]

The regular expressions used in the attached patch where taken from ​jquery.validate.js. Not sure if there are ​better ones out there or maybe even already in WP?

[DrewAPicture]

The version field is used to track when an issue was first reported.

[ryansatterfield]

Why does the patch include ftp on line 290? I was going to submit a ticket for mailto: support since I just had a client complain about not being able to link to their email address and only to their site. I am glad there is a patch :).

[tillkruess]

The script catches not only email addresses, it also handles HTTP and FTP URLs.

[ryansatterfield]

Cool. I am always for keeping WordPress as simple as possible and not confusing end-users. Do you agree that it would be easier for most users just to have http, https and mailto? How many people need to link to FTP? If you know about FTP, you probably can add it into the HTML.

[wonderboymusic]

19992.diff​ refreshes so it applies against modern-day trunk

[wonderboymusic]

In 28705:

When a plain URL or email address is selected in the editor and "Insert/edit link" is clicked, pre-populate the URL field with the URL or mailto:<email-address>.

Props tillkruess.
Fixes #19992.

[SergeyBiryukov]

These regexes should be put into variables, as ​suggested in IRC.

[azaozz]

Do we want this to work in the Text editor too? Also, why the "super bulky" regexp? We are not validating user input, just suggesting. Maybe can use something much smaller/faster?

[azaozz]

In [28750]:

wpLink: use the proper instance of the editor when looking at the selection. Move the email and URL regexp to variables. See #19992

[SergeyBiryukov]

19992.3.diff​ uses much simpler expressions, based on ​http://www.regular-expressions.info/email.html.

[SergeyBiryukov]

In 28751:

wpLink: use simpler expressions to detect email address or URL. We are not validating user input here, just suggesting.

see #19992.

[azaozz]

The last patch was limiting the URLs only to the domain. 19992.4.diff matches whole URLs including query/fragment and fixes double encoding of &. It still excludes IDNs and the new(er) TLDs that are longer than 4 letters. Should we support these?

[azaozz]

In 28828:

wpLink: support whole URLs including query/fragment when pre-filling the URL field. Fixes #19992

[SergeyBiryukov]

Replying to azaozz:

It still excludes IDNs and the new(er) TLDs that are longer than 4 letters. Should we support these?

I think it's OK to exclude those for now for the sake of simplicity.