Docs
Docs copied to clipboard
modx-pro
Улучшение адаптивной вёрстки #2
Описание улучшений
@dimasites Протестировал. Идея хорошая, но реализация к сожалению не очень. Т.к. я не понял как при расширении контентной части ты аж задел блок оглавлений.
Всё вернул и в одну строчку расширил контентную часть страницы
Originally posted by @GulomovCreative in https://github.com/modx-pro/Docs/issues/723#issuecomment-1605487644
Если бы ты просто всё вернул и одной строчкой сохранил все правки, я бы не делал повторный запрос, но это не так. Твоё решение аффектит только ширину контейнера, но не привносит других правок, которые я выполнил и предлагал внести, поэтому проблемы отображения сохранились, например:
1920px
Поэтому, я предлагаю совместить оба решения для получения максимальной пользы!
Посмотри, пожалуйста, на скриншоты:
1366px
~1200px
Так мы получим эффективное использование экрана, когда места много, и влезающий контент, когда мало, при этом ничего не слипается и дизайн остаётся легким, но с контентом на первом месте в качестве главного элемента.
Контейнер шире, и на широких экранах лучше используется пространство
Дмитрий, бесконечно широкое пространство не является преимуществом. Попробуйте почитать книгу со страницами шириной в метр
@dimasites Раздели пожалуйста PR, чтобы я мог протестировать конкретные правки, спасибо
поэтому проблемы отображения сохранились, например
Что ты называешь проблемами?
- Неширокая контентная часть.
- Некомпактные отступы.
Мне кажется, что ты просто генерируешь в нейросети проблемы.
Отвечу пожалуй на всё сразу, чтобы не плодить тред:
По проблемам отображения: Смотрите внимательно скриншоты:
- Среди прочего:
- Также, на 1200 неоправданно большие отступы, занимающие суммарно ~треть контентной области (3й скриншот)
По rebase: Специально не делал, т.к. это не новая ветка, а продолжение предыдущей. К тому же в ней хранится less-версия приведенный изменений, которая ещё пригодится.
По разделению PR: Правки комплексные и их не много, предлагаю тестировать как есть. Излишняя декомпозиция только усложнит всем работу.
Контейнер шире, и на широких экранах лучше используется пространство
Дмитрий, бесконечно широкое пространство не является преимуществом. Попробуйте почитать книгу со страницами шириной в метр
Сравнение с книгой в данном контексте не уместно. У нас адаптивный дизайн. Вот вам более релевантное сравнение:
Вот у github ширина контейнера 830px при меньшем шрифте (font-size одинаковый, но из-за более family он меньше). Неплохой референс же?
Предлагаемая мной в данном PR ширина контента - 925px (примерно как Github по читаемости, но с учетом корректировки под используемый у нас шрифт):
Текущая ширина контента в документации для 1920px - 767px:
И да, это проблема, особенно с учетом, как развивалась документация, под какой размер контента авторы её наполняли, и проверяли читаемость . Я напомню, там был ~1400px контент на 1920 экране:
Считаю приведенные аргументы показывают наглядно, что предлагаемое мной в данном PR решение является оптимальным. В старой документации 1400 было явно перебор, как раз ваш, @GulomovCreative пример с книгой - это про неё. Но вы же всё и исправили, за что вам низкий поклон! Но давайте ещё пару штрихов, чтобы было прям ещё выше всех похвал!
Если отправишь PR с исправлением "бага" сетки, то я приму. С остальными изменениями я не согласен, извини.
@Ibochkarev Если будет желание и время, протестируй плиз и если тебя будет устраивать, то делай что хочешь. Повторюсь, "проблемы" надуманные, а изменить, чтобы изменить меня не устраивает