Also this
Code:
<ul id="erupdate_menu">
<li><img src="/images/icons/fugue/plus.png" alt="{vb:phrase erupdate_menu_add}" title="{vb:phrase erupdate_menu_add}" onclick="openDialog('add')" /></li>
<vb:if condition="$publish == 1"><li><img src="/images/icons/fugue/minus.png" alt="{vb:phrase erupdate_menu_delete}" title="{vb:phrase erupdate_menu_delete}" onclick="openDialog('delete')" /></li></vb:if>
<vb:if condition="$publish == 1"><li><img src="/images/icons/fugue/tick.png" alt="{vb:phrase erupdate_menu_publish}" title="{vb:phrase erupdate_menu_publish}" onclick="openDialog('publish')" /></li></vb:if>
<vb:if condition="$publish == 1"><li><img src="/images/icons/fugue/slash.png" alt="{vb:phrase erupdate_menu_hide}" title="{vb:phrase erupdate_menu_hide}" onclick="openDialog('hide')" /></li></vb:if>
<vb:if condition="$publish == 1"><li><img src="/images/icons/fugue/pencil.png" alt="{vb:phrase erupdate_menu_edit}" title="{vb:phrase erupdate_menu_edit}" onclick="openDialog('preedit')" /></li></vb:if>
</ul>
could be placed into a if condition instead of 4 or?