search

fiction13 / pkg_radicalmicro

Radical Micro - решение для добавления микроразметки для CMS Joomla
GNU General Public License v3.0
8 stars 2 forks source link

RadicalMicro Meta Element #2

Closed alex-revo closed 2 years ago

alex-revo commented 2 years ago

Нужно доработать перевод на английский image

Это поле не нужно делать динамическим image Если там дискретный набор значений, то лучше задать его списком

fiction13 commented 2 years ago

Глобально я не вижу ничего плохого в добавлении динамики полю.

Но на самом деле я бы сделал это поле disabled, потому что оно может смущать неопытного пользователя, а с использованием других типов я ни разу не сталкивался.

Вот список типов для разметки - https://ruogp.me/#types

Вообще все поля настроек элементов формируются и добавляются в конфиг элемента динамически, поэтому для этого конкретного поля конечно можно сделать список, но он будет очень большой.

Переводы будут в следующей версии, пропустил.

alex-revo commented 2 years ago

Disabled - хороший вариант.

В целом этот элемент у меня не работает или я что-то не понимаю. На странице после добавления элемента появился только такой код image

fiction13 commented 2 years ago

Я тебе напишу, проверим.

alex-revo commented 2 years ago

Дима мне подсказал, оказывается этот плагин зависим от настроек в основном плагине, где они все по умолчанию выключены, поэтому никакой Og/Twitter разметки не появляется. Как минимум нужно добавить в описание плагина что зависимые настройки нужно искать в другом плагине. Как мамсимум по умолчанию включать все опции внутри основного плагина.

fiction13 commented 2 years ago

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

Пока оставлю открытым.

Сейчас в элементе делаются правки объединения типов (OG и Twitter), чтобы не добавлять для каждого типа свой элемент.

alex-revo commented 2 years ago

Согласен, в одном элементе будет удобней.

fiction13 commented 2 years ago

Настройки по умолчанию включил.

Добавление текста в элементы, что они зависят от настроек основного плагина, добавил в отдельный Issue.