Code review comment for lp://staging/~diogenesethecynical/lernid/Bug793793
Revision history for this message
| John S. Gruber (jsjgruber) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
513534c
demo site
(Get the code!)
This isn't for sharing a url, however, it is a shortcut for someone using the Open Url... I don't think the browser will work with ftp:// ,etc., at this point anyway, but I guess it could in the future.
Diogenese, you could check have urlparse do some work here and have it find the scheme. If it finds no scheme you could do the prepend of http:// . That might make the code more robust. I'm not sure this is terribly important.