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

feat(YfmTabs): switch between tabs using tabs-extension runtime #550

Merged
merged 2 commits into from
Jan 22, 2025

Conversation

d3m1d0v
Copy link
Member

@d3m1d0v d3m1d0v commented Jan 16, 2025

No description provided.

@gravity-ui-bot
Copy link
Contributor

Preview is ready.

@d3m1d0v d3m1d0v marked this pull request as ready for review January 20, 2025 13:55
@makhnatkin
Copy link
Collaborator

I believe this is an issue with the current tab implementation in Diplodoc (id of tabs), but it seems to have become more noticeable with the recent changes to tabs. Storybrook scripts intercept clicks

Here’s the scenario:
1. I create tabs with 3 empty sections.
2. I start entering data.

In the old code, this issue doesn’t occur in this case (though it does if I fill in everything except the first tab).

Screen.Recording.2025-01-20.at.17.45.15.mov

In the new code, I can’t enter anything at all.

Screen.Recording.2025-01-20.at.17.46.21.mov

Please check it

@d3m1d0v d3m1d0v force-pushed the d3m1d0v/yfm-tabs-switch-runtime branch from d612670 to b278a8d Compare January 22, 2025 09:22
@d3m1d0v d3m1d0v force-pushed the d3m1d0v/yfm-tabs-switch-runtime branch from b278a8d to ee8095c Compare January 22, 2025 09:48
@d3m1d0v
Copy link
Member Author

d3m1d0v commented Jan 22, 2025

Please check it

fixed

@d3m1d0v d3m1d0v merged commit 3d223cc into main Jan 22, 2025
3 checks passed
@d3m1d0v d3m1d0v deleted the d3m1d0v/yfm-tabs-switch-runtime branch January 22, 2025 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants