Skip to content

fix: change navigation to be blocked when switching tabs#1257

Merged
lucaseduoli merged 1 commit into
release-0.4.0from
fix/disable_change_tabs
Mar 25, 2026
Merged

fix: change navigation to be blocked when switching tabs#1257
lucaseduoli merged 1 commit into
release-0.4.0from
fix/disable_change_tabs

Conversation

@lucaseduoli

Copy link
Copy Markdown
Collaborator

This pull request updates the navigation component to improve user experience during loading states. The main change is that navigation tabs (except the currently active one) are now visually disabled and non-interactive when a loading state is active. This prevents users from navigating away while content is loading.

Navigation loading state improvements:

  • Updated the navigation tab rendering logic in frontend/components/navigation.tsx to render non-active tabs as disabled (with reduced opacity and a not-allowed cursor) and as non-clickable divs instead of links when a loading state is present. This ensures users cannot navigate to other tabs while loading. [1] [2]

Closes #1216

@lucaseduoli lucaseduoli requested a review from Wallgau March 25, 2026 14:43
@lucaseduoli lucaseduoli self-assigned this Mar 25, 2026
@github-actions github-actions Bot added the frontend 🟨 Issues related to the UI/UX label Mar 25, 2026
@github-actions github-actions Bot added the lgtm label Mar 25, 2026
@lucaseduoli lucaseduoli changed the title Change navigation to be blocked when switching tabs fix: change navigation to be blocked when switching tabs Mar 25, 2026
@github-actions github-actions Bot added the bug 🔴 Something isn't working. label Mar 25, 2026
@lucaseduoli lucaseduoli merged commit d913ffc into release-0.4.0 Mar 25, 2026
10 of 11 checks passed
@github-actions github-actions Bot deleted the fix/disable_change_tabs branch March 25, 2026 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 🔴 Something isn't working. frontend 🟨 Issues related to the UI/UX lgtm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants