Skip to content

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

Merged
lucaseduoli merged 1 commit into
mainfrom
fix/disable_change_tabs
Mar 26, 2026
Merged

fix: change navigation to be blocked when switching tabs#1268
lucaseduoli merged 1 commit into
mainfrom
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 self-assigned this Mar 26, 2026
@github-actions github-actions Bot added frontend 🟨 Issues related to the UI/UX bug 🔴 Something isn't working. labels Mar 26, 2026

@mpawlow mpawlow left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@lucaseduoli

Code Review 1

  • ✅ LGTM / Approved
  • Approval based on changes already reviewed & approved for release-0.4.0

@github-actions github-actions Bot added the lgtm label Mar 26, 2026
@lucaseduoli lucaseduoli merged commit 75ca269 into main Mar 26, 2026
11 checks passed
@github-actions github-actions Bot deleted the fix/disable_change_tabs branch March 26, 2026 18:08
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.

[Bug]: Buggy behaviour on chat interface if tabs are switched mid response generation

2 participants