feat(app-switcher): added new component (#DS-4040) #1012
Conversation
|
Visit the preview URL for this PR (updated for commit a71fe1a): https://koobiq-next--prs-1012-t01aovqx.web.app (expires Sun, 12 Oct 2025 06:25:19 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c |
lskramarov
force-pushed
the
feat/DS-4040
branch
from
5df7084 to
3e6904d
Compare
|
Я ожидаю общий скролл на всё окно. В единой ленте прокручивается верхняя и нижняя часть. Поиск залипает 
|
lskramarov
force-pushed
the
feat/DS-4040
branch
from
1c5f677 to
2606f83
Compare
| changeDetection: ChangeDetectionStrategy.OnPush | ||
| }) | ||
| class DevExamples { | ||
| sites: KbqAppSwitcherSite[] = [ |
There was a problem hiding this comment.
не используется
подразумевалось, что в DevExamples будет содержаться только <ComponentName>ExamplesModule
| } | ||
| } | ||
|
|
||
| @mixin kbq-app-switcher-dropdown-app-theme() { |
There was a problem hiding this comment.
можно использовать приватные миксины, тогда stylelint подскажет, если миксин не будет использоваться
| @mixin kbq-app-switcher-dropdown-app-theme() { | |
| @mixin _kbq-app-switcher-dropdown-app-theme() { |
| overlayContainer.ngOnDestroy(); | ||
| }); | ||
|
|
||
| it('kbqTrigger = hover', fakeAsync(() => { |
There was a problem hiding this comment.
хотелось бы увидеть тесты
There was a problem hiding this comment.
позже, техдолгом...
| /** @docs-private */ | ||
| protected scrollStrategy: () => ScrollStrategy = inject(KBQ_APP_SWITCHER_SCROLL_STRATEGY); | ||
|
|
||
| // not used |
There was a problem hiding this comment.
нет, это унаследовано от базового popUp, который ранее был предназначен только для поповера и тултипа, нужно будет порефакторить и это уйдет
|
ввел в поиск букву "n" |
| standalone: true, | ||
| selector: '[kbqDropdownStaticContent]' | ||
| }) | ||
| export class kbqDropdownStaticContent {} |
There was a problem hiding this comment.
| export class kbqDropdownStaticContent {} | |
| export class KbqDropdownStaticContent {} |
| changeDetection: ChangeDetectionStrategy.OnPush, | ||
| standalone: true, | ||
| selector: 'app-switcher-overview-example', | ||
| imports: [ |
There was a problem hiding this comment.
большинство импортов не используется
|
Серая подпись по ошибке расположена чуть правее, будто пробел в начале строки есть |
|
Управления с клавы нет по элементам. Можно потом доработать. В режиме поиска тоже нужна навигация с клавы по элементам |
|
в режиме поиска выпадашка выходит за границу экрана |
|
между обычным режимом и поиском прыгает расположение подзаголовка и элементов списка |
|
@lskramarov @artembelik аналогичная ситуация. Ввожу одну цифру/букву, после чего дерево полностью раскрывается. Если ввести букву далее, то поиск отрабатывает корректно
|
Сделаю отдельно в DS-4246 |
Воспроизводится только при резайзе окна, тут видимо с позиционированием что то не в порядке, завел отдельный баг позже посмотрю DS-4248 |
lskramarov
force-pushed
the
feat/DS-4040
branch
from
d0bbdb1 to
c3ea8f9
Compare
No description provided.