1

I am using the script beneath to make a toggle menu onclick. This works fine, but I don't know if it is good to use Javascript with an onclick attribute. Would this be a problem? It is visible now in the DOM if you use e.g. Firebug. 

<a href="#" class="button" onclick="javascript:showElement('v-menu')">
<span>Click Here</span>
</a>
<ul id="v-menu" class="v-menu" style="display:none;">
</ul>

If it is harmful; how can I hide it? What should I have to do?

PS: I am using Wordpress and jQuery did not work there. THat's why I used this method.

4

2 回答 2

2

我不知道使用带有 onclick 属性的 Javascript 是否好

它不是。它违反了关注点分离。另请参阅渐进式增强和不显眼的 JavaScript。

addEventListener或者事件抽象库（包含在 YUI、jQuery 和其他中）通常是首选。

它现在在 DOM 中可见

那不是问题。它是客户端代码。客户可以看到。

我正在使用 Wordpress，而 jQuery 在那里不起作用。

Wordpress 不会阻止您使用 jQuery。

于 2013-07-01T10:04:57.787 回答
0

我不认为这有什么严重的问题，尽管很多人会建议将你的 JS 与你的 HTML 分开。

因此，您也可以给锚点一个 ID，然后在页面底部执行以下操作：

<script>
    document.getElementById("idname").onclick = function(){
        javascript:showElement('v-menu');
    };
</script>

...或将其放入脚本文件并使用该src属性引用该文件。

于 2013-07-01T10:07:33.020 回答