Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(components/tabs): update tab #759 #808

Merged
merged 11 commits into from
Oct 13, 2023
Merged

fix(components/tabs): update tab #759 #808

merged 11 commits into from
Oct 13, 2023

Conversation

ZurabDev
Copy link
Contributor

@ZurabDev ZurabDev added the state:ready ready for release (pass tested) label Oct 11, 2023
@github-actions
Copy link

github-actions bot commented Oct 11, 2023

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

https://prizm-v3--pr808-feat-tabs-update-tab-qquxk9w3.web.app

(expires Wed, 18 Oct 2023 12:45:26 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 6cf3db9b3e1bf46180849c1db2e28221d0904ddf

@github-actions
Copy link

github-actions bot commented Oct 11, 2023

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

https://prizm-v2--pr808-feat-tabs-update-tab-ihfss5w0.web.app

(expires Wed, 18 Oct 2023 12:47:03 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: ab7e21e23e82e687a92ac5a830ac2768df6e61b5

@github-actions
Copy link

github-actions bot commented Oct 11, 2023

Visit the preview URL for this PR (updated for commit 5ce66be):

https://prizm-v1--pr808-feat-tabs-update-tab-djmr47aa.web.app

(expires Wed, 18 Oct 2023 12:47:37 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 3f40bac4652dbd13be483e7e4de02de469ed4b3f

@ZurabDev
Copy link
Contributor Author

вот можно проверять, обновил последний пример для вывода active idx

Copy link

@alexhawkins94 alexhawkins94 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ощущение что индексы сбиваются. см доработанный пример Closable tabs

закрыл несколько табов - номер 2 и 4, у оставшихся получились индексы 0, 1, 3
image
ожидал что будут индекс 0 1 2

другой кейс
4й таб закрыл. у последнего таба индекс стал 4, ожидал что будет 3 - т.е идут 0, 1, 2 и сразу 4
image

Copy link

@alexhawkins94 alexhawkins94 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

переключаюсь сперва на таб№2.
открываю кебаб
закрываю из кебаба таб №5
пытаюсь открыть таб№1
ФР: он не становится активным. не обновляется active index по клику на таб№1
ОР: корректное переключение между табами и проброс индекса активного таба
image

Copy link

@alexhawkins94 alexhawkins94 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

в примере по умолчанию active index = 0
жму на кнопочку открытия дропдауна
первый таб визуально теряет свой active state (перестает быть синим)
закрываю дропдаун
ФР:меняется active index на =1 и active state появляется у таба 2
ОР: открытие и закрытие дропдауна с табами не должно влиять на активный таб и приводить к переключению на другой таб

chrome_OgKE14F7Y6

chrome_LoC2tsIw9h

chrome_54xeqfYw1W

@ZurabDev ZurabDev added this to the 3.2.0 milestone Oct 13, 2023
ZurabDev and others added 2 commits October 13, 2023 15:38
…nt.ts

Co-authored-by: ickisIckis <darinafjordur@gmail.com>
…nt.ts

Co-authored-by: ickisIckis <darinafjordur@gmail.com>
@ZurabDev ZurabDev merged commit 911e6be into main Oct 13, 2023
5 of 9 checks passed
This was referenced Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
state:ready ready for release (pass tested)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] @prizm-ui/components - PrizmTabsComponent отдаёт неверный индекс вкладки
3 participants