Цитата:
Сообщение от ghostpast
Приведение файлов skin/*/skin.php к общепринятым стандартам:
- теги HTML и их атрибуты пишутся прописными буквами;
- значения атрибутов тегов заключаются в двойные кавычки;
- размеры в атрибутах измеряются в пикселах, если не указано иное.
|
Почти "УРА"!
"УРА!" - за то, что думаем о стандартах.
"Почти" - за "программистский код".
Это ругательство у меня такое, когда мои программисты в верстке косячат.
Пример: 69-73 строки патча
PHP код:
+ <tr>
+ <td valign="top" width="14"><img height="13" src="skin/dark/img/1.gif" width="14" border="0"></td>
+ <td valign="top" width="100%" background="skin/dark/img/2.gif"></td>
+ <td valign="top" width="13"><img height="13" src="skin/dark/img/3.gif" width="13" border="0"></td>
+ </tr>
в двух ячейках стоит ширина в пикселях, в одной - проценты. При этом в ячейках с пикселями размеры ставить НЕ НАДО, потому что у них внутри стоят картинки, распирающие их на эту же ширину. Ну и, по мелочи, вместо установки valign="top" на каждую ячейку, достаточно было поставить его на всю строку. То есть, если по уму, то должно быть:
PHP код:
+ <tr valign="top">
+ <td><img height="13" src="skin/dark/img/1.gif" width="14" border="0"></td>
+ <td width="100%" background="skin/dark/img/2.gif"></td>
+ <td><img height="13" src="skin/dark/img/3.gif" width="13" border="0"></td>
+ </tr>
Если так по всему пройтись (я только за одно место глазом зацепился), то код получится и компактнее, и правильнее.