Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fix #126: urlize: open in new tab (target=_blank) #128

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
3 participants

hoffi commented May 20, 2012

Here is the original issue: mitsuhiko#126

I enhanced the urlize function to give a fourth argument to it to add target="_blank" to the link.

rbu commented Nov 12, 2012

I would prefer the API to be more flexible here, how about a urlize(target='_blank') which would allow for generic user agent contexts and literals (_self, _parent, _top).

Owner

mitsuhiko commented May 19, 2013

Yeah, closing this. Accepting a patch for making the target a keyword argument however.

@mitsuhiko mitsuhiko closed this May 19, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment