Opened 4 months ago

Closed 4 months ago

Last modified 4 months ago

#1546 closed enhancement (fixed)

Pasting links with "http://" results in broken links

Reported by: isaackeyet Owned by:
Priority: trivial Milestone: 3.4
Component: Compose Version:
Keywords: Cc:

Description

Classifying as an Enhancement.

If you copy a link that includes the http protocol into the Link Helper in the Post Editor, the result will be a broken link (e.g. "​http://http://example.com". Solution would be to check for what's been pasted and remove the first ​http:// if there's two. Note that it also has to work with other types of links (e.g. "​https://").

Change History (3)

comment:1 Changed 4 months ago by jkv1

Sent a github pull request with the fix - ​https://github.com/wordpress-mobile/WordPress-iOS/pull/13

The code is already dealing with a URL with the ​http:// or other protocols already added but it does not deal with spaces before the protocol. I just added a trim to the text field.

comment:2 Changed 4 months ago by koke

  • Resolution set to fixed
  • Status changed from new to closed

(In [4009]) Fixes #1546 - link helper adding duplicate protocol. props jkv1

The URL text field needs to be trimmed of whitespace since the user
could paste text that has spaces in it.

http://ios.trac.wordpress.org/ticket/1546

comment:3 Changed 4 months ago by beaucollins

  • Milestone changed from Future Release to 3.4
Note: See TracTickets for help on using tickets.