This QE/QR custom BBcode is ok?

[Sworm]

Hi all, i have added the php/html/code and all the custom bbcodes in my Quick Reply and Quick Reply. But before to confirm in my live forum, i've need of a coder opinion about.

This is the code that i have used to show it in my QR/QE:

In my postbit_quickedit and in my showthread_quick_reply
After

        <if condition="$show['quote_bbcode']">
            <td><img src="$stylevar[imgdir_editor]/separator.gif" width="6" height="20" alt="" /></td>
            <td><div class="imagebutton" id="{$editorid}_cmd_wrap0_quote"><img src="$stylevar[imgdir_editor]/quote.gif" width="21" height="20" alt="$vbphrase[wrap_quote_tags]" /></div></td>                      
        </if>

I have added

                <if condition="$show['code_bbcode']">
                        <td><img src="$stylevar[imgdir_editor]/separator.gif" width="6" height="20" alt="" /></td>
                        <td><div class="imagebutton" id="{$editorid}_cmd_wrap0_code"><img src="$stylevar[imgdir_editor]/code.gif" width="21" height="20" alt="$vbphrase[wrap_code_tags]" /></div></td>
            </if>
                <if condition="$show['html_bbcode']">
                <td><div class="imagebutton" id="{$editorid}_cmd_wrap0_html"><img src="$stylevar[imgdir_editor]/html.gif"  width="21" height="20" alt="$vbphrase[wrap_html_tags]" /></div></td>
            </if>
                <if condition="$show['php_bbcode']">
                <td><div class="imagebutton" id="{$editorid}_cmd_wrap0_php"><img src="$stylevar[imgdir_editor]/php.gif" width="21" height="20" alt="$vbphrase[wrap_php_tags]" /></div></td>
<td><img src="$stylevar[imgdir_editor]/separator.gif" width="6" height="20" alt="" /></td>
    </if>
$vBeditTemplate[extrabuttons]

It is working, but i have need one opinion about from a coder. It is looking ok? Or there's something to edit?

Thanks in advance