Merge pull request #884 from ArthurHoaro/hotfix/bookmarklet-url-limit

Selection is now limited to 2k characters using bookmarklets

1 files changed, 12 insertions, 2 deletions

diff --git a/tpl/default/tools.html b/tpl/default/tools.html
index 6951ad28..35173d17 100644
--- a/tpl/default/tools.html
+++ b/tpl/default/tools.html
@@ -72,10 +72,15 @@
           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 @@
          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'
             );
           }