rfc - helpful change to assist in styling payment confirmation pages etc

Started by Jason Farmer, October 27, 2012, 17:44:38 PM

Previous topic - Next topic

Jason Farmer

request for change - helpful change to assist in styling payment confirmation pages etc

vm2.012b

/administrator/components/com_virtuemart/plugins/vmpsplugin.php line 810
$html = "<tr >\n<td " . $class . ">" . $key_text . "</td>\n <td align='left'>" . $value . "</td>\n</tr>\n";


the css class is applied to the first cell in the row, would much prefer it to be applied to the whole row (as css does not have a parent selector at the moment..)  and change the first cell to be a th - all makes much more sense symantically (imho)...
would also be good to remove the 'align="left"' style - if you're going to create html in core files, please make it so that we can style it freely.

ie.
$html = "<tr " . $class . "><th>" . $key_text . "</th><td>" . $value . "</td></tr>\n";


Thanks


[tr][td][/td][td]
Development[/td][td]Production[/td][/tr]
[tr][td]VirtueMart   [/td][td]
2.0.12b
[/td][td]
1.1.3
[/td][/tr]
[tr][td]Joomla!   [/td][td]
2.5.6
[/td][td]
1.5.14
[/td][/tr]
[tr][td]Mysql  [/td][td]
5.5.8
[/td][td]
5.0.51
[/td][/tr]
[tr][td]PhP   [/td][td]
5.3.5
[/td][td]
5.2.4
[/td][/tr]
[/table]