hexlet-correction icon indicating copy to clipboard operation
hexlet-correction copied to clipboard

[#153] add footer (internationalized)

Open RedGradient opened this issue 1 year ago • 15 comments

Демонстрация: https://hexlet-corrector-fork.onrender.com/

RedGradient avatar Jul 01 '23 14:07 RedGradient

@Malcom1986 посмотри, пжлста

ola-9 avatar Jul 03 '23 07:07 ola-9

@RedGradient Привет! Здорово, такой подход будет работать, но можно пойти еще дальше. Давай попробуем сделать наследование шаблонов. То есть у нас будет общий костяк страницы - layout, который будет содержать общую для всех страниц разметку и такие фрагменты типа плейсхолдера. А шаблоны конкретных страниц будут дополнять layout (как бы наследоваться от него) и будут содержать только уникальную для них разметку. Когда мы рендерим конкретную страницу, будет браться контент конкретной страницы и вставляться на место плейсхолдера в layout. И получится такая комбинированная разметка. В чем плюс. Если нам нужно будет добавить еще како-то общий элемент - хедер, панель навигации и так далее, нам понадобится изменить только layout. Все остальные страницы останутся неизменными. При нынешнем подходе придется опять вставлять их при помощи th:replace на каждую страницу

Malcom1986 avatar Jul 03 '23 13:07 Malcom1986

Посмотри как у нас вот тут сделано https://github.com/hexlet-components/java-javalin-blog/blob/main/app/src/main/resources/templates/layouts/application.html

Malcom1986 avatar Jul 03 '23 13:07 Malcom1986

Добрый день. А что за пустое пространство между хедером и верхним краем страницы? image

fey avatar Jul 04 '23 19:07 fey

Еще обратите внимание, что работа ведется также в #155 Предлагаю вам скооперироваться, чтобы не делать одно и то же).

fey avatar Jul 04 '23 19:07 fey

Привет, кто работает над Пром можешь мне в телеграмм написать хочу распросить по поводу запуска этого шаблона, мой телеграмм @Bakkcha

amirhraj avatar Jul 15 '23 02:07 amirhraj

@RedGradient Привет! Есть успехи по этому ПРу?

Malcom1986 avatar Jul 17 '23 12:07 Malcom1986

@Malcom1986 Привет 👋 Я подумал, что лучше оставлю эту задачу amirhraj-у, поскольку после некоторой паузы он снова подключился к работе.

RedGradient avatar Jul 17 '23 13:07 RedGradient

Возвращаюсь к работе над ПР. Последний отправленный коммит - промежуточный результат.

RedGradient avatar Jul 19 '23 20:07 RedGradient

@fey, добрый день! Уточните, пожалуйста, какое поведение футера предпочтительнее:

  1. По-умолчанию футер находится внизу экрана. Если контента много, футер опускается ниже за пределы экрана, следуя за контентом.
  2. Футер прикреплен к низу экрана и виден всегда. В случае, если контента много, он его будет перекрывать.

RedGradient avatar Jul 20 '23 10:07 RedGradient

Скорее первый вариант, чем второй, футер не должен перекрывать контент. Если контента мало, то на странице должен быть стиль что-то типа min height, когда футер все равно будет внизу.

Можете посмотреть, как ведет себя футер на хекслете,хекслет-цв, код бейзикс и так далее.

fey avatar Jul 20 '23 14:07 fey

т.е. футер должен быть частью страницу, а не оставаться в поле видимости при скроллинге

fey avatar Jul 20 '23 14:07 fey

прежде чем приступить к ревью, я предлагаю пересобрать ветку, чтобы в истории не было лишнших коммитов. Сейчас там коммиты идут из основной ветки. Можете создать новую ветку от текущей и в ней потренироваться.

Тут можно почитать, как это можно сделать - https://ru.hexlet.io/qna/git/questions/kak-aktualizirovat-vetku-v-pull-request

fey avatar Jul 21 '23 07:07 fey

волонтеры или фронтенд https://t.me/hexletcommunity/12 https://t.me/hexletcommunity/19

fey avatar Jul 21 '23 20:07 fey

Взял паузу в работе.

RedGradient avatar Jul 25 '23 18:07 RedGradient