-
-
Notifications
You must be signed in to change notification settings - Fork 130
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Улучшение адаптивной вёрстки #2 #726
base: master
Are you sure you want to change the base?
Conversation
Уменьшены отступы для нешироких desktop-ов, увеличен контейнер с контентом для широких
…/Docs into feature/adaptive-styles
Дмитрий, бесконечно широкое пространство не является преимуществом. Попробуйте почитать книгу со страницами шириной в метр |
@dimasites Раздели пожалуйста PR, чтобы я мог протестировать конкретные правки, спасибо |
Что ты называешь проблемами?
Мне кажется, что ты просто генерируешь в нейросети проблемы. |
Сравнение с книгой в данном контексте не уместно. У нас адаптивный дизайн. Вот у github ширина контейнера 830px при меньшем шрифте (font-size одинаковый, но из-за более family он меньше). Неплохой референс же? Предлагаемая мной в данном PR ширина контента - 925px (примерно как Github по читаемости, но с учетом корректировки под используемый у нас шрифт): Текущая ширина контента в документации для 1920px - 767px: И да, это проблема, особенно с учетом, как развивалась документация, под какой размер контента авторы её наполняли, и проверяли читаемость . Я напомню, там был ~1400px контент на 1920 экране: Считаю приведенные аргументы показывают наглядно, что предлагаемое мной в данном PR решение является оптимальным. В старой документации 1400 было явно перебор, как раз ваш, @GulomovCreative пример с книгой - это про неё. Но вы же всё и исправили, за что вам низкий поклон! Но давайте ещё пару штрихов, чтобы было прям ещё выше всех похвал! |
Если отправишь PR с исправлением "бага" сетки, то я приму. С остальными изменениями я не согласен, извини. @Ibochkarev Если будет желание и время, протестируй плиз и если тебя будет устраивать, то делай что хочешь. Повторюсь, "проблемы" надуманные, а изменить, чтобы изменить меня не устраивает |
Описание улучшений
@dimasites Протестировал. Идея хорошая, но реализация к сожалению не очень. Т.к. я не понял как при расширении контентной части ты аж задел блок оглавлений.
Всё вернул и в одну строчку расширил контентную часть страницы
Originally posted by @GulomovCreative in #723 (comment)
Если бы ты просто всё вернул и одной строчкой сохранил все правки, я бы не делал повторный запрос, но это не так. Твоё решение аффектит только ширину контейнера, но не привносит других правок, которые я выполнил и предлагал внести, поэтому проблемы отображения сохранились, например:
1920px
Поэтому, я предлагаю совместить оба решения для получения максимальной пользы!
Посмотри, пожалуйста, на скриншоты:
1366px
~1200px
Так мы получим эффективное использование экрана, когда места много, и влезающий контент, когда мало, при этом ничего не слипается и дизайн остаётся легким, но с контентом на первом месте в качестве главного элемента.