instantsoft / icms2

Self-hosted Site Management System
https://instantcms.ru
GNU General Public License v2.0
294 stars 120 forks source link

Страницы и виджеты: список виджетов с картинками #1107

Closed asnenne closed 4 years ago

asnenne commented 4 years ago

Предлагаю добавить возможность к виджетам прикреплять картинку, а не только название. Как-то так:

3в3в3

Для этого надо в таблицу виджетов добавить поле image, где прописывается имя картинки. Картинку поставлять в корне дистрибутива и при установке грузить в какую-то конкретную папку. И дальше условие в шаблоне админки, если картинка указана, то ее выводить, а если нет, то не выводить. Работы на несколько минут, если знать, где копать.

Неверное будет отклонено с пометкой "не требует решения". На всякий случай попробую объяснить, для чего это может понадобиться. Когда-нибудь, возможно, в этой жизни, планирую запилить шаблон (мини-конструктор лендингов), там будет много виджетов разных со странными названиями. А если бы визуально было видно, что за виджет, было бы гораздо удобнее конечному пользователю. Как в списке доступных виджетов, так и в схеме позиций.

Спасибо.

fuzegit commented 4 years ago

Такой виджет с картинкой показывать только в блоке всех виджетов? На схеме обычно?

asnenne commented 4 years ago

Можно было бы и на схеме)))

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

Спасибо.

fuzegit commented 4 years ago

Выделение_110

Как то так устроит?

asnenne commented 4 years ago

В зависимости от картинки, она может сливаться с текстом. Тут можно было бы как-то так сделать: Шапка - в ней название и иконка перетаскивания Картинка под шапкой, если она есть для виджета. Ширина картинки 100%, а высота auto. 2019-11-07_151015 В схеме так же: в шапке название, кнопки и т.д. А если есть картинка, то она под шапкой. Чтобы не мешала.

fuzegit commented 4 years ago

Выделение_111

asnenne commented 4 years ago

Вот, то что надо)))

fuzegit commented 4 years ago

https://github.com/instantsoft/icms2/commit/19ef497a40fad4a08c1e670876b0062e790fc2a0 https://docs.instantcms.ru/dev/packages#блок-info

asnenne commented 4 years ago

Всё хорошо, кроме одного момента. Если картинка большая, то она в списке виджетов растягивается на всю ширину экрана. Ломается верстка вся. Предлагаю сделать так:

#cp-widgets-list li > img{max-width:250px;} Вместо 250px можно другое значение, подходящее для шаблона.

Спасибо.

asnenne commented 3 years ago

Всё хорошо, кроме одного момента. Если картинка большая, то она в списке виджетов растягивается на всю ширину экрана. Ломается верстка вся. Предлагаю сделать так:

#cp-widgets-list li > img{max-width:250px;} Вместо 250px можно другое значение, подходящее для шаблона.

Спасибо.

В 2.14 проблема исправлена в одном месте, но добавлена в другом))) 0

#cp-widgets-list li > img{max-width:200px;} надо было только для #cp-widgets-list li > img делать, а не для всех.

Спасибо.

asnenne commented 3 years ago

Вот так предполагалось 0

fuzegit commented 3 years ago

исправил https://github.com/instantsoft/icms2/commit/b9fed817a726c7ac68b8853d8517e7c8cef4fed4#diff-d33ff82509d184b61001b188570adef7599bc8eca225276822b4a4d7fbeda52aR769