Как можно улучшить этот фрагмент JavaScript? - PullRequest
Новая
       12

Как можно улучшить этот фрагмент JavaScript?

2 голосов
/ 23 сентября 2010

Этот код (ниже) взят из проекта с открытым исходным кодом (SemanticScuttle) Я немного изменил исходный код, пытаясь преобразовать букмарклет, в «пригодный для использования в Интернете» Javascript.

Текущий статус: Google Chrome = отлично работает!

Fiefox = Функционально, но открывается в полноразмерной вкладке вместо предопределенного размера.

IE = DOA = Не работает, пожалуйста, помогите ... 

    <script type="text/javascript">
var selection = '';
if (window.getSelection) {
    selection = 'window.getSelection()';
} else if (document.getSelection) {
     selection = 'document.getSelection()';
 } else if (document.selection) {
     selection = 'document.selection.createRange().text';
 }

 document.write('<a href="javascript:x=document;a=encodeURIComponent(x.location.href);t=encodeURIComponent(x.title);d=encodeURIComponent('+selection+');open(\'http://example.com/share/bookmarks.php/Energy?action=add&amp;popup=1&amp;address=\'+a+\'&amp;title=\'+t+\'&amp;description=\'+d,\'Site Name\',\'modal=1,status=0,scrollbars=1,toolbar=0,resizable=1,width=885,height=765,left=\'+(screen.width-885)/2+\',top=\'+(screen.height-725)/2);void 0;"><img src="images/bar/book22_5.jpg" alt="Example Code" title="Example Code" /></a>');

</script>

Так как он работает в Chrome и работает в Firefox, мне бы очень хотелось иметь возможность использовать его и с iE. Большое спасибо.

Ответы [ 3 ]

2 голосов
/ 23 сентября 2010

Соответствующая часть вашего кода (переформатирована и сокращена) выглядит следующим образом:

open( urlToOpen, 'Site Name', options );

Вместо нее следует читать:

open( urlToOpen, '_blank', options );

Проблема в том, как вы используете функцию window.open ().IE думает, что вы пытаетесь открыть URL-адрес в цели под названием «Имя сайта».Если вы измените его на «_blank», оно откроется в новом окне.

Проверьте раздел о параметре sName @ MSDN: http://msdn.microsoft.com/en-us/library/ms536651(VS.85).aspx

2 голосов
/ 23 сентября 2010 
var selection;
if (window.getSelection) {
    selection = window.getSelection();
} else if (document.getSelection) {
     selection = document.getSelection();
 } else if (document.selection) {
     selection = document.selection.createRange().text;
 } else {
     selection = null;
 }

 if (selection !== null) {
    // Do your thing i.e. take out the stuff within the document.write 
    // and have it execute from here.
 }
1 голос
/ 23 сентября 2010

Вы можете написать функцию для получения текущего выбора следующим образом, а затем вызывать его, когда вам это нужно: 

  function getSelectedText() {
     // get the selection function
     var selection = window.getSelection || document.getSelection 
                        || document.selection;

     if(selection)   {
        var range = selection.createRange;
        if(range) {
           selection = range();
        }
        // do something with selection
        return selection.text || selection();
     }
  }

Вышеописанное работает во всех браузерах (IE, FF, Chrome)

Я не тестировал эту часть, но я думаю, что если вы напишите свой код по одной строке за раз, а потом присоединитесь к нему, будет легче отладить, вот пример (Предупреждение! Требуется проверка): 

  var href = [
     'x=document;',
     'a=encodeURIComponent(x.location.href);',
     't=encodeURIComponent(x.title);',
     'd=encodeURIComponent(getSelectedText());', // call doSelection here.
     'url="http://example.com/share/bookmarks.php/Energy";',
     'opts="modal=1,status=0,scrollbars=1,toolbar=0,resizable=1,width=885,height=765,left=" + ((screen.width-885)/2) + ",top=" + ((screen.height-725)/2);',
     'open(url + "?action=add&amp;popup=1&amp;address=" +a + "&amp;title="+ t + "&amp;description=" +d ,',
        '"Site Name", opts);',
     'void 0;'
  ].join("");

document.write('<a href="' + href + '"><img src="images/bar/book22_5.jpg" alt="Example Code" title="Example Code" /></a>');
...