-
Notifications
You must be signed in to change notification settings - Fork 65
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
Ченджлог на главной: первое приближение #1286
base: main
Are you sure you want to change the base?
Conversation
Пока кажется, что на мобилках трудно тапнуть по нужной ссылке из-за того, что ссылка на статью сильно прижалась к ссылке на автора. Ещё может быть есть смысл слегка увеличить нижний паддинг у даты на мобилках? |
В макете ссылочки тоже стоят друг над другом https://share.cleanshot.com/PvrPmrz7 |
Да-да, я понимаю. Просто обратила внимание сейчас и подумала, что может вызвать ошибку касания цели такое близкое расположение. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Наверное, ещё надо удалить ссылку в баннере выше про бэклог?
Блин, как круто получилось. Нашла только пару мееееелких штук, которые не супер важные, и, если есть время, можно поправить в этом пиаре. Если что, могу донести потом отдельным. @skorobaeus, продолжаю душнить про слегка увеличенный отступ между ссылкой на материал и страницу автора на мобилках. Реально сложно прицелиться куда надо. Как-то так это может быть: Шрифт у строчек с кодом сейчас дефолтный, надо будет накинуть на
Возможно, точечки уменьшать можно чуть пораньше, чтобы автор не уезжал под них. Дело в длине названия материала, так что не уверена, как лучше поступить, @skorobaeus . Пример: Может заменим внутри И, да, кажется, что можно удалить потом отдельным пиаром ссылку на горящий бэклог в карточке, чтобы два раза не повторять одну ссылку на странице. |
Всем привет! Невероятно крутая фича, давно её ждал 🤩 Не могу пройти мимо и предлагаю несколько улучшений, как сторонний наблюдатель.
Как выше отметил Игорь, ссылки на бэклог действительно задублировались, и лучше оставить одну из них. На мой взгляд, та, что в баннере, лучше. В нём собранны три главные ссылки Доки, а ченджлог — про ченджлог. |
@@ -0,0 +1,23 @@ | |||
<div class="changelog"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<div class="changelog"> | |
<section class="changelog"> |
У нас есть заголовок, и эта часть выглядит, как отдельная смысловая. Может выделим её в section
?
@@ -0,0 +1,23 @@ | |||
<div class="changelog"> | |||
<h2 class="visually-hidden">Новые статьи</h2> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
{% endfor %} | ||
</ul> | ||
<div class="changelog-footer"> | ||
<p class="changelog-footer__item">Новые материалы в <a href="https://github.com/doka-guide/content/blob/main/CHANGELOG.md" class="link changelog-footer__link">ченджлоге</a></p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<div class="changelog-item__author"> | ||
{% for author in item.authors %} | ||
{% if loop.index0 !== 0 %}, {% endif %}<a href="{{ author.url }}" class="changelog-item__author-link font-theme--code link">{{ author.name }}</a> | ||
{% endfor %} | ||
</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Возможно, добавим условие, что если авторов несколько, то помещаем их в список ul
?
@Inventoris, спасибо за фидбек! Я тоже думала над заголовком для секции ченджлога, но он пока не прижился в макете, потому что на главной вовсе нет заголовков в привычном понимании слова. Но подумаю ещё :) Если не трудно, скажи, пожалуйста, есть ли такой же эффект, что не сразу понятно, что происходит, когда скроллишь через фичеринг (большие цветные блоки) и рубрикатор (пять колонок с разделами)? Ссылка на горящий бэклог действительно должна быть одна, и я переместила её на новое место намеренно, потому что метрика показывает, что на ссылки в блоке "Друг для друга" вообще никто не кликает 😅 Хотим проверить гипотезу, будет ли эта ссылка работать лучше, завёрнутая в осмысленный призыв словами. |
@skorobaeus, у меня было ощущение перегруза, когда первый раз увидел секцию ченджлога. Масса точечек, даты итд. Это не плохо, она прекрасная, просто я сразу начал искать заголовок для понимания что это, где границы этой части, как её понять. Поэтому как будто хочется добавить заг, чтобы пользователь смог сразу понять, что его ждёт ниже. Фичеринг. Понятно не сразу, но не критично. Тем более у карточек есть описания, из которых сразу понимаешь, что это именно карточки статей. Рубрикатор. Эффекта, что я описал выше, точно нет. Всё сразу понятно, там также есть видимые заголовки |
Превью контента из c49ea7e опубликовано. |
Привет, спасибо за новую фичу!
|
Нужно! Спасибо, что нашёл время пофидбечить!
Чтобы ответить на любой вопрос, нужно отталкиваться от того, какую задачу мы решаем :) Задача ченджлога в целом — показывать, что проект активен и обновляется, а в частности — показывать, что именно в нём нового. Поэтому даты играют достаточно заметную роль. Дальше, например, идёт блок фичеринга, единственная задача которого — выдавать максимум спотлайта статьям и иллюстрациям. Если мы будем подсвечивать статьи и там, и там, становится непонятно, зачем тогда нужны два блока? Может быть — МОЖЕТ БЫТЬ — и правда нужно объединить сущности и сделать один блок, который будет и фичерингом, и ченджлогом одновременно? 🤔 Тем более что @solarrust не единожды предлагала фичеринг вообще убрать.
На заголовок есть запрос, зафиксировала ✍
Да, у меня были разные способы показать принадлежность к разделу в первых макетах. В какой-то момент отказалась от этого, чтобы блок не выглядел слишком аляповато. Немного "кухни" :)
На самом деле точек быть не должно, заголовок может выводиться полностью. Не доверстали маленько.
Добро пожаловать в дизайнерскую ловушку! Если выделить всё, ничто не будет выделенным ☝ Приоритет на главной у философии проекта и фичеринга, не предполагается, что ченджлог изменит это.
Интересная мысль. И звучит как +1 к идее объединить блоки.
Мой косяк 😄 |
Всё таки добавлю сомнений в идею показывать даты первой колонкой: что будет если несколько материалов будет смержено в один день (ну да, звучит неправдоподобно учитывая текущие темпы :(, но всё же ) ? |
Выведется одна и та же дата, а что в этом такого? И как порядок колонок влияет на это? В любом случае, это не та битва, где нужно биться насмерть, не нужно выискивать всё новые аргументы, чтобы меня убедить :) Достаточно уже того, что даты смущают, значит, я буду думать :) |
Первый подход к вёрстке блока с ченджлогом на главной
Десктоп
![CleanShot 2024-08-08 at 10 40 47@2x](https://private-user-images.githubusercontent.com/13676514/356168615-9c25df74-91c7-4f00-ae29-0bcde4a5473a.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjYxMDQ2OTUsIm5iZiI6MTcyNjEwNDM5NSwicGF0aCI6Ii8xMzY3NjUxNC8zNTYxNjg2MTUtOWMyNWRmNzQtOTFjNy00ZjAwLWFlMjktMGJjZGU0YTU0NzNhLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA5MTIlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwOTEyVDAxMjYzNVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTNkYTk1YzJlZWQ4MjI2MzlmMjEwOTIxODk4MzU3ZWE2MTlkNmZlYzM4YzUyMGVmODI5ZDVjODI1MDg4YjQ4NDkmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.Ek9AoS2tBnCsKYS-yU-NlUikYAuxvJA4iRTcjtMgCSE)
Планшет
![CleanShot 2024-08-08 at 10 43 15@2x](https://private-user-images.githubusercontent.com/13676514/356168800-f78072dc-5db8-4378-aca3-b96c186889c9.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjYxMDQ2OTUsIm5iZiI6MTcyNjEwNDM5NSwicGF0aCI6Ii8xMzY3NjUxNC8zNTYxNjg4MDAtZjc4MDcyZGMtNWRiOC00Mzc4LWFjYTMtYjk2YzE4Njg4OWM5LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA5MTIlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwOTEyVDAxMjYzNVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTZhMDBhMmNlNmMwMjM2YmI4N2Y4NjAxODZiNDNiZDM4ZDliYWI5NGJlMGU2M2Q4MmY4MzA5YWJhZDcyYzAwNDYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.0nKrYDuPK2Tey9MDQJVru1oZXYEO7HrYrT3p_Xgvmzk)
Мобилка
![CleanShot 2024-08-08 at 10 43 35@2x](https://private-user-images.githubusercontent.com/13676514/356168928-66d435a3-df4e-4852-92a3-37b889abb98e.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjYxMDQ2OTUsIm5iZiI6MTcyNjEwNDM5NSwicGF0aCI6Ii8xMzY3NjUxNC8zNTYxNjg5MjgtNjZkNDM1YTMtZGY0ZS00ODUyLTkyYTMtMzdiODg5YWJiOThlLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA5MTIlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwOTEyVDAxMjYzNVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWJhODk1MjQ5YzIwOTdkYmQ4MmJhODA3MDkwMzM0YjQxMjNiNGYzMWUyZDVhOGExMDdkNzlhZTE3OTNjNDMzN2YmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.z9LGS7VPNLNk1qAgK9FBHSQwvMFqciLUrZMaTNctj0I)
На мобилке пока расположила всё в столбик. Если у @igsekor получится отображать месяц не словом, а цифрой, то переверстаю.
@skorobaeus нужен твой глаз-алмаз.
@igsekor нужно прикрутить логику, чтобы инфа подтягивалась из ченджлога.