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

add component for language markup #1623

Merged
merged 4 commits into from
May 23, 2024
Merged

add component for language markup #1623

merged 4 commits into from
May 23, 2024

Conversation

DmitriySmolin
Copy link
Contributor

@DmitriySmolin DmitriySmolin commented May 20, 2024

Pull request details

Issues fixed

#1392

Link to demo

https://hexlet-sicp-wums.onrender.com/ru

@fey
Copy link
Collaborator

fey commented May 20, 2024

@DmitriySmolin задеплойте, пожалуйста, демку на Render

@DmitriySmolin
Copy link
Contributor Author

@fey демка задеплоена на render
https://hexlet-sicp-wums.onrender.com/ru

@fey
Copy link
Collaborator

fey commented May 20, 2024

  1. каким редактором пользуетесь? У вас в ПРе исправления отступов в шаблонах - много шумных изменений. Можете их откатить? В проекте есть команда для преттира, которая фиксит отступы (проверьте по ней отступы)
  1. Можете словами описать что сейчас делает миддлвара?
  2. Кажется генерировать HTML самой мидлваре и менять html - это не тот способ, который нам нужен.

Можем начать с конца. Наша задача в чем? Сформировать HTML, он формируется в шаблонах. Можно сделать компонент шаблона, в котором будет нужная логика и выводить этот компонент. Сам компонент может принимать текущую локаль (или брать из приложения).

@DmitriySmolin
Copy link
Contributor Author

DmitriySmolin commented May 20, 2024

  1. PHP Storm использую. Очень странно что с отступами так произошло, я долго не мог запушить свои коммиты, несколько раз подряд прогонял через make lint-fix
  2. миддлвара фомирует и рендерит в зависимости от выбранной локали языковые теги HTML. Локаль берется из приложения.
    В соответстви с этим:

Для https://sicp.hexlet.io/ru. Набор тегов:

link rel="alternate" hreflang="en" href="https://sicp.hexlet.io/"
link rel="alternate" hreflang="x-default" href="https://sicp.hexlet.io/"

Для https://sicp.hexlet.io/chapters. Набор тегов:

link rel="alternate" hreflang="ru" href="https://sicp.hexlet.io/ru/chapters"
link rel="alternate" hreflang="x-default" href="https://sicp.hexlet.io/chapters"

  1. Понятно, я предложил свое видение решения через middleware. Почитаю про компоненты

@DmitriySmolin DmitriySmolin changed the title add middleware for language markup add component for language markup May 22, 2024
@fey fey merged commit ce527f6 into Hexlet:main May 23, 2024
2 checks passed
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.

2 participants