Skip to content

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