Make links open in new window?

Celeste

Is there a way to modify the plugin so links inserted with CLEditor open in a new window?

Tama

I added some javascript to the bottom of my default.master.tpl, to ensure that all links even if they are added in source mode open in a new window.

Maybe a bit of a ghetto fix but it's easy to install

http://pastebin.com/GKPPQ3HP

Celeste

@Tama Looks great, thank you!

vrijvlinder

@Celeste If you don't want all the links to open in a new page but just the attachments you can also use this small code also in between {literal } tags for tpl . If you copy this remove the spaces I added between the < > so they could show here.

< script type="text/javascript" > $(document).ready(function() {
   $(".Attachment a").attr("target", '_blank');
   });
< /script >

Celeste

@vrijvlinder Is an inserted link considered an attachment?

hgtonight

We can use the target="_blank" attribute to force opening in a new window/tab.

$(document).ready(function() {
  $('.MessageList a').attr('target', '_blank');
});

This is javascript (using jQuery) that finds all a children of the .MessageList element and adds the attribute target set to _blank. Any links posted in the discussion will open in a blank tab.

Celeste

@hgtonight Thank you!

Celeste

Actually, @hgtonight — that has a flaw, which is that it seems to also affect links like usernames and date permalinks. I just want it to work if someone inserts a link to another website in their discussion post/comment. How does CLEditor form the code that makes the links? It seems like there should be some way to edit that as opposed to the overall theme.

hgtonight

If you are familiar with CSS selectors, you can choose which ever you would like to put in the $('Selectors here') part of the previous code snippet.

For example:

$(document).ready(function() {
  $('.MessageList .Message a').attr('target', '_blank');
});

Celeste

@hgtonight Perfect! Thanks again

NabUser

Hello guys,
I'm trying this solution without any luck.

Where is this code supposed to be placed on default.master.tpl?
It's making my page crash

NabUser

NVM found the solution. we need to add it as follows:

{literal}  
<script type="text/javascript">
      $(document).ready(function() {
  $('.MessageList .Message a').attr('target', '_blank');
});
</script>
{/literal}

Mind the extra space after < and before > . These should be removed

peregrine

@NabUser said:
Hello guys,
I'm trying this solution without any luck.

Where is this code supposed to be placed on default.master.tpl?
It's making my page crash

post what you did!

NabUser

It's up there !