reactjs / ru.react.dev

React documentation website in Russian / Официальная русская версия сайта React
https://ru.react.dev
Creative Commons Attribution 4.0 International
678 stars 380 forks source link

Fixing header, search bar no more overlaps link #231

Closed alkhasov closed 5 years ago

alkhasov commented 5 years ago

Current:

image

With PR:

image

En original:

image
netlify[bot] commented 5 years ago

Deploy preview for ru-reactjs ready!

Built with commit 645843e0b4eba735cebf3485745151932f60fb35

https://deploy-preview-231--ru-reactjs.netlify.com

lex111 commented 5 years ago

Спасибо, про "лишний" пункт не знал.

Это, конечно, хорошо, но пока ждём решение на самом оф. сайте, поскольку это появилось после появления пункта с языками (вроде как здесь решается https://github.com/reactjs/reactjs.org/pull/1751)

Хотя в качестве временного решения можем принять? сс @another-guy я не уверен на этот счёт.

alkhasov commented 5 years ago

Оу, а я не знал про тот PR.

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

lex111 commented 5 years ago

Неизвестно, что там дальше будет, обычно это не быстро, к сожалению. Поэтому склоняюсь к тому, чтобы принять данное решение, в принципе почему бы и нет.

alkhasov commented 5 years ago

Тогда можно локально подумать и над вариантом вроде этого https://github.com/reactjs/reactjs.org/pull/1751#issuecomment-469481374

lex111 commented 5 years ago

@musewick классно выглядит, но давайте ограничимся самым минимумом, либо уже решать в том же PR.

lex111 commented 5 years ago

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

alkhasov commented 5 years ago

@musewick классно выглядит, но давайте ограничимся самым минимумом, либо уже решать в том же PR.

Хорошо.

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

У вебпака отличный вариант, нравится что можно сразу в детали перейти.

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

lex111 commented 5 years ago

@musewick это здорово, но не факт, что его примут, но попробовать, конечно, стоит :+1:

alkhasov commented 5 years ago

Ну я надеюсь, что напишут фидбек хотя бы, отстой, не отстой, все остальное это дополнительно :D

another-guy commented 5 years ago

@lex111 @musewick прошу прощения, у меня нет сложившегося мнения.

В плане UX/UI, однозначно, решение мне нравится. А вот применять ли его прямо сейчас в ru.react.org или идти через родительский репозиторий, я не советчик. Это классический эффект второй системы. Если мы пойдём этим путём, то придётся поддерживать "форк". Лично у меня не хватит компетенции в CSS, чтобы решать проблемы. 🤷‍♂️

Полностью доверяю вашему решению. Может @gcor, @Heegiiny, @ntishkevich могут что-то более осмысленное подсказать?

lex111 commented 5 years ago

@musewick спасибо!