Conversation
| @@ -0,0 +1,4 @@ | |||
| Дока описывает публичное апи компонента, без привязки к стэку. | |||
| То есть работа с формами, состояния (hover/foces/etc), валидации в этой документации не описываются, так как способы реализации и связанные с этим свойства для каждого стэка свои. | |||
There was a problem hiding this comment.
опечатка: foces вместо focus
| | ---------------- | ----------------------------------------------- | ----------------- | | ||
| | `label` | текст названия поля | `string` | | ||
| | `label-position` | положение лейбла | `default \| left` | | ||
| | `note` | текст пояснения под лейблом | `string` | |
There was a problem hiding this comment.
note – предлагаю заменить на caption
| | `label` | текст названия поля | `string` | | ||
| | `label-position` | положение лейбла | `default \| float \| left` | | ||
| | `clearable` | отображение иконки для очистки поля | `boolean` | | ||
| | `note` | текст пояснения под полем | `string` | |
There was a problem hiding this comment.
note – предлагаю заменить на caption
| | `filter-placeholder` | текст подсказки в фильтре | `string` | | ||
| | `options` | список элементов или список групп с элементами | `ExtraMultiselectGroup[] \| ExtraMultiselectOption[] \| any[]` | | ||
| | `optionLabel` | наименование поля, содержащего отображаемое значение | `string` | | ||
| | `note` | текст пояснения под полем | `string` | |
There was a problem hiding this comment.
note – предлагаю заменить на caption
| @@ -0,0 +1,28 @@ | |||
| [Открыть в Figma →](https://www.figma.com/design/Q1BWgZ7zoV5UzlBOnjW0cM/UI-Kit--DS--v2.1?node-id=1207-3811&m=dev) | |||
|
|
|||
| # ExtraCarousel | |||
There was a problem hiding this comment.
по фигме у карусели можно менять иконки кнопок например
нужно проверить соответствие апи
|
|
||
| | Свойство | Описание | Типизация | | ||
| | -------------- | ------------------------------------------- | ---------- | | ||
| | `modal` | отображать маску (overlay) поверх интерфейса | `boolean` | |
There was a problem hiding this comment.
Всегда бесило это в старых библиотеках. Поле modal отвечает за наличие overlay.
Может тут нормально назовем? hasOverlay или withOverlay? (проверить остальные modal флаги)
Вопрос 2.
Есть некий стайл гайд на именование bool-полей? with-/ has- or just overlay?
| | Свойство | Описание | Типизация | | ||
| | -------------- | ------------------------------------------- | ---------- | | ||
| | `modal` | отображать маску (overlay) поверх интерфейса | `boolean` | | ||
| | `maximizable` | кнопка разворота окна на весь экран | `boolean` | |
There was a problem hiding this comment.
Тут тоже косяк. Речь идет про наличие такой возможности. Но флаг определяющий это почему-то прилагательное. Хотя в этом же компоненте для таких целей используются существительные.
| | `maximizable` | кнопка разворота окна на весь экран | `boolean` | | ||
| | `show-close` | кнопка закрытия окна | `boolean` | | ||
|
|
||
| # Слоты (content projection) |
There was a problem hiding this comment.
думаю стоит учесть возможность передачи обычных строк для прямого отображения, в случае отсутствующей необходимости кастомизации модалки.
| | Свойство | Описание | Типизация | | ||
| | ---------------- | ----------------------------------------------- | ----------------- | | ||
| | `label` | текст названия поля | `string` | | ||
| | `label-position` | положение лейбла | `default \| left` | |
There was a problem hiding this comment.
default / left - не понятно что в таком случае делает default?
| | `timer` | таймер автоскрытия | `boolean` | | ||
| | `message` | заголовок сообщения | `string` | | ||
| | `caption` | заголовок сообщения | `string` | | ||
| | `icon` | класс иконки tabler icon (вместо стандартной для `severity`) | `string` | |
There was a problem hiding this comment.
нужна возможность скрытия икнонки. Что-то из:
icon=null
icon=none
| | `severity` | тип сообщения | `info \| success \| warning \| danger` | | ||
| | `timer` | таймер автоскрытия | `boolean` | | ||
| | `message` | заголовок сообщения | `string` | | ||
| | `caption` | заголовок сообщения | `string` | |
There was a problem hiding this comment.
caption - текст сообщения
| | -------- | --------------- | ------------------ | | ||
| | `name` | текст варианта | `string` | | ||
| | `code` | идентификатор | `string \| number` | | ||
| | `icon` | класс иконки tabler icon | `string` | |
There was a problem hiding this comment.
Это поле можно не указывать - в спеке должна быть пометка
3 participants
No description provided.