Merge pull request #884 from ArthurHoaro/hotfix/bookmarklet-url-limit
Selection is now limited to 2k characters using bookmarklets
This commit is contained in:
commit
268309df5d
1 changed files with 12 additions and 2 deletions
|
@ -72,10 +72,15 @@ <h2 class="window-title">Bookmarklets</h2>
|
|||
function(){
|
||||
var%20url%20=%20location.href;
|
||||
var%20title%20=%20document.title%20||%20url;
|
||||
var%20desc=document.getSelection().toString();
|
||||
if(desc.length>4000){
|
||||
desc=desc.substr(0,4000)+'...';
|
||||
alert('{function="str_replace(' ', '%20', t('The selected text is too long, it will be truncated.'))"}');
|
||||
}
|
||||
window.open(
|
||||
'{$pageabsaddr}?post='%20+%20encodeURIComponent(url)+
|
||||
'&title='%20+%20encodeURIComponent(title)+
|
||||
'&description='%20+%20encodeURIComponent(document.getSelection())+
|
||||
'&description='%20+%20encodeURIComponent(desc)+
|
||||
'&source=bookmarklet','_blank','menubar=no,height=800,width=600,toolbar=no,scrollbars=yes,status=no,dialog=1'
|
||||
);
|
||||
}
|
||||
|
@ -89,9 +94,14 @@ <h2 class="window-title">Bookmarklets</h2>
|
|||
class="bookmarklet-link"
|
||||
href="javascript:(
|
||||
function(){
|
||||
var%20desc=document.getSelection().toString();
|
||||
if(desc.length>4000){
|
||||
desc=desc.substr(0,4000)+'...';
|
||||
alert("{function="str_replace(' ', '%20', t('The selected text is too long, it will be truncated.'))"}");
|
||||
}
|
||||
window.open(
|
||||
'{$pageabsaddr}?private=1&post='+
|
||||
'&description='%20+%20encodeURIComponent(document.getSelection())+
|
||||
'&description='%20+%20encodeURIComponent(desc)+
|
||||
'&source=bookmarklet','_blank','menubar=no,height=800,width=600,toolbar=no,scrollbars=yes,status=no,dialog=1'
|
||||
);
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue