Conversation
nasty23-star
left a comment
There was a problem hiding this comment.
Большое спасибо! этого очень не хватало<3
igsekor
left a comment
There was a problem hiding this comment.
@vitya-ne очень круто ❤️
Но есть нюансы...
- Мы с самого начала все скрипты хранили в scripts, именно поэтому их не надо было отдельно подключать. Почему было выбрано решение поместить
doka-baseline.jsв отдельную папочкуlibs? Отсюда появились проблема с тем, что нарушена консистентность, и необходимость добавлять в.eleventy.jsиmeta.njkновые строки, хотя необходимости в этом не было. - Кажется хорошо бы добавить отступ снизу, чтобы было совсем хорошо:
А может попробовать две ссылки внизу в одну строку поместить, будет немного компактнее? |
|
@igsekor, привет
Я сомневался куда лучше положить веб-компонент. Нет проблем, перенесу в |
|
@furtivite, привет
Да, можно попробовать, перенесу. |
Спасибо, исправлю. Вроде я это фиксил. Проверю |
|
А чем «Подробнее» от «Спецификация» отличаются? Надо бы, наверное, более точно назвать, как думаешь? Нет ли тут неоднозначности? А может быть и нет.... Не знаю, но сомнения есть. Что скажете? @solarrust @skorobaeus @TatianaFokina @HellSquirrel |
'Подробнее' - это ссылка на страницу Web platform features explorer. Тоесть там подробности как фича выглядет с точки зрения baseline |
|
@igsekor Я перенёс веб-компонент в папку |
There was a problem hiding this comment.
Не стоит хранить минифицированную версию в репозитории. Тут нужны либо полные исходники, либо выносить оригинальный код в npm-пакет.
There was a problem hiding this comment.
Возможно это именно то, что меня долго тормозило на старте - непонимание как "правильно" веб-компонент должен подключаться к платформе.
- Полные исходники - не уверен, кто-то (что-то) же должно минимизировать код выполняющийся на стороне клиента.
- npm-пакет - возможно, но хотелось бы сначала понимать, что код компонента готов, заапрувлен и его не нужно менять через день )
There was a problem hiding this comment.
кто-то (что-то) же должно минимизировать код выполняющийся на стороне клиента.
Для этого есть бандлер
There was a problem hiding this comment.
Для этого есть бандлер
ок. добавлю исходник )
There was a problem hiding this comment.
В самом начале файла есть минифицированный код, как полагаю, для Lit. В идеале нужно поставить его как зависимость и импортировать:
import { LitElement } from 'lit';Это нужно для того, чтобы бандлер смог включить лишь одну версию Lit, если его захочется использовать его где-то ещё.
There was a problem hiding this comment.
да это Lit.
Звучит разумно, но не знаю как это осуществить. Сборка компонента делается рекомендованным конфигом Lit.
There was a problem hiding this comment.
@monochromer Вроде получилось, спасибо за комент
Привет, |
|
|
А почему их два?
|
две группы -> две плашки |
Могу добавить показ имени группы в каждой плашке если их несколько |
|
В MDN просто звёздочку лепят, мол, может не всё поддерживаться
Думаю, можно призвать @skorobaeus, чтобы обсудить, как оно лучше будет выглядеть P.S. Кстати, скрин MDN с той же статьи |
Данные которые отдаёт запрос, используемый в веб-компоненте - это только данные по группе, а не по фичи в группе. |
Я думаю, стоит попробовать. Две одинаковые плашки кофузят :) |
@skorobaeus @StarHamster |
Добавляет показ состояния загрузки
|
@igsekor @StarHamster Я учёл все (надеюсь) пожелания. Было бы здорово если бы вы посмотрели, на то как это выглядит сейчас и дали свой вердикт |
|
Превью контента из b3d428c опубликовано. |
igsekor
left a comment
There was a problem hiding this comment.
Кажется, стало совсем здорово! Витя, ты молодец!
|
@furtivite @igsekor @StarHamster @solarrust ПроблемаОсновной недостаток нового подхода: Например: <doka-baseline groupid="light-dark"></doka-baseline>На web-platform-dx/web-features для Но для многих CSS-свойств, это совсем не так. Яркий пример Anchor positioning: Тоесть для точности (см. MDN) нужно иметь возможность передать не только baseline:
- group: anchor-positioning
features:
- css.properties.width.anchor-size
- css.properties.height.anchor-size
- css.properties.inline-size.anchor-size
- css.properties.block-size.anchor-sizeПредложение
Вопрос: Как вы считаете, что лучше ? |
Может быть сделать отдельный атрибут |
Вопрос не совсем в синтаксисе, а в том как именно проверять фичи: делать доп. запрос или использовать данные web-features. |
Ну если вопрос в этом, то предлагаю всё таки подгружать данные дополнительным запросом на этапе сборки. |








Удаляет пакет web-features, содержащего базу baseline и нуждающегося в переодическом обновлении.
Добавляет веб-компонент
<doka-baseline>:Репозиторий веб-компонента: https://github.com/vitya-ne/doka-baseline
Особенности:
newlyилиwidly;showFeatLink="true"показывает ссылку на страницу https://web-platform-dx.github.io c подробной информацией о фиче;showSpecLinks="true"показывает ссылки на спеку фичи;