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.

Sign up for GitHub

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

Jump to bottom

Витрина. Navigation. Лишние уведомления об activeItemChanged при переходе из live demo компонента Navigation обратно в Examples #571

Closed
alexhawkins94 opened this issue Aug 7, 2023 · 0 comments · Fixed by #1080
Closed

Витрина. Navigation. Лишние уведомления об activeItemChanged при переходе из live demo компонента Navigation обратно в Examples #571

alexhawkins94 opened this issue Aug 7, 2023 · 0 comments · Fixed by #1080
Assignees
@ickisIckis
Labels
source:internal проблемы, найденные внутренним тестированием/командой; поставленные задачи из беклога и тд state:ready ready for release (pass tested) type:bug Something isn't working
Milestone
3.6.0

Comments

@alexhawkins94
Copy link

alexhawkins94 commented Aug 7, 2023 •
edited
Loading

-Перейти в live demo компонента Navigation
-Перейти на вкладку examples
ФР: появляются два уведомления о событии active item changed, эти уведомления отображаются на вкладке Examples
ОР: на мой взгляд такого быть не должно, все эти уведомления д.б. ограничены разделом live demo
chrome_Cx7g1BbItm
@alexhawkins94 alexhawkins94 added type:bug Something isn't working source:internal проблемы, найденные внутренним тестированием/командой; поставленные задачи из беклога и тд labels Aug 7, 2023
@ZurabDev ZurabDev added the s2 label Oct 27, 2023
@ZurabDev ZurabDev added this to the 3.6.0 milestone Oct 27, 2023
@ickisIckis ickisIckis self-assigned this Nov 13, 2023
ickisIckis pushed a commit that referenced this issue Nov 23, 2023
fix: navigation extra notification for active item change #571
4bb5da5
@ickisIckis ickisIckis mentioned this issue Nov 23, 2023
Merged
@ickisIckis ickisIckis added the state:ready ready for release (pass tested) label Nov 23, 2023
ZurabDev added a commit that referenced this issue Nov 27, 2023
@ZurabDev
Merge pull request #1080 from zyfra/fix/navigation-active-item-change
b1fd525 
fix: navigation extra notification for active item change #571
@ZurabDev ZurabDev mentioned this issue Nov 27, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ickisIckis ickisIckis

Labels
source:internal проблемы, найденные внутренним тестированием/командой; поставленные задачи из беклога и тд state:ready ready for release (pass tested) type:bug Something isn't working
Projects
Roadmap Prizm
Archived in project
Milestone
3.6.0
Development

Successfully merging a pull request may close this issue.

fix: navigation extra notification for active item change #571 zyfra/Prizm
3 participants
@ickisIckis @ZurabDev @alexhawkins94