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

[Tab] Async context must not replace different tab #3156

Open
mvorisek opened this issue Jan 27, 2025 · 0 comments · May be fixed by #3160
Open

[Tab] Async context must not replace different tab #3156

mvorisek opened this issue Jan 27, 2025 · 0 comments · May be fixed by #3160
Labels
state/awaiting-investigation Anything which needs more investigation state/has-pr An issue which has a related PR open type/bug Any issue which is a bug or PR which fixes a bug

Comments

@mvorisek
Copy link
Contributor

mvorisek commented Jan 27, 2025 •
edited
Loading

Bug Report

Steps to reproduce

  1. open https://dev.agiletoolkit.org/demos/interactive/tabs.php
  2. click "Dynamic Form" tab
  3. click (quickly - before the AJAX completes) other/"Hello" tab
  4. notice when the AJAX load of "Dynamic Form" tab finishes, the "Hello" tab is replaced

related with atk4/ui#2244

Expected result

When tab is switched, any ongoing async load must be canceled.

Actual result

Tab content must never be replaced with content of different tab.

Version

2.9.3/latest
@mvorisek mvorisek added state/awaiting-investigation Anything which needs more investigation state/awaiting-triage Any issues or pull requests which haven't yet been triaged type/bug Any issue which is a bug or PR which fixes a bug labels Jan 27, 2025
@mvorisek mvorisek linked a pull request Jan 31, 2025 that will close this issue
Draft
@lubber-de lubber-de added state/has-pr An issue which has a related PR open and removed state/awaiting-triage Any issues or pull requests which haven't yet been triaged labels Feb 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
state/awaiting-investigation Anything which needs more investigation state/has-pr An issue which has a related PR open type/bug Any issue which is a bug or PR which fixes a bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix "$tab.activate.tab" called after renavigate wrongly mvorisek/Fomantic-UI
2 participants
@mvorisek @lubber-de