search

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

Fix scrolling to anchor with fixed menu #283

Closed EPolovniy closed 5 years ago

netlify[bot] commented 5 years ago

Deploy preview for ru-reactjs ready!

Built with commit 3ce4cca190f1970f2bf789fd907d4f2134caf24c

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

another-guy commented 5 years ago

@EPolovniy я так понимаю, вы решаете проблему, не специфичную для ru.reactjs.org, а общую для reactjs.org. Пожалуйста, откройте PR в https://github.com/reactjs/reactjs.org/

EPolovniy commented 5 years ago

Суть бага - при переходе по ссылке, котороя встраивается в заголовки, прокрутка страницы происходить не корректно и сам заголовок скрывается за меню. Этот баг есть не только в русской версии. Pull request больше для review и совета как подготовить request в основной проект.

another-guy commented 5 years ago

@EPolovniy

Pull request больше для review и совета как подготовить request в основной проект.

Наш репозиторий должен фокусироваться на русском переводе и тех багах сайта, которые проявляются только в русской версии. В родительском репозитории вы получите более толковое ревью и объяснение, что надо подготовить. У ru.reactjs.org не хватает ресурсов для непереводной работы, извините. Кроме того, это и не то репо, куда надо применять таблетку от проблемы.

Ваше изменение кода полезно. Поэтому не бросайте дело, а откройте PR, там, где ему и место, пожалуйста.

EPolovniy commented 5 years ago

Спасибо, сейчас подготовлю request в основной репозиторий.

another-guy commented 5 years ago

Вам спасибо! Дайте знать, пожалуйста, как откроете реквест там.

EPolovniy commented 5 years ago

@another-guy Создал: https://github.com/reactjs/reactjs.org/pull/1855