Детали изменений в оформлении тем — обратная связь

Вы правы, но проблема кроется в различном способе настройки этих компонентов: один использует d-icon, а другой в данный момент не позволяет этого сделать. Было бы здорово увидеть изменения, так что не стесняйтесь помочь, если знаете, как это сделать.

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

Это постоянные споры среди дизайнеров с момента появления переключателя. Нет правильного или неправильного ответа.


Screenshot 2023-11-14 at 1.48.35 PM

Это была одна из идей с дизайнерской сессии, которая мне лично очень понравилась. Она решает (часть) замечаний Sam и cvx. Я попробую реализовать это в следующей итерации.

3 лайка
мои мысли об этом пока что...

Выглядит вполне неплохо!

4 лайка

Я думаю, что шеврон лучше разместить слева. Я считаю, что большинство людей не заметят его справа. Он выглядит как обычный текст (что-то вроде блока кода), если не навести на него курсор. Я почти его пропустил.

3 лайка

(14. :wink:) Недавнее добавление обработчика кликов означает, что клик по ссылке, использование кнопки копирования кода, попытка развернуть изображение или даже попытка выделить любой текст приведут к сворачиванию блока деталей… :eyes:

4 лайка

Я полностью согласен с этим. Я (почти) пропустил чтение нескольких блоков с деталями за последние пару дней. Мне кажется, это потому, что они на первый взгляд выглядят похожими на блоки кода.

это не код

пусто

это код
8 лайков

Странно, у меня клик по ссылке работает. Блок details сворачивается, но ссылка при этом тоже открывается. Остальные сценарии я не проверял. И я согласен, что тут можно немного доработать JS :winking_face_with_tongue:

Касательно положения стрелочки, @jordan.vidrine, что думаешь?
Стоит ли мне сделать ещё одну итерацию? Может, как я изначально задумывал: с обычным треугольником вместо стрелочки и фоном только в открытом состоянии?

1 лайк

Я думаю, галочку всё же стоит использовать, но размещение слева — отличная идея.

1 лайк

Вы имеете в виду что-то другое? Мы не добавляли обработчик кликов здесь.

Обработчик был. Чарли удалил его сегодня в 2c941c4. :slightly_smiling_face:

Ха! Вау, я wondered, что это было. Прямо в лоб, а я не заметил.

2 лайка

Да, мне очень понравилась идея возможности навести курсор в любое место → выделить → кликнуть, чтобы закрыть, вместо того чтобы кликать только на часть с кратким описанием, но я упустил распространение других кликов :woman_facepalming:

Также очень легко упустить детали в цитатах, так как фоновая подсветка почти невидима, а стрелка находится далеко от текста

5 лайков

Несколько придирчивых мыслей о последней итерации…

Не стоит ли выравнивать стрелку по базовой линии, а не по центру? Для меня это выглядит лучше, чем центрирование в многострочных примерах… хотя это может быть субъективно:

по центру

по базовой линии

должны ли отступы соответствовать цитатам? Верхняя часть сейчас кажется немного тесной

если мы добавим отступ ко всему контейнеру, возможно, стоит убрать отступы у последнего элемента… это может оказаться слишком сложным, так как у нас много типов контента… но, возможно, что-то вроде:

details > :last-child {
  padding-bottom: 0;
}

Я согласен, что в цитате стоит сделать цвет немного темнее, чтобы фон не терялся

Summary

Этот текст будет скрыт

Думаю, можно было бы использовать какую-то переменную для решения этой задачи… если бы у details фон задавался чем-то вроде

:root {
  --d-details-bg: var(--primary-100);
}

details {
  background: var(--d-details-bg);
}

то в цитате можно было бы переопределить это… возможно, даже ввести универсальную переменную «нужно сделать темнее» для цитат? Не помню, есть ли у нас подобные ситуации…

blockquote {
  --darker-quote-bg: var(--primary-200);
  --d-details-bg: var(--darker-quote-bg); 
}
4 лайка

На все 100% согласен — если сейчас это не причина, то это просто ошибка.

3 лайка

Мне нравится использовать для таких целей что-то вроде rgba(var(--primary-rgb),0.25), так как это просто немного затемняет текущий фон элемента, в который оно включено.

2 лайка

В целом у меня складывается впечатление, что последняя версия очень-очень близка к готовности к релизу :hugs:

Есть один небольшой нюанс, который, возможно, невозможно устранить:

Это не является критической проблемой, но серая панель создаёт впечатление, что вы смотрите на единый объект.

3 лайка

Да, я уверен, что мы сможем это решить, добавив, возможно, границу или что-то подобное, чтобы сделать это более понятным. Посмотрю сегодня.

1 лайк

В целом, мне это нравится. Тем не менее, это предложение привлекло моё внимание.

Пытаясь выбрать фон для текстового элемента, иногда чувствуешь себя так, будто ищешь идеальный оттенок светло-серого.

var(--primary-not-so-light-you-cant-see-it-but-not-so-dark-it-looks-clunky)

Но, похоже, вы уже близко, и в целом мне это нравится.

5 лайков

Я добавляю такое же затемнение фона для блока кода, как и для цитаты

Это должно решить проблему, @sam

2 лайка

Ещё два момента (15. и 16., если угодно :wink:)

  • Фактическая кликабельная область элемента summary меньше, чем область, которая меняет курсор и подсвечивается при наведении
  • Из-за этого при наведении цвет фона и последующее изменение при раскрытии элемента приводят к кратковременной вспышке цвета фона при раскрытии

edit:

  1. Несоответствие между отображаемым сообщением и редактором:

    против
2 лайка