Правки в дефолтном шаблоне двойки

#1 19 декабря 2016 в 21:12
Здравствуйте!
Предлагаю здесь высказываться о замеченных вами недочётах дефолтного шаблона. Подчёркиваю: речь только о шаблоне.
Очень прошу высказываться внятно, прилагать скрины как есть и как хотелось бы, предлагать решения (если они у вас есть).
Группе, к которой вы принадлежите, запрещено просматривать этот скрытый текст

Цель: сделаем что можем самостоятельно, обратимся к разработчикам внести что посчитают нужным в релиз.
Всё, что буду писать ниже не есть окончательным и требует обсуждения. Изменённые файлы прикрепляю к записи. Стили, которые не понятно на 100% куда вставлять, — в файле add.css
1. В формах обязательные к заполнению поля не отмечены никак
Решение: templates\default\assets\ui\form.tpl.php:

  1.  
  2. $classes = array(
  3. 'field',
  4. 'ft_'.strtolower(substr(get_class($field), 5))
  5. );
  6.  
заменить на:
  1.  
  2. if($field->getOption('is_required')){
  3. $classes = array(
  4. 'field',
  5. 'ft_'.strtolower(substr(get_class($field), 5)),
  6. 'reguired'
  7. );
  8. } else {
  9. $classes = array(
  10. 'field',
  11. 'ft_'.strtolower(substr(get_class($field), 5))
  12. );
  13. }
  14.  
Ну и стиль для примера в add.css

2. При добавлении (редактировании) типа контента кнопка "сохранить" есть, а "отменить" нет
Не полноценное решение: templates\default\controllers\content\item_form.tpl.php после

  1. 'action' => '',
добавить:
  1. 'cancel' => array('show' => true, 'href' => href_to($ctype['name'])),
Надо добавить как-то проверку для ссылки на случай, если в типе контента запрещён просмотр списка.
Сделано согласно предложению, г-на Loadырь здесь: instantcms.ru/forum/thread26235-1.html#251961

3. Слишком маленькие иконки "поделиться" || слишком близко находятся друг возле друга(на мобилках трудно попасть)
Вариант решения (увеличиваем отступ): в add.css

4. Текущий стиль для элементов поля "Набор изображений":
  1.  
  2. .first_type_images, .second_type_images {
  3. margin: 0 0 15px 0;
  4. }
  5.  
Если будет второя строка, будут "прилипать"
Вариант решения: в theme-content.css, стр. 521, 524

5. Поля не группируются в группы при выводе (только при заполнении)
Решено в этой теме: instantcms.ru/forum/thread26144-1.html
Не забудьте поблагодарить г-на Val instantcms.ru/users/Val

6. На маленьких разрешениях скрываются иконки редактора при добавлении комметариев.
Вариант решения в theme-layout.css, строка 635

7. Остутсвует кнопка "отменить" при создании категории в типе контента с фронта
Решение — файл templates\default\controllers\content\category_form.tpl.php
Сделано аналогично коду, предолженному г-ном Loadырь

8. Пункт "Главная" меню (селект на маленьких разрешениях) не содержит текст
Вариант решения: — core.js, строка 31 архива
=================================================
Группе, к которой вы принадлежите, запрещено просматривать этот скрытый текст
#2 20 декабря 2016 в 06:19
1. Лучше заменить на это:
  1. $classes = array(
  2. 'field',
  3. 'ft_'.strtolower(substr(get_class($field), 5))
  4. );
  5. if($field->getOption('is_required')){ $classes[] = 'reguired'; }
2. Можно добавить как-то так
  1. 'cancel' => array('show' => true, 'href' => $back_url ? $back_url : 'javascript:history.back()'),
#3 20 декабря 2016 в 11:35


1. Лучше заменить на это:
2. Можно добавить как-то так

Loadырь
1. Принято
2. В принципе, даже лучше, чем "отменить" в меню действий (тут вернёмся на страницу записи или в админку (в 99% случаев), а из меню — на главную типа контента или в админку).
Кстати, только теперь заметил, что "отменить" в некоторых случаях отсутствует и в меню действий. Пример: дефолтная установка с демо-данными 2.6.1: при редактировании записи из типа контента "Страницы" (просмотр списка не разрешён) ссылка "отменить" не сформируется.
Спасибо!
#4 20 декабря 2016 в 11:41

В принципе, даже лучше

Олег Васильевич я
Ещё бы желательно стилями, визуально кнопку "Отменить" сделать второстепенной.
#5 20 декабря 2016 в 11:59


В принципе, даже лучше

Олег Васильевич я
Ещё бы желательно стилями, визуально кнопку "Отменить" сделать второстепенной.

Loadырь
так и сделано (отдельным правилом):


Если покажете, как в эту конструкцию:

  1. 'cancel' => array('show' => true, 'href' => $back_url ? $back_url : 'javascript:history.back()'),
класс для кнопки добавить, можно обойтись без отдельного правила (подобная кнопка с селектором "button-cancel" уже есть в системе)
Учитывая, что по шаблону запланировано ещё немного изменений, пожалуй, сделаю демку.
#6 20 декабря 2016 в 12:27

1. В формах обязательные к заполнению поля не отмечены никак

Олег Васильевич я
В идеале хотелось бы иметь проверку заполненности поля, например, если не заполнено, то красная обводка, если заполнено, то зеленая.
#7 20 декабря 2016 в 13:33

Если покажете, как в эту конструкцию:

Олег Васильевич я
В эту конструкцию, можно добавлять хоть-что, только работать будет название и ссылка у кнопки. Надо тут менять на это
  1. <?php if ($cancel['show']) { echo html_button($cancel['title'], 'cancel', "location.href='{$cancel['href']}'", array('class' => 'button-cancel')); } ?>
#8 20 декабря 2016 в 13:36

В идеале хотелось бы иметь

rdimrs
В идеале много чего тут можно реализовать, но не всем это нужно. Насколько я понял, тут рассматриваются уже имеющиеся проблемы в шаблоне, а не создаются новые laugh
#9 20 декабря 2016 в 15:59


В идеале хотелось бы иметь проверку заполненности поля, например, если не заполнено, то красная обводка, если заполнено, то зеленая.

rdimrs
Поля на заполненность система проверяет. Если обязательное поле не заполнено, система сообщит об этом (в том числе, выделит поле цветом).
В данном случае, мы пытаемся сразу обозначить поля, обязательные к заполнению. Стиль я сделал на свой вкус. Каждый сможет его изменить как посчитает нужным.


#10 20 декабря 2016 в 16:10

Надо тут менять

Loadырь
Такой код "потянет"?

  1. <?php if ($cancel['show']) { echo html_button($cancel['title'], 'cancel', "location.href='{$cancel['href']}'", array('class'=>'button-cancel')); } ?>
#11 20 декабря 2016 в 16:43

Такой код "потянет"?

Олег Васильевич я
Да, но стиль применится сразу ко всем формам, где есть кнопка отмены.
#12 20 декабря 2016 в 17:07


Да, но стиль применится сразу ко всем формам, где есть кнопка отмены.

Loadырь
Это очень радует! dance
Группе, к которой вы принадлежите, запрещено просматривать этот скрытый текст
#14 20 декабря 2016 в 18:00
Демка: inst.instantcms.com.ua/
Группе, к которой вы принадлежите, запрещено просматривать этот скрытый текст
#15 22 декабря 2016 в 22:13
Добавлен п.8.
Архив перезалит.
Вы не можете отвечать в этой теме.
Войдите или зарегистрируйтесь, чтобы писать на форуме.
Используя этот сайт, вы соглашаетесь с тем, что мы используем файлы cookie.