Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#153] Add footer to website templates #310

Merged
merged 18 commits into from
Oct 11, 2024

Conversation

shamshaev
Copy link
Contributor

I added footer to all displayable templates.

  • footer itself is updated to the same style as new main page
  • links to open projects are updated (not actual link is replaced)
  • 2 languages are supported

I checked previous PRs on this issue and aware about putting footer just on one layout page. But I didn't found that layout page which works for all templates. I found that the head and the navbar are also added on each template in the same way as I did with footer.

Example screenshot:
Screenshot from 2024-09-30 23-01-06

@fey fey requested a review from Malcom1986 October 1, 2024 10:41
@fey
Copy link
Collaborator

fey commented Oct 1, 2024

lgtm (can you deploy demo)

I found that the head and the navbar are also added on each template in the same way as I did with footer.

I think we should use main layout and extends it 🫠

@shamshaev
Copy link
Contributor Author

lgtm (can you deploy demo)

I found that the head and the navbar are also added on each template in the same way as I did with footer.

I think we should use main layout and extends it 🫠

This means:

  1. Create main layout (now it's not existing)
  2. All templates extend to this main layout

This is my understanding. Is it correct?

@shamshaev
Copy link
Contributor Author

I checked documentation. No problem with extending one layout. But all templates (except from folders "fragments" and "widget") should be reworked (add extending, remove head and navbar on each template, etc.)

@fey
Copy link
Collaborator

fey commented Oct 1, 2024

@Malcom1986 can you review? =)

@fey
Copy link
Collaborator

fey commented Oct 1, 2024

This means:

  1. Create main layout (now it's not existing)
  2. All templates extend to this main layout

This is my understanding. Is it correct?

yes, it can be done in another PR (refactoring templates)

@shamshaev
Copy link
Contributor Author

here is the demo: https://hexlet-correction-4.onrender.com/

@Malcom1986
Copy link
Collaborator

@shamshaev Андрей, приветсвую. А может от обратного пойдем - один общий макет будем использовать и конкретные страницы уже будут дополнять этот макет? Так и нагляднее будет, видно структуру структуру страницы целиком. И не придется в каждый шаблон включать общие части. Со временем их может больше появится. Посмотрите как тут сделно. Шаблонизатор правда другой, но суть не меняется

@shamshaev
Copy link
Contributor Author

@Malcom1986 Максим, добрый день. Предлагаемое решение мне понятно, тут нет проблем. Единственный нюанс - по уму надо будет все переделать все шаблоны, потому что сейчас Head и Navbar тоже добавлены на каждый шаблон. А их нужно будет включить в общий layout, как и Footer, и затем просто от этого общего layout наследоваться.

Хорошо, тогда я переделаю сегодня-завтра.

@shamshaev
Copy link
Contributor Author

@Malcom1986 Максим, все переделал. Добавил в проект Thymeleaf Layout Dialect, сделал основной шаблон layout.html, затем использовал его во всех отображаемых шаблонах. Плюс еще добавил немного порядка и стиля - поставил актуальное название проекта в navbar и header, поправил стиль и добавил английский в страничку про проект, добавил хекслетовский favicon (использовал решение для Spring Boot).

Ссылка на демо: https://hexlet-correction-4.onrender.com/

@Malcom1986
Copy link
Collaborator

Супер, большую работу проделали, спасибо. Посмотрю ПР. Зато сейчас один раз порядок в шаблонах наведем, потом проще будет добавлять общий функционал

@Malcom1986 Malcom1986 merged commit 9ed1055 into Hexlet:main Oct 11, 2024
1 check passed
@shamshaev shamshaev deleted the fix-issue-#153-add-footer branch October 11, 2024 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants