Как я могу остановить появление нескольких букмарклетов? - PullRequest
Новая
       5

Как я могу остановить появление нескольких букмарклетов?

0 голосов
/ 14 июня 2011

Я нахожусь в процессе создания букмарклета, который выскакивает div с различными вещами в нем ... когда вы нажимаете на ссылку, чтобы дважды открыть букмарклет, появляются два букмарклета.Как я могу предотвратить это?

index.html: 

<html>
<head>
<title>Bookmarklet Home Page</title>
<link rel="shortcut icon" href="favicon.ico" />
</head>
<body>
<a href="javascript:(function(){code=document.createElement('SCRIPT');code.type='text/javascript';code.src='code.js';document.getElementsByTagName('head')[0].appendChild(code)})();">click here</a>
</body>
</html>

code.js: 

function toggle_bookmarklet() {
    bookmarklet = document.getElementById("bookmarklet");
    if (bookmarklet.style.display == "none") {
        bookmarklet.style.display = "";
    }
    else {
        bookmarklet.style.display = "none";
    }
}
div = document.createElement("div");
div.id = "bookmarklet";
div.style.margin = "auto";
div.style.position = "fixed";
content = "";
content += "<a href='javascript:void(0);'><div id='xbutton' onClick='javascript:toggle_bookmarklet();'>x</div></a>";
div.innerHTML = content;
document.body.appendChild(div);

1 Ответ

2 голосов
/ 14 июня 2011

Просто проверьте наличие div перед его созданием. 

var div = document.getElementById("bookmarklet");
if (!div)
{
    div = document.createElement("div");
    div.id = "bookmarklet";
    div.style.margin = "auto";
    div.style.position = "fixed";
}

Кроме того, поскольку у вас уже есть глобальная ссылка на div, вам не нужно искать ее по идентификатору в toggle_bookmarklet. Вы можете просто сослаться на div. Я бы попытался выбрать более уникальное имя, чтобы избежать столкновения с именами.

Редактировать: В этом отношении, если вы собираетесь использовать глобальную переменную, вы можете упростить дальнейшее. Даже не называйте его идентификатором, просто используйте глобальную ссылку: 

function toggle_bookmarklet() {
    bookmarkletEl.style.display = bookmarkletEl.style.display == "none" ? "" : "none";
}
if (!window.bookmarkletEl) {
    var bookmarkletEl = ddocument.createElement("div");
    bookmarkletEl.style.margin = "auto";
    bookmarkletEl.style.position = "fixed";
}
...