Skip to content

Подключение футера - #17

Open
Tabarzin wants to merge 2 commits into
mainfrom
add-connect-to-new-footer
Open

Подключение футера#17
Tabarzin wants to merge 2 commits into
mainfrom
add-connect-to-new-footer

Conversation

@Tabarzin

@Tabarzin Tabarzin commented Aug 6, 2026

Copy link
Copy Markdown

Получает футер. Есть несколько моментов:

  • Добавил social-icons.tsx Имеет смысл переделать это и для хедера, чтобы не было двух разных реализаций для соц. иконок.
  • При наведении курсора подчеркивается весь блок, а не одно слово, пока не понял, как это исправить.
  • На главной есть фраза Журнал о цифровых технологиях в культуре, искусстве и образовании, которая не приходит, а захардкожена. На других страницах в футере ее нет.

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

@Tabarzin Tabarzin self-assigned this Aug 6, 2026
@Tabarzin
Tabarzin requested a review from Feretj August 6, 2026 15:02
@Feretj

Feretj commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator
  • На главной есть фраза Журнал о цифровых технологиях в культуре, искусстве и образовании, которая не приходит, а захардкожена. На других страницах в футере ее нет.

Это для SEO тег H1 сделан. Можешь использовать usePathname чтоб проверить главная ли usePathname() === '/'

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

Ага, с Вячеславом по SEO уже обсуждаем это, он предлагает удалить ссылки на теги и категории

Comment thread app/layout.tsx
)
}
;<Footer />
// const Footer = () => {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Удалить коммент

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Иконки как на проде можешь найти на fontello, там можно пройти и на первоисточник

Comment thread components/Footer.tsx

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Названия файлов у нас с маленькой буквы

Comment thread components/Footer.tsx
youtube: YoutubeIcon,
}

const SOCIAL_LINKS = [

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это можно в отдельный фаил, как раз menu.config.ts подойдет, потом переиспользовать в хедере

Comment thread components/Footer.tsx
import Link from 'next/link'
import type { ComponentType, SVGProps } from 'react'

const socialIcons: Record<string, ComponentType<SVGProps<SVGSVGElement>>> = {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это переместить в файл самих иконок

Comment thread components/Footer.tsx
'[&>div+div]:mt-6 ' +
headingClasses +
' ' +
'[&_a]:text-muted-foreground [&_a]:hover:text-foreground [&_a]:hover:underline [&_a]:underline-offset-4 ' +

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Нашел фикс тут для ховера. Надо заменить [&_a]:hover на [&_a:hover]. Но честно говоря дабавить стили в globals.css кажется лучше и ограничиться tailwind в частях которые не приходят с админки.

Comment thread components/Footer.tsx
return (
<footer>
<Section>
<Container>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ширина, фон и размеры шрифтов не совпадают, кажется легче новый контейнер сделать
Image

Comment thread components/Footer.tsx
</div>
</Container>

<Container className="border-t not-prose flex flex-col md:flex-row md:gap-2 gap-6 justify-between md:items-center">

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тоже сильно отличается
Image

Comment thread components/Footer.tsx

<Container className="border-t not-prose flex flex-col md:flex-row md:gap-2 gap-6 justify-between md:items-center">
<p className="text-muted-foreground">
&copy; 2018–2026 <a href="https://sysblok.ru">Системный Блокъ</a>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ссылка тут не нужна

@Feretj
Feretj force-pushed the add-connect-to-new-footer branch from 4a0199e to 5789e2d Compare August 11, 2026 17:13
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