Safari 11 и улетающий в корзину товар

Правила раздела: faq.php?mode=okay
Модератор: Модераторы

makki M
makki M
Репутация: 199
Сообщения: 697
Зарегистрирован: 12.08.2016
С нами: 7 лет 7 месяцев
Откуда: Киев
Сайт

Сообщение #1 makki » 20.09.2017, 11:14

На macOS в новой версии Safari 11, эффект улетающей в корзину картики товара - не работает. Может это хороший знак? Лично мое мнение что эффект выглядит дешево. Куда лучше простое уведомление "товар добавлен в корзину"
Cоздание и расширение функционала интернет-магазина на платформе OkayCMS 2 (с 3-й и 4-й версией не работаю)

korshunov
korshunov
Репутация: 146
Сообщения: 1854
Зарегистрирован: 03.12.2015
С нами: 8 лет 3 месяца
Skype

Сообщение #2 korshunov » 20.09.2017, 11:57

Скорее всего, это не относится к багам CMS.

Если не нравится эффект, то отключить его - проще простого.
Сделать уведомление - не особо сложно - есть много готовых способов.
Вот совсем свеженький:
viewtopic.php?f=10&t=535

Только что Вы будете говорить, если кто-то напишет - "а вот у меня на такой-то платформе в таком-то браузере новый эффект не работает" ?

makki M
makki M
Репутация: 199
Сообщения: 697
Зарегистрирован: 12.08.2016
С нами: 7 лет 7 месяцев
Откуда: Киев
Сайт

Сообщение #3 makki » 20.09.2017, 18:27

Спасибо, что направили меня в мою тему :)

Что касается указанной проблемы, считаю что отсутствие совместимости с браузером - это недостаток движка, а не браузера. Кстати говоря, в админке до сих пор не исправили несколько багов, которые есть при работе в Safari
Cоздание и расширение функционала интернет-магазина на платформе OkayCMS 2 (с 3-й и 4-й версией не работаю)

OkayCMS M
Администратор
Аватара
OkayCMS M
Администратор
Репутация: 216
Сообщения: 1627
Зарегистрирован: 12.11.2015
С нами: 8 лет 4 месяца
Сайт Skype

Сообщение #4 OkayCMS » 20.09.2017, 20:35

makki писал(а):Что касается указанной проблемы, считаю что отсутствие совместимости с браузером - это недостаток движка, а не браузера. Кстати говоря, в админке до сих пор не исправили несколько багов, которые есть при работе в Safari
А можно об этом поподробнее, может успеем в новую версию добавить?

makki M
makki M
Репутация: 199
Сообщения: 697
Зарегистрирован: 12.08.2016
С нами: 7 лет 7 месяцев
Откуда: Киев
Сайт

Сообщение #5 makki » 21.09.2017, 04:59

1) Не удаляются одиночно из перечня Товары, Категории, Бренды…
2) Каталог - Свойства. Выпадающий список Использовать в категориях не перекрывает футер. См.скриншот. Этот баг наблюдается и в других баузерах.
Screen Shot.jpg
Cоздание и расширение функционала интернет-магазина на платформе OkayCMS 2 (с 3-й и 4-й версией не работаю)

OkayCMS M
Администратор
Аватара
OkayCMS M
Администратор
Репутация: 216
Сообщения: 1627
Зарегистрирован: 12.11.2015
С нами: 8 лет 4 месяца
Сайт Skype

Сообщение #6 OkayCMS » 21.09.2017, 08:32

О второй проблеме знаем, уже исправили.
Первую не можем воспроизвести. С какого устройства/браузера не удаляется?

makki M
makki M
Репутация: 199
Сообщения: 697
Зарегистрирован: 12.08.2016
С нами: 7 лет 7 месяцев
Откуда: Киев
Сайт

Сообщение #7 makki » 21.09.2017, 11:17

OkayCMS писал(а):О второй проблеме знаем, уже исправили.
Первую не можем воспроизвести. С какого устройства/браузера не удаляется?
macOS Sierra (10.12.6), браузер Safari 11.0
См.видео

Как видно на видео, если не поставить вручную галочку возле удаляемого товара, ничего не происходит.
Cоздание и расширение функционала интернет-магазина на платформе OkayCMS 2 (с 3-й и 4-й версией не работаю)

makki M
makki M
Репутация: 199
Сообщения: 697
Зарегистрирован: 12.08.2016
С нами: 7 лет 7 месяцев
Откуда: Киев
Сайт

Сообщение #8 makki » 28.09.2017, 06:16

Если в файле backend/design.html/index.tpl в js функции

Код: Выделить всё

function success_action ($this){
вместо

Код: Выделить всё

                if ( !(/Android|webOS|iPhone|iPad|iPod|BlackBerry/i.test(navigator.userAgent)) ) {
                    $this.closest(".fn_row").find('input[type="checkbox"][name*="check"]').trigger("click");
                }

напишем

Код: Выделить всё

                if ( !(/Android|webOS|iPhone|iPad|iPod|BlackBerry|Version\/[\d\.]+.*Safari/i.test(navigator.userAgent)) ) {
                    $this.closest(".fn_row").find('input[type="checkbox"][name*="check"]').trigger("click");
                }

тогда в Safari удаление работает корректно.
Последний раз редактировалось makki 28.09.2017, 11:20, всего редактировалось 1 раз.
Cоздание и расширение функционала интернет-магазина на платформе OkayCMS 2 (с 3-й и 4-й версией не работаю)

korshunov
korshunov
Репутация: 146
Сообщения: 1854
Зарегистрирован: 03.12.2015
С нами: 8 лет 3 месяца
Skype

Сообщение #9 korshunov » 28.09.2017, 10:36

1. Проверялось ли, не испортит ли это работу с Safari 10 и другими младшими версиями?
2. Набор Android|webOS|iPhone|iPad|iPod|BlackBerry встречается в CMS еще в некоторых местах, одно из них - в том же файле. Там тоже надо менять? Если да, то как-то унифицировать надо бы, чтоб не плодить один код в разных местах...

makki M
makki M
Репутация: 199
Сообщения: 697
Зарегистрирован: 12.08.2016
С нами: 7 лет 7 месяцев
Откуда: Киев
Сайт

Сообщение #10 makki » 28.09.2017, 11:25

korshunov писал(а):1. Проверялось ли, не испортит ли это работу с Safari 10 и другими младшими версиями?
Код взят из stackoverflow. Там написано - It identifies Safari 3.0+ and distinguishes it from Chrome.
2. Набор Android|webOS|iPhone|iPad|iPod|BlackBerry встречается в CMS еще в некоторых местах, одно из них - в том же файле. Там тоже надо менять? Если да, то как-то унифицировать надо бы, чтоб не плодить один код в разных местах...
Во втором месте в том же файле менять не надо.

Упростил немного код в первоначальном посте.
Cоздание и расширение функционала интернет-магазина на платформе OkayCMS 2 (с 3-й и 4-й версией не работаю)


Название раздела: Баг-репорт OkayCMS (Сообщения об ошибках)
Правила раздела: faq.php?mode=okay

Быстрый ответ


Введите код в точности так, как вы его видите. Регистр символов не имеет значения.
Код подтверждения

   

Вернуться в «Баг-репорт OkayCMS (Сообщения об ошибках)»

Кто сейчас на форуме (по активности за 5 минут)

Сейчас этот раздел просматривают: 18 гостей