Skip to content

Conversation

@NikGurev
Copy link
Contributor

@NikGurev NikGurev commented Aug 7, 2025

No description provided.

@NikGurev NikGurev self-assigned this Aug 7, 2025
@NikGurev NikGurev added the enhancement New feature or request label Aug 7, 2025
@github-actions
Copy link

github-actions bot commented Aug 7, 2025

Visit the preview URL for this PR (updated for commit 9fd9c2f):

https://koobiq-next--prs-967-gf35j1dc.web.app

(expires Tue, 26 Aug 2025 11:04:41 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c

@NikGurev NikGurev marked this pull request as ready for review August 12, 2025 15:06
@NikGurev NikGurev requested a review from lskramarov as a code owner August 12, 2025 15:06
@rmnturov
Copy link
Contributor

второе слово в названии компонента плиз с маленькой буквы в доках (наши редстандарты)
изображение

@rmnturov
Copy link
Contributor

rmnturov commented Aug 12, 2025

с рамкой пример выглядит как текстовое поле, так не надо
изображение

@rmnturov
Copy link
Contributor

текст в примере бы осмысленный, можно взять из фигмы
изображение

@rmnturov
Copy link
Contributor

rmnturov commented Aug 12, 2025

у кнопки Развернуть не надо подчеркивать иконку. Подчеркивается только текст в ховеру и active состояниях. так у всех псевдоссылок и ссылок по спеке

@NikGurev
Copy link
Contributor Author

NikGurev commented Aug 13, 2025

второе слово в названии компонента плиз с маленькой буквы в доках (наши редстандарты)

@rmnturov для заголовка оставить? я просто на фигму ориентировался

Везде с маленькой второе слово. Я в Фигме исправлю — я тоже ошибся

@rmnturov
Copy link
Contributor

rmnturov commented Aug 13, 2025

@NikGurev
Иконка не та в реализации. Неправильный отступ иконки от текста. Используется базовый для ссылки-псевдоссылки
изображение

Спека. Должный использоваться иконки обычного размера, без суффикса -s
изображение

@artembelik
Copy link
Contributor

а для чего baseClass в константу выносится?

@artembelik
Copy link
Contributor

не отображается код примера
image

@artembelik
Copy link
Contributor

предложение по улучшению теста документации, @rmnturov @NikGurev что думаете?


KbqClampedText — компонент для компактного отображения длинных текстовых блоков с возможностью их сворачивания и разворачивания. Позволяет ограничить количество видимых строк, обеспечивая удобство чтения и экономию пространства в интерфейсе.

example(clamped-text-overview)

Количество строк

Компонент поддерживает настройку числа отображаемых строк при помощи атрибута rows:

<kbq-clamped-text [rows]="3">Текст, который нужно обрезать после трех строк.</kbq-clamped-text>

@NikGurev
Copy link
Contributor Author

а для чего baseClass в константу выносится?

удобно было при разработке предыдущего компонента, здесь оно действительно не нужно

@NikGurev NikGurev requested a review from artembelik August 20, 2025 07:22
@NikGurev
Copy link
Contributor Author

не отображается код примера

поправил

@NikGurev NikGurev merged commit 9ad2b6a into main Aug 21, 2025
4 checks passed
@NikGurev NikGurev deleted the feat/DS-4051 branch August 21, 2025 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants