Добавляет кастомный виджет со звёздочками GitHub #1316
Conversation
Inventoris
requested review from
HellSquirrel,
furtivite,
igsekor and
solarrust
as code owners
|
Мне нравится начинание, но я немного потерял смысловое содержание. Это личное мнение, возможно, на что-то натолкнёт. В оригинальном виджете есть значёк гитхаба, есть слово Stars. Понятно, что это какие-то звёзды на гитхабе. В PR я вижу значёк звезды, и слово звёзды. Как старый житель Доки, который знает, что Дока лежит на гитхабе, я пойму. Как новый — увы, будет сложнее. Я бы призвал @skorobaeus пошаманить над визуалом. |
|
Интересное начинание, но, кажется, способ запроса совсем не подходит нам. Прямые запросы на GitHub уже в прошлом. Так что эту часть нужно будет переделать. Доберусь в январе и с удовольствием помогу :) Но это техническая сторона. А вот дизайнерская сторона вопроса — тут @skorobaeus. И надо будет сделать проверку на тысячу. Мы должны проработать все сценарии, поддержу @vitya-ne |
UPD: заберу всё в работу, как доберусь o(￣︶￣)o |
|
Вернулся к этому пиару =) @vitya-ne добавил проверку на 1000; Пример статьи, смотреть футер: https://platform-1316.dev.doka.guide/css/box-model/ |
|
Превью контента из a378497 опубликовано. |
| @@ -0,0 +1,61 @@ | |||
| .github-widget { | |||
| --widget-padding: .4em .4em .25em; | |||
There was a problem hiding this comment.
Как по мне, лучше выранивать по-центру сейчас иконка и текст смещены вниз:
There was a problem hiding this comment.
Как по мне, лучше выранивать по-центру сейчас иконка и текст смещены вниз:
Дак вот я выравнил специально так) Потому что если сделать по центру, то визуально надпись оказывается не по центру. Заглавная буква и иконка выше цифры получаются.
Сейчас как будто ровно, разве нет?
Попробуй в девтулзе сделать одинаковые паддинги.
There was a problem hiding this comment.
Вот как у меня:
Разве по вертикали ровно ?
There was a problem hiding this comment.
Смотрю на ноуте 1920*1080 с 125% размером текста. Это по-моему распространённая настройка
There was a problem hiding this comment.
Ты знаешь, иконка действительно пляшет, если менять масштаб. У меня есть идея как это исправить. Но сперва я бы предложил узнать мнение Светы @skorobaeus, а вообще этот MVP-дизайн ей нравится? Стоит ли им заниматься дальше или лучше применить другой UI в целом.
|
Я ещё немного позанудствую на тему запроса: |
|
|
||
| @media (width >= 768px) { | ||
| .github-widget__star { | ||
| width: 15px; |
There was a problem hiding this comment.
А почему ширина 15px ?
Видно, что иконка не влезает (если запрос завершился неудачей)
There was a problem hiding this comment.
Про иконку если запрос упал интересная идея. Правда я видел у нас ещё сервайс воркер, мб через него можно придумать решение. То есть будет брать из кеша, если есть, а если и там нет, то тогда иконку с какой-нибудь надписью, вроде "Наш репозиторий".
Только какой тогда смысл давать ссылку на репозиторий для автономщиков, если они ее не смогут открыть?
There was a problem hiding this comment.
А почему ширина 15px ? Видно, что иконка не влезает (если запрос завершился неудачей)
Не понял если честно о чем ты, можешь раскрыть пж? На скрине кажется всё ок
There was a problem hiding this comment.
Обрати внимание на правый край круга: он скошен
Возможно это можно поймать только на ноуте с 125% текстом
There was a problem hiding this comment.
Обрати внимание на правый край круга: он скошен
Возможно это можно поймать только на ноуте с 125% текстом
Думаю это баг рендера браузера, или какая-то история с пиксельным отображением именно на стороне экрана. Потому что по идее там нету того, чем обрезаться, т.к. пустое пространство между иконкой и текстом. Такое бывает.
There was a problem hiding this comment.
Я ж начинал с того что в стилях ширина svg - 15px.
Ставлю 16px и всё ок:
Ну мы же отображаем другие ссылки. Может я выключил WiFi, а потом включил его не обновляя страницы. |
Может быть, давай узнаем мнение других. P.S. Если не сложно, можешь пж накидывать ревью не отдельными сообщениями, а в тредах. Я немножко начинаю путаться, где на что отвечать) |
Сорри, но вообще-то там тред про стили иконки. А это отдельная тема |
5 participants
Навеяно ишью #1308
Предлагаю добавить кастомный виджет с поддержкой смены цветовой темы (а ещё в дизайн-коде Доки и на русском).
Если про сам дизайн, то использовал скругления и цвета с переключателя цветовой темы. По общей структуре ориентировался на такой вариант:
Разумеется всё обсуждаемо, особенно дизайн @skorobaeus 🙌
Было:
Стало:
В тёмной теме: