Skip to content

Fix nudges gradient#2037

Closed
mfortman11 wants to merge 5 commits into
mainfrom
fix-nudges-gradient
Closed

Fix nudges gradient#2037
mfortman11 wants to merge 5 commits into
mainfrom
fix-nudges-gradient

Conversation

@mfortman11

@mfortman11 mfortman11 commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Removes gradient on the right of the nudges in saas

Before:
Screenshot 2026-07-07 at 10 29 36 AM

Summary by CodeRabbit

  • Bug Fixes
    • Removed a visual overlay from the suggestion list for a cleaner scrolling experience.
    • Improved refresh behavior after bulk delete and role changes, making updates appear faster and more reliably.
    • Streamlined connector status checks so available integrations load more efficiently.

@github-actions github-actions Bot added community frontend 🟨 Issues related to the UI/UX labels Jul 7, 2026
@mfortman11 mfortman11 closed this Jul 7, 2026
@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

React Doctor found no new issues. 🎉

Reviewed by React Doctor for commit 61ae5e4.

@coderabbitai

coderabbitai Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: ea67b3b0-2e57-4d1b-9f4a-0c5738cdd79a

📥 Commits

Reviewing files that changed from the base of the PR and between 758541f and 61ae5e4.

📒 Files selected for processing (4)
  • frontend/app/chat/_components/nudges.tsx
  • frontend/app/knowledge/page.tsx
  • frontend/components/dev-role-toggle.tsx
  • frontend/components/knowledge-dropdown.tsx

Walkthrough

Three components (knowledge page, dev-role toggle, knowledge dropdown) were refactored to run previously sequential async refresh/invalidation/fetch operations concurrently via Promise.all. Additionally, a gradient fade overlay was removed from the Nudges suggestion scroller UI.

Changes

Concurrency Refactors and UI Cleanup

Layer / File(s) Summary
Parallelize bulk delete refresh
frontend/app/knowledge/page.tsx
handleBulkDelete refreshes tasks and invalidates/refetches search and listFiles queries concurrently via Promise.all instead of sequential awaits.
Parallelize dev role toggle refresh
frontend/components/dev-role-toggle.tsx
onSuccess runs refreshPermissions and two query invalidations concurrently via Promise.all.
Parallelize cloud connector status/token fetches
frontend/components/knowledge-dropdown.tsx
checkAvailability filters to connector types present in results, then fetches status and token endpoints concurrently per type with per-type error handling.
Remove suggestion scroller fade overlay
frontend/app/chat/_components/nudges.tsx
Removes the absolutely positioned right-edge gradient overlay from the suggestion scroller.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested labels: bug

Suggested reviewers: Wallgau

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-nudges-gradient

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@mfortman11
mfortman11 deleted the fix-nudges-gradient branch July 7, 2026 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community frontend 🟨 Issues related to the UI/UX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant