0 Пользователей и 1 Гость просматривают эту тему.
  • 12 Ответов
  • 2184 Просмотров
*

beliyadm

  • Легенда
  • 9758
  • 1664 / 66
  • Севастополь, Россия
Opencart и все все все
« : 14.03.2018, 02:48:16 »
В юмор наверно писать, куда же еще.
Достался мне вчера сайт на Opencart версии 1.5.5.1.2 от 14-года (вроде бы не очень старый двиг) и задача - натянуть на него адаптив (медиа квери).
Полез разбираться и решать и ...

1 - открыть div в одном файле шаблона и закрыть его в другом - это нормальная практика этого движка или левая нога верстальщика так захотела?

2 - имеем разметку базовую (к примеру три колонки везде с одними классами) описана в \template\common\header.tpl (почему в хедер?), ладно изменил, открываю другую страницу, а там оказывается отвечает шаблон \template\information\information.tpl с той же самой разметкой как в header.tpl только в блоке контента (центральная колонка) своя информация.
И так по каждой странице. В итоге нашел десяток одинаковых разметок для разных страниц одного сайта.
В Joomla мы стали бы делать 10 разных шаблонов для совершенно одинакового вывода информации в плане структуры?

3 - захожу в админку, изменяю материал, хочу применить посмотреть а нет такой опции, плюс заголовок не служит ссылкой на изменить. Только сохранить\отменить. При сохранении попадаю в общий список и снова ищу тот же материал для редактирования. Фильтрации поиска никакого в помине нет по статьям.

Вопросы 1 и 2 даже не знаю кому адресовать - разрабам или верстала так схитрил? Ну да ладно, переходим к вопросам именно тому, кто ентот сайт делал

То что верстала описывал стили через #ID промолчу, нет классов у базовых элементов разметки (которые дублируются по разным файлам). Про стиль верстки даже говорить не буду, меня упрекают, что мол работаю не по методике, без БЭМ и фреймворков), но здесь же.... В горизонтальной полосе известной ширины и высоты расположить блок надо только через абсолют.

В JS файле нахожу следующую функцию с комментом (даже в 2014 году были решения на уровне разметки\стилей)

Код
    // Фиксим траблы с левым отступ при загрузве страницы
    fixMarginLeftAndHeightBlock();
    function fixMarginLeftAndHeightBlock()
    {
            /* Каждому четвертому блоку убираем левый отступ, чтобы была ровная колонка товаров */
            $('.product-grid > div:nth-child(4n+1)').css({marginLeft:"0px"});

            /* Ставим одинаковую высоту блоку с названием товара */
            setEqualHeight($(".product-grid .name"));
    }

Ну и еще однотипного
Код
    /* Каждому третему блоку убираем левый отступ, чтобы была ровная колонка товаров */
    $('#content .box-product > div:nth-child(3n+1)').css({marginLeft:"0px"});

Даже я на уровне JS не фиксю баги стилей или разметки.


Идем дальше, в контенте нахожу такой код (который физически не может быть сформирован даже вордом)
Код
<img alt="" src="картинка" style="font-family: tahoma, geneva, sans-serif; font-size: 14px; text-align: center;" />
такого я точно никогда за 15 лет не видел, что это и зачем?

Этот момент просто шедеврален - две строки в таблице!

Про три вложенных друг в друга дива с фиксированной одинаковой шириной 1012px даже говорить не буду. Зачем так делать? При этом нет ни max\min-width, тупо фиксированный контейнер для дочерний блоков которого указана его же ширина.


За свою практику раз десятый сталкиваюсь с опенкартом и везде одинаковые проблемы - div открылся тут закрылся там, дублирование сущностей разметки.
Многие "умные" умы хают Joomla за доступность и тупость, но лучше системы в мире открытых CMS (да и закрытых тоже) я не встречал.
Простите за простыню, надо было выплакаться, у меня там в блокнотике еще много чего зафиксировано, но это пока самые перлы что нашел.

« Последнее редактирование: 14.03.2018, 02:52:18 от beliyadm »
Все истины, которые я хочу вам изложить, — бесстыдная ложь. Сделать всё хорошо
TLG: @Beliyadm
*

effrit

  • Легенда
  • 10132
  • 1118 / 13
  • effrit.com
Re: Opencart и все все все
« Ответ #1 : 14.03.2018, 06:44:32 »
хы. ды ты просто повторил мой скорбный путь. я же месяц назад подобную тему создавал  ;D
могу помочь только с кнопкой "применить" - которая является самым популярным расширением для опенкарта, так что скачай, установи и возрадуйся  ;D
я так понял, там 1 человек это все "развивает", так что не удивительно, что никаких революций с двигом не происходит с рождения.
*

beliyadm

  • Легенда
  • 9758
  • 1664 / 66
  • Севастополь, Россия
Re: Opencart и все все все
« Ответ #2 : 14.03.2018, 10:11:15 »
хы. ды ты просто повторил мой скорбный путь. я же месяц назад подобную тему создавал  ;D
тоже думал что не одинок, пытался найти на форуме но не нашел, дай ссылку, перекину туда чтоб не плодить сущности

могу помочь только с кнопкой "применить" - которая является самым популярным расширением для опенкарта, так что скачай, установи и возрадуйся  ;D
я так понял, там 1 человек это все "развивает", так что не удивительно, что никаких революций с двигом не происходит с рождения.
Не, кнопка не нужна, адаптив натянул насколько возможно и забил.
Но мне казалось что двиг вроде как популярный, не думал что один человек разраб... :)
Все истины, которые я хочу вам изложить, — бесстыдная ложь. Сделать всё хорошо
TLG: @Beliyadm
*

AlekVolsk

  • Гуру
  • 6915
  • 415 / 4
Re: Opencart и все все все
« Ответ #3 : 14.03.2018, 10:31:42 »
открыть div в одном файле шаблона и закрыть его в другом - это нормальная практика этого движка или левая нога верстальщика так захотела
в zoo частенько та же хрень


И так по каждой странице. В итоге нашел десяток одинаковых разметок для разных страниц одного сайта.
хмм, модыкс так же утроен, не сталкивался разве еще?
« Последнее редактирование: 14.03.2018, 10:34:51 от AlekVolsk »
*

beliyadm

  • Легенда
  • 9758
  • 1664 / 66
  • Севастополь, Россия
Re: Opencart и все все все
« Ответ #4 : 14.03.2018, 10:33:13 »
в zoo частенько та же хрень
вопрос только один - зачем и кто так делает?  *suicide*
Все истины, которые я хочу вам изложить, — бесстыдная ложь. Сделать всё хорошо
TLG: @Beliyadm
*

kern.USR

  • Давно я тут
  • 827
  • 68 / 1
Re: Opencart и все все все
« Ответ #5 : 14.03.2018, 12:32:17 »
хмм, модыкс так же утроен, не сталкивался разве еще?
Он так был раньше устроен. Последние 2-3 года там всё можно свести к одному шаблону и куче чанков, подключающихся при разных условиях.
1 - открыть div в одном файле шаблона и закрыть его в другом - это нормальная практика этого движка или левая нога верстальщика так захотела?
Это также нормальная практика в битриксе и вордпрессе. Задолбали уже, если честно. Понабирают идиотов в штат, а потом другие люди страдают.
И так по каждой странице. В итоге нашел десяток одинаковых разметок для разных страниц одного сайта.
Всё теже идиоты, которые не слышали про повторное использование кода
*

beliyadm

  • Легенда
  • 9758
  • 1664 / 66
  • Севастополь, Россия
Re: Opencart и все все все
« Ответ #7 : 14.03.2018, 17:10:07 »
Столкновение с OpenCart
Мда, тебе было хуже :) Благо на меня только адаптацию верстки под мобильные повесили.

Это также нормальная практика в битриксе и вордпрессе. Задолбали уже, если честно. Понабирают идиотов в штат, а потом другие люди страдают.
Кстати да, в вордпрессе тоже, причем в дефолтном шаблоне наткнулся лет 7 назад когда начал им интересоваться, собственно увидев это и хранение настроек в БД (хрен вам быстренько перенести сайт на другой хостинг) и забил на него.
Все истины, которые я хочу вам изложить, — бесстыдная ложь. Сделать всё хорошо
TLG: @Beliyadm
*

Aleks.Denezh

  • Живу я здесь
  • 3406
  • 428 / 4
Re: Opencart и все все все
« Ответ #8 : 14.03.2018, 17:13:58 »
del
« Последнее редактирование: 04.05.2018, 22:32:19 от Aleks.Denezh »
*

beliyadm

  • Легенда
  • 9758
  • 1664 / 66
  • Севастополь, Россия
Re: Opencart и все все все
« Ответ #9 : 14.03.2018, 17:18:06 »
Ну не в десятки, раза в полтора, по быстрому нагуглил https://itrack.ru/research/cmsrate/
Все истины, которые я хочу вам изложить, — бесстыдная ложь. Сделать всё хорошо
TLG: @Beliyadm
*

Aleks.Denezh

  • Живу я здесь
  • 3406
  • 428 / 4
Re: Opencart и все все все
« Ответ #10 : 14.03.2018, 17:30:59 »
del
« Последнее редактирование: 04.05.2018, 22:32:15 от Aleks.Denezh »
*

beliyadm

  • Легенда
  • 9758
  • 1664 / 66
  • Севастополь, Россия
Re: Opencart и все все все
« Ответ #11 : 14.03.2018, 17:39:44 »
ну три раза это ж не десятки :)
я просто не в ту строку посмотрел
Все истины, которые я хочу вам изложить, — бесстыдная ложь. Сделать всё хорошо
TLG: @Beliyadm
*

Aleks.Denezh

  • Живу я здесь
  • 3406
  • 428 / 4
Re: Opencart и все все все
« Ответ #12 : 14.03.2018, 18:33:16 »
del
« Последнее редактирование: 04.05.2018, 22:32:11 от Aleks.Denezh »
Чтобы оставить сообщение,
Вам необходимо Войти или Зарегистрироваться