SamDark Jul 2 2008 at 15:33

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

1 min

1.2K

JavaScript*

Comments 25

Fanatik Jul 2 2008 at 16:13

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

-1

Antohins Jul 2 2008 at 16:53

стопудово

-1

amgorb Jul 2 2008 at 16:15

молодец!

-2

M_K Jul 2 2008 at 16:29

Спасибо!

-1

sha1dy Jul 2 2008 at 16:30

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

prolis Jul 2 2008 at 16:36

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

SamDark Jul 2 2008 at 16:53

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

SamDark Jul 2 2008 at 16:57

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

SamDark Jul 2 2008 at 16:58

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

lahmatiy Jul 2 2008 at 19:27

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

SamDark Jul 2 2008 at 19:29

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

lahmatiy Jul 2 2008 at 19:43

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

SamDark Jul 2 2008 at 19:54

Красиво!

SamDark Jul 2 2008 at 20:00

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

lahmatiy Jul 2 2008 at 20:14

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



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

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

SamDark Jul 2 2008 at 20:26

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

lahmatiy Jul 2 2008 at 20:06

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



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 Jul 2 2008 at 20:13

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

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

SamDark Jul 2 2008 at 17:00

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

serf Jul 2 2008 at 17:07

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

arty Jul 2 2008 at 17:54

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

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

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

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

SamDark Jul 2 2008 at 17:57

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

SamDark Jul 2 2008 at 18:08

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

Kessoron Jul 10 2008 at 14:04

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

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

SamDark Jul 16 2008 at 18:07

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

Show the best of all time