Bug in classes declaration on virtuemart admin files!

[poisom]

Please make the appropriate fixes because now virtuemart conflicts with some other plugins i use in administrator panel.

So on file:
     /administrator/components/com_virtuemart/views/product/tmpl/product_edit_information.php

line 44
       

Code Select Expand

<input class="required" type="text" class="inputbox"  name="product_name" id="product_name" value="<?php echo $this->product->product_name; ?>" size="32" maxlength="255" />

should be
       

Code Select Expand

<input class="required inputbox" type="text" name="product_name" id="product_name" value="<?php echo $this->product->product_name; ?>" size="32" maxlength="255" />

on file:
        /administrator/components/com_virtuemart/views/product/tmpl/product_edit_price.php

line 28
       

Code Select Expand

<table class="adminform" class="productPriceTable">

should be
       

Code Select Expand

<table class="adminform productPriceTable">



You have to use multiple classes inside the class attribute, separated by whitespace and not in different class attribute. This is the W3C HTML5 recommendation.

Please let me know if and when you will implement those fixes!
Thank you!

[Milbo]

is implemented, I see that as a kind of typo. We all know how it should be, thank you. If you find more, dont hesitate to tell us.

[nppsbh]

Hi.
Found some more typos

file
/administrator/components/com_virtuemart/views/product/tmpl/product_edit_price.php
line 28

Code Select Expand

<table class="adminform" class="productPriceTable">

file
/administrator/components/com_virtuemart/views/product/tmpl/product_edit_customer.php
line 45, 46

Code Select Expand

</div>
</label>
should be

Code Select Expand

</label>
</div>

line 61
missing </div>

file
/administrator/components/com_virtuemart/views/product/tmpl/product_edit_information.php
line 255 remove </tr>
line 282~291 shouldn't it be inside a table?

file
/administrator/components/com_virtuemart/views/product/tmpl/product_edit_custom.php
line 144 think is missing a closing div

In shop->shop and configuration->configuration there are also some typos

[Milbo]

Thank you, yepp. Good work,... wow some of these are really old. The "problem" is here the autocorrection of the browsers.