UI bug with text on phones

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

UI bug with text on phones #253

Closed gcor closed 5 years ago

gcor commented 5 years ago

avevlad commented 5 years ago

Текстом поправить наверно не получится, только стилями:

картинка

![image](https://user-images.githubusercontent.com/2889662/54295560-55637a00-45c4-11e9-8957-b7611622d883.png)

Могу заняться этим.

another-guy commented 5 years ago

@gcor эта проблема специфична для 🇷🇺 версии сайта? Если нет, то лучше решать её на уровне reactjs.org. 🤔

Я не силён в CSS, так что точно не смогу помочь. И да, я согласен, что надо проблему решать стилями надо.

AlLevykin commented 5 years ago

src\pages\index.js line 316 50% заменить на 100%

avevlad commented 5 years ago

@AlLevykin отличное решение!

Работает и на других языках тоже:

картинка

![image](https://user-images.githubusercontent.com/2889662/54374967-3aab0700-4691-11e9-8497-a3d3b3cc0cdf.png)

Отправишь PR в https://github.com/reactjs/reactjs.org ?

another-guy commented 5 years ago

@avevlad есть время и желание открыть PR в родительский проект?

avevlad commented 5 years ago

@another-guy да, на днях займусь. Но issue оказалась чуть сложнее чем я думал, по поим тестам решение:

src\pages\index.js -> line 316 50% -> заменить на 100%

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

another-guy commented 5 years ago

@avevlad понял. В любом случае, у нас тут этот issue не должен "висеть", имхо

lex111 commented 5 years ago

Исправлено в #306