Селектбоксы, баттоны и прочее

Issue #81 open
Alexsander Vyshnyvetskyy created an issue

Селектбоксы, баттоны и прочее... включите пожалуйста обратно

Comments (14)

  1. Ivan Kudryavsky

    Стилизация селектов занимала около 5 секунд времени, на страницах, где их было много. Это критическое значение. По этому поступила задача на оптимизацию. Пришлось отказаться от Jquery UI. Возможно где-то криво затер, и остались баги. Но каталог и раздел структуры работают нормально. Если есть конкретный список ошибок, напишите пожалуйста. Исправлю в срочном порядке.

    Даже сейчас после получения контента от сервера, на JS скрипты уходит около одной секунды, что тоже много. Но что еще грузит интерфейс искать не стал.

  2. Alexsander Vyshnyvetskyy reporter

    Да я не против)) Иван, просто как то неожиданно, взяли и отказались Jquery UI, сделали админку глючной... это как бы не ваша личная система, мнение других нужно учитывать.

    Я заметил багу со слетающим шаблоном для страниц в структуре, и там опечатка в коде ranatr, но и это не меняет ситуацию, и selected и selected="selected"всё равно не проставляют активным выбранный элемент, во всяком случае в Safari 9, Chrome 31 на OSX.

    Складывается впечатление, что вы либо не тестировали правки перед отправкой, либо это особенность браузеров под яблоком (что вряд ли, но возможно).

  3. Alexsander Vyshnyvetskyy reporter

    Обычно люди оптимизируют архитектуру вывода, нежели упрощают интерфейс... аля "давайте снимем двери и багажник, чем выкинем хлам из машины..."

  4. Ivan Kudryavsky

    Задачу поступила от главного по движку. Он одобрил. Если у вас есть время оптимизировать JS, то мы будем рады. Я посмотрел, но ничего не нашел. Учитывая то, что плагин JQuery UI писали не школьники, я думаю там все пучком. Просто в санте очень много селектов на страницах и 6 секунд на селекты - стыд! Вывод один - убрать стилизацию.

    Правки я тестировал. Я проверил сохранение состояния селекта в товарах и на странице структуры где "действия для меток". Проверять все селекты, которые есть на сайте - не самое лучшее дело. Не думал, что в свойствах страницы что-то особенное. Кстати очень удивлен этому факту. Также, от людей, которые ежедневно используют админку, жалоб не поступало. Вот и выкатил. Исправлю в ближайшие дни.

    На сколько я понимаю, там селект сохраняется, просто значение не выставляется после апдейта страницы

  5. Bubek

    Обычно люди оптимизируют архитектуру вывода, нежели упрощают интерфейс... аля "давайте снимем двери и багажник, чем выкинем хлам из машины..."

    С языка снял :)

  6. Ivan Kudryavsky

    Если есть время, то кто-то из вас может этим заняться. Вернуть Jquery UI можно без проблем.

  7. Alexsander Vyshnyvetskyy reporter

    "Если есть время, то кто-то из вас может этим заняться. Вернуть Jquery UI можно без проблем."

    Мы уже у себя вернули, не волнуйтесь... типа решили задачу и создали задачу другим)) Если пойти дальше, ну в вашей оптимизации, где очищенный CSS от теперь не нужных стилей чекбоксов, кнопок и прочего... где пересобранный Jquery UI.

    Где наконец таймлайн тесты, что проблема тормозов, о которых вы пишите именно из-за большого количества чекбоксов и прочего из Jquery UI... нет Вы не обязаны никого в чем либо убеждать, но можно было бы осветить проблему, прежде чем вырезать так топорно. Смотри вместе бы все правильно бы и сделали, все проверили, протестировали бы... а теперь, "ну рябят у вас если время есть - вы и поправляйте дальше, я что мог всё сделал"

  8. Ivan Kudryavsky

    Не коверкайте мои слова! Я сказал, что этот баг исправлю. А вот на счет бесполезной оптимизации архитектуры - да, я сказал, что делайте сами. Я в этом не вижу смысла, так как 4я версия санты на подходе, какой смысл оптимизировать архитектуру.

    И слишком много шума, по поводу одного неработающего списка.

  9. Alexsander Vyshnyvetskyy reporter
    • changed status to open

    Вновь открою задачу... Практика показала, что многие не согласны с данным решением выпилить кнопки и селектбоксы, да прирост в производительности АИ очень скромный.

    Есть ветка (jquery-ui), поддерживаемая мною около двух лет. Естественно пушить изменения сразу в две ветки с jquery-ui и без - лишняя работа.

    Предлагаю слить!? Отпишитесь кто, что думает.

  10. Nikolai Latskov

    За jquery-ui. Библиотеки можно ссылками сделать с cdnjs.com/libraries . Меньше запросов на сервер будет

  11. Ivan Kudryavsky

    Попробую на выходных провести тест на крупном магазине. Если разница в секунду, то я за, если как была раньше в 5 секунд и больше, то сомнительное решение. Отпишу потом результаты

  12. Ivan Kudryavsky

    Тестировал на странице категорий. Мало того, что оно само по себе грузится от 20 секунд, так еще и обработка скриптов и рендер занимает много времени с JUI.

    Без JUI все отрабатывает (после получения ответа от сервера) менее чем за секунду, с ним, если верить FF, то секунд 7 (хром показал колоссальные 18 секунд, возможно я как-то не так замеры делал). В общем скрины прикладываю, судите сами.

    Я не хотел бы добавлять JUI. Возможно мой случай единичный и связан с железом (8 gb RAM, i5 4 поколения, софт на SSD, сайты на HDD). Коротко о базе каталога: 84 группы, 435 категорий, 26 000 товаров

    Chrome: chrome.png

    Chrome UI: chrome_ui.png

    Firefox: firefox.png

    Firefox UI: firefox_ui.png

  13. Log in to comment