Skip to content

Подключение нового хедера - #16

Merged
Feretj merged 1 commit into
mainfrom
change-pishu-tebe-to-dh
Aug 11, 2026
Merged

Подключение нового хедера#16
Feretj merged 1 commit into
mainfrom
change-pishu-tebe-to-dh

Conversation

@Tabarzin

@Tabarzin Tabarzin commented Aug 6, 2026

Copy link
Copy Markdown

Обновленное / актуальное верхнее меню.

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

Feretj commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

Претензий к коду у меня тут нет, но глянул локально, и десктопное и мобильное меню выглядят не как на проде.

На счет размера шрифта, rem это относительная величина от размера шрифта тега html. Стандартный размер 16px (как сейчас в Next), на проде 10px или 9.5px в зависимоти от размера экрана. Раз будет редизайн, можно не заморачиваться и везде ставить размер в пикселях (в devtools если навести мышку на значение rem, он покажет его в пикселях).

Иконки на проде сделаны через шрифт. Вк и Телеграм меняют цвет при наведении мышки. В целом и через svg можно, тогда цвет можно поменять с помощью css значения fill.

На ширине <1230px у тебя лучше выглядит хедер без переноса, это давай оставим)

Screenshot_20260811_162857 Screenshot_20260811_162936 Screenshot_20260811_163042 Screenshot_20260811_164027

@Feretj
Feretj merged commit 5a36915 into main Aug 11, 2026
1 check 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