Добавляет кастомный виджет со звёздочками GitHub #1316
Conversation
Inventoris
requested review from
furtivite,
solarrust,
igsekor and
HellSquirrel
as code owners
|
Превью контента из опубликовано. |
| const response = await fetch('https://api.github.com/repos/doka-guide/content') | ||
| const { stargazers_count: stargazersCount } = await response.json() | ||
|
|
||
| stargazers.textContent = `${(stargazersCount / 1000).toFixed(1).replace(/\.0$/, '')}к` | ||
| } catch (error) { | ||
| console.error('Ошибка формирования количества звёзд репозитория контента', error) | ||
| } |
There was a problem hiding this comment.
Привет,
смотрится как отличная замена текущему виджету!
У меня тут пару сомненений:
- вычисление кол-ва звёзд кажется не учитывает вариант что звёзд будет (надеюсь не будет) меньше 1000. Может стоит добавлять
kтолько после проверки числа ? - текст ошибки не подходит под случай если запрос упал
There was a problem hiding this comment.
Привет, супер)
- Не думаю, что стоит добавлять такую проверку. Я рассматривал этот вариант, но мне показалось, что он нереалистичный. Около трёхсот человек придёт и заберет звёздочку? Если только на главной Доки будет какой-то острый политический контент, и пройдёт новость об этом, после чего люди пойдут жёстко минусить. Разве что так.
- Почему? Запрос упал, звёзды не сформировались.
There was a problem hiding this comment.
Запрос упал, но в консоли сообщение: "Ошибка формирования количества звёзд репозитория контента".
Скорее уж в этом случае это ошибка получения данных о репозитории
Впрочем, может это не важно )
|
Мне нравится начинание, но я немного потерял смысловое содержание. Это личное мнение, возможно, на что-то натолкнёт. В оригинальном виджете есть значёк гитхаба, есть слово Stars. Понятно, что это какие-то звёзды на гитхабе. В PR я вижу значёк звезды, и слово звёзды. Как старый житель Доки, который знает, что Дока лежит на гитхабе, я пойму. Как новый — увы, будет сложнее. Я бы призвал @skorobaeus пошаманить над визуалом. |
|
Интересное начинание, но, кажется, способ запроса совсем не подходит нам. Прямые запросы на GitHub уже в прошлом. Так что эту часть нужно будет переделать. Доберусь в январе и с удовольствием помогу :) Но это техническая сторона. А вот дизайнерская сторона вопроса — тут @skorobaeus. И надо будет сделать проверку на тысячу. Мы должны проработать все сценарии, поддержу @vitya-ne |
Спасибо за интересное начинание! Огонь, окей с тысячью. Думаю я тут смыслю в коммерческом поле, что за трату часов на такую проверку дали бы по шапке. Но в парадигме open-source кода, с анлимом времени и работой на вольных хлебах имеет смысл делать все проверки, все условия и всё всё всё. Согласен ещё раз! Очень интересно с тейком про прямые запросы. Могу лишь гадать про некое ограничения на количество запросов, которое может аукнуться, или желание сделать некое кеширование. В общем жду с нетерпением январь) |
Навеяно ишью #1308
Предлагаю добавить кастомный виджет с поддержкой смены цветовой темы (а ещё в дизайн-коде Доки и на русском).
Если про сам дизайн, то использовал скругления и цвета с переключателя цветовой темы. По общей структуре ориентировался на такой вариант:
Разумеется всё обсуждаемо, особенно дизайн @skorobaeus 🙌
Было:
Стало:
В тёмной теме: