Букмарклет, подгружающий внешний скрипт

[Fanatik]

ну пипец, умно

[Antohins]

стопудово

[amgorb]

молодец!

[M_K]

Спасибо!

[sha1dy]

просто интересно: нахрена if(typeof s != 'object')?

[prolis]

кроссбраузерность наверно

[SamDark]

Кроссбраузерность + это короче, чем:
d.standardCreateElement ? d.standardCreateElement('script') : d.createElement('script');

[SamDark]

Вообще это для некоторых особо умных сайтов.

[SamDark]

В IE6 есть нехорошее ограничение на длину кода букмарклета.

[lahmatiy]

М.б. тогда попробывать:
s = d.createElement('script') || d.standardCreateElement('script');
?

[SamDark]

Дело в том, что createElement перекрывается некоторыми сайтами для того, чтобы туда ничего не подгружали. Функция может чего-нибудь и вернуть…

[lahmatiy]

Логично предположить. Тогда
s = (d.standartCreateElement || d.createElement)('script');
?

[SamDark]

Красиво!

[SamDark]

Однако в FF3 не работает…
Illegal operation on WrappedNative prototype object.

[lahmatiy]

и в FF2 тоже :)
ему нужно

s = (d.standartCreateElement || d.createElement).call(d, 'script');

Function.call() не работает в IE5, но думаю это вас не сильно огорчит ;)

[SamDark]

Ну, лучше конечно, чтобы работало… в код включать не буду, но на заметку обязательно возьму.

[lahmatiy]

и еще один вариант :)

var d = document, s = (d.getElementsByTagName('script')[0] || d.createElement('script')).cloneNode(true);
s.src = 'http://example.com/js/bookmarklet.js';
s.type = 'text/javascript';
d.body.appendChild(s);


Тут d.createElement('script') нужен только для страниц где нет ни одного script. Помоему таких не много :) потому можно обойтись и вариантом только с getElementsByTagName/cloneNode. Ну и еще немного поиздеваться над кодом :)

(s = (d = document).getElementsByTagName('script')[0].cloneNode(true)).src = 'http://example.com/js/bookmarklet.js';
s.type = 'text/javascript';
d.body.appendChild(s);


[SamDark]

Страниц без скрипта приличное количество, так что совсем убирать создание ноды нехорошо :)

Что касается первой части — разница в производительности получится незначительная, а вот длина выросла (standartCreateElement выкидывать не очень хочется).

[SamDark]

Кстати, если кто знает как улучшить или уменьшить код — не стесняйтесь…

[serf]

В общем кто знает что такое доры будут очень довольны :) зы "чужие" стили таким же образом добавляются

[arty]

все писатели букмарклетов такое используют

короче тоже можно сделать: забить на setAttribute, и нормально присваивать атрибуты напрямую: s.src='...'

getElementsByTagName работает не очень шустро, а скрипты можно в любое место страницы вставлять, поэтому проще (и короче) будет d.body.appendChild()

и w = window вы тоже не используете, сотрите ; )

[SamDark]

w = window остался от попапного букмарклета :) Не заметил. Спасибо.
Забить на setAttribute и вставлять в body попрбую.

[SamDark]

Проверил. Работает замечательно. Спасибо.

[Kessoron]

Один аспект - что произойдет, если страница еще не загружена, DOM не построен (к примеру, один из тучи банеров подтормаживает), а мы вызываем body.appendChild? Правильно - ошибка ;)

По-уму надо юзать что-то вроде body.insertBefore(scriptElement, body.firstChild), или добавлять линк в head, где проблем с построением DOM'а не возникнет

[SamDark]

Хорошее замечание.