Рейтинг: 3.3265  
На голосовании
Предложил AndreyChursin 16.08.2017 12:54:17

Bitrix, про MVC знаешь? - не, не слышал.. js строит верстку для шаблона!(

Компонент корзина(sale.basket.basket) написан край как неудачно! И не только он...

В js идет построение верстки, но mvc и шаблонное проектирование вообще против такой парадигмы программирования! js должно только менять значения, а не конструировать верстку. Для js можно оставлять id в верстке и их же передавать массивом в js для дальнейшей работы с данными.

Сейчас же, для банальной стилизации компонента приходится его переписывать почти на половину!
Предлагаю разработчикам пересмотреть стиль написания стандартных шаблонов, для уменьшения стоимости работы по персонализации их CMS, а соответственно и увеличения популярности для конечных клиентов!

p.s. там же, в sale.basket.basket/class.php видел строки ужаса, где ресайзят картинки! в ЯДРЕ компонента, РЕСАЙЗ картинок, и только на 110x110px!
Рейтинг: 30.2462  
На голосовании
Предложил Беликов Олег 17.10.2013 11:56:00

Убрать устаревшый HTML и отклонения от MVC

  1. Теги <font> убрать из шаблонов сообщений (ошибки, уведомления) и постраничной навигации в компонентах (Validation Output: The font element is obsolete)
  2. Отказаться от  CDBResult::GetNavPrint() использующий зашитый шаблон постранички с тегами <font> (отклонение от MVC)
  3. Атрибут border="0" тега img убрать при вставки изображения из медиатеки (Validation Output: The border attribute is obsolete)
  4. В компоненте form.result.new теги полей создаются в component.php, не приходят имена полей. А что если мне нужен HTML5 и нужны свои типы инпутов? Что если нужен атрибут "placeholder" или класс поля и т.д.? (отклонение от MVC)
  5. Компонет iblock.element.add - аналогично пункту 4