Skip to content
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

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

dimasites
Copy link
Contributor

Описание улучшений

@dimasites Протестировал. Идея хорошая, но реализация к сожалению не очень. Т.к. я не понял как при расширении контентной части ты аж задел блок оглавлений.

Всё вернул и в одну строчку расширил контентную часть страницы

Originally posted by @GulomovCreative in #723 (comment)

Если бы ты просто всё вернул и одной строчкой сохранил все правки, я бы не делал повторный запрос, но это не так. Твоё решение аффектит только ширину контейнера, но не привносит других правок, которые я выполнил и предлагал внести, поэтому проблемы отображения сохранились, например:

1920px

2023-06-24_16-39-00

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

Посмотри, пожалуйста, на скриншоты:

1366px

2023-06-24_16-44-37

~1200px

2023-06-24_17-09-11

Так мы получим эффективное использование экрана, когда места много, и влезающий контент, когда мало, при этом ничего не слипается и дизайн остаётся легким, но с контентом на первом месте в качестве главного элемента.

@GulomovCreative
Copy link
Member

Контейнер шире, и на широких экранах лучше используется пространство

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

@GulomovCreative
Copy link
Member

@dimasites Раздели пожалуйста PR, чтобы я мог протестировать конкретные правки, спасибо

@GulomovCreative
Copy link
Member

поэтому проблемы отображения сохранились, например

Что ты называешь проблемами?

  • Неширокая контентная часть.
  • Некомпактные отступы.

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

@dimasites
Copy link
Contributor Author

Отвечу пожалуй на всё сразу, чтобы не плодить тред:

По проблемам отображения:
Смотрите внимательно скриншоты:

  1. Среди прочего:
    image
  2. Также, на 1200 неоправданно большие отступы, занимающие суммарно ~треть контентной области (3й скриншот)

По rebase:
Специально не делал, т.к. это не новая ветка, а продолжение предыдущей. К тому же в ней хранится less-версия приведенный изменений, которая ещё пригодится.

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

@dimasites
Copy link
Contributor Author

Контейнер шире, и на широких экранах лучше используется пространство

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

Сравнение с книгой в данном контексте не уместно. У нас адаптивный дизайн.
Вот вам более релевантное сравнение:

Вот у github ширина контейнера 830px при меньшем шрифте (font-size одинаковый, но из-за более family он меньше). Неплохой референс же?

Предлагаемая мной в данном PR ширина контента - 925px (примерно как Github по читаемости, но с учетом корректировки под используемый у нас шрифт):
2023-06-24_18-19-59

Текущая ширина контента в документации для 1920px - 767px:
image

И да, это проблема, особенно с учетом, как развивалась документация, под какой размер контента авторы её наполняли, и проверяли читаемость . Я напомню, там был ~1400px контент на 1920 экране:
2023-06-24_18-29-47

Считаю приведенные аргументы показывают наглядно, что предлагаемое мной в данном PR решение является оптимальным. В старой документации 1400 было явно перебор, как раз ваш, @GulomovCreative пример с книгой - это про неё. Но вы же всё и исправили, за что вам низкий поклон! Но давайте ещё пару штрихов, чтобы было прям ещё выше всех похвал!

@GulomovCreative
Copy link
Member

Если отправишь PR с исправлением "бага" сетки, то я приму. С остальными изменениями я не согласен, извини.

@Ibochkarev Если будет желание и время, протестируй плиз и если тебя будет устраивать, то делай что хочешь. Повторюсь, "проблемы" надуманные, а изменить, чтобы изменить меня не устраивает

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants