chukcha

В помощь верстальщикам?

Рекомендуемые сообщения

chukcha    830

Имею следующую схему размещения модулей на главной

 

latest

featured

bestseller

 

Каждый товар имеет опции, и хочется продавать без захода в товар.

 

Функционал готовый, написан скрипт, сверстан блок для опций,

С этим все в порядке.

Первая опция  - checked

 

 

И тут...

некоторые товары не имеют checked.

В коде есть, на странице нет.

 

Не долгие исследования показали, что input type="radio" name="nameoptNNN" находятся в одной области видимости имен,

т.е. было

<div class="options">
input type="radio" name="nameoptNNN" checked
input type="radio" name="nameoptNNN"
</div>

<div class="options">
input type="radio" name="nameoptNNN" checked
input type="radio" name="nameoptNNN"
</div>

Стало:

<form class="options">
input type="radio" name="nameoptNNN" checked
input type="radio" name="nameoptNNN"
</form>

<form class="options">
input type="radio" name="nameoptNNN" checked
input type="radio" name="nameoptNNN"
</form>

Очень много элементов форм находятся вне <form>

пытайтесь обернуть тегом  соответствующие элементы

Не забывайте, что <form> блочный элемент.

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
rb2    390

Очень много элементов форм находятся вне <form>

Это где?

В дефолтном шаблоне?

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
chukcha    830

А вы посмотрите

 

открываем product.tpl

 

Где <form>? Нет, а элементы формы есть.

 

или, например

Опции в категориях (модуль), например (radio)

Проектировщик об этом не задумался.

 

Или

product-filter

 

Почему не форма, а блок?

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
rb2    390

Почему не форма, а блок?

Потому что Даниэль - упёртый осёл и к его квалификации много вопросов. С этим мало кто спорит.

Я же не оспариваю это, а прошу показать, где это встречается.

Посмотрел, увидел. Спасибо.

В принципе-то, в Опенкарт вполне можно было ожидать того, что открывающая `form` где-то в header.tpl, а закрывающий тег в footer.tpl, но тут в шаблонах хотя бы стараются такого не делать и не тот случай. А случай как раз некорректной вёрстки.

Такое желательно пофиксить и предложить пулл реквест в главный репозиторий, но в ответ от Д. можно услышать традиционное "not required". Хотя такое может и примут - там кроме него теперь есть Джеймс Олсоп. Если отфутболят - в `opencart-ce`. Здесь такое примут с вероятностью 0.99. Если и там не примут (единственная причина - нежелание углублять разницу кодовой базы между проектами), тогда в ocStore. Здесь пофиг, т.к. он опять стал развиваться по старым рельсам - с большими отличиями в коде и ручными мержами.

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

Для публикации сообщений создайте учётную запись или авторизуйтесь

Вы должны быть пользователем, чтобы оставить комментарий

Создать учетную запись

Зарегистрируйте новую учётную запись в нашем сообществе. Это очень просто!

Регистрация нового пользователя

Войти

Уже есть аккаунт? Войти в систему.

Войти


  • Последние посетители   0 пользователей онлайн

    Ни одного зарегистрированного пользователя не просматривает данную страницу