fix(ai-builder): Fix chat fade, keyboard shortcuts, node tidy-up, and feedback styling#26200
Merged
mike12345567 merged 4 commits intomasterfrom Feb 25, 2026
Merged
Conversation
… feedback styling - Remove gradient fade from chat messages area and prompt container background (AI-2097) - Stop stealing focus to chat input when clicking canvas nodes, so keyboard shortcuts (copy/paste/delete) work while WFB is open (AI-2100) - Run tidy-up on every workflow update and add final tidy-up when streaming ends so nodes stay arranged during generation (AI-2101) - Remove full-width background/borders from feedback icons wrapper and add top padding to expanded feedback form Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Bundle ReportChanges will increase total bundle size by 17.48kB (0.04%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: editor-ui-esmAssets Changed:
Files in
Files in
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
This comment has been minimized.
This comment has been minimized.
aalises
previously approved these changes
Feb 25, 2026
Contributor
There was a problem hiding this comment.
2 issues found across 8 files
Prompt for AI agents (all issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="packages/frontend/editor-ui/src/features/ai/assistant/components/Agent/AskAssistantBuild.vue">
<violation number="1" location="packages/frontend/editor-ui/src/features/ai/assistant/components/Agent/AskAssistantBuild.vue:429">
P1: `accumulatedNodeIdsToTidyUp` must be cleared after the final tidy-up so subsequent prompts don't incorrectly re-arrange previously generated nodes.</violation>
<violation number="2" location="packages/frontend/editor-ui/src/features/ai/assistant/components/Agent/AskAssistantBuild.vue:432">
P2: Final tidy-up layout changes will be isolated in a separate undo step because they are emitted asynchronously after `historyStore.stopRecordingUndo()` has already closed the undo recording block.</violation>
</file>
Architecture diagram
sequenceDiagram
participant U as User
participant C as Canvas Component
participant WFB as AI Chat Panel (WFB)
participant WUC as useWorkflowUpdate (Composable)
participant EB as Canvas Event Bus
participant BS as Builder Store
Note over U, EB: Workflow Generation & Node Arrangement Flow
rect rgb(23, 37, 84)
Note right of WFB: NEW: Enhanced Tidy-up Logic
WFB->>WUC: updateWorkflow(nodes, edges)
WUC->>BS: setBuilderMadeEdits(true)
WUC->>EB: NEW: emit('tidyUp') (Triggered on every update)
EB-->>C: Re-layout nodes
end
opt On AI Streaming Completion
WFB->>EB: NEW: emit('tidyUp') (Final layout pass)
WFB->>EB: emit('fitView')
end
Note over U, EB: Canvas Interaction & Focus Flow
U->>C: Click Node
C->>BS: confirmNodes(nodeId)
Note over C, WFB: CHANGED: No longer requests focus for Chat Input<br/>Allows keyboard shortcuts (Copy/Paste) to stay on Canvas
Note over U, EB: User Feedback Flow
U->>WFB: Click Rating/Feedback
WFB->>WFB: Toggle Feedback Form
Note over WFB: CHANGED: Dynamic styling based on expansion state<br/>(Removes full-width borders, adjusts padding)
Note over WFB: UI Visual Changes
Note right of WFB: REMOVED: Gradient fades in chat/prompt areas<br/>CHANGED: Solid backgrounds for message/input containers
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
packages/frontend/editor-ui/src/features/ai/assistant/components/Agent/AskAssistantBuild.vue
Show resolved
Hide resolved
packages/frontend/editor-ui/src/features/ai/assistant/components/Agent/AskAssistantBuild.vue
Outdated
Show resolved
Hide resolved
aalises
approved these changes
Feb 25, 2026
Merged
Tuukkaa
pushed a commit
that referenced
this pull request
Mar 2, 2026
… feedback styling (#26200) Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
|
Got released with |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Addresses: https://linear.app/n8n/issue/AI-2101/bug-tidy-up-of-nodes-after-wfb-finishes-not-working-correctly
Addresses: https://linear.app/n8n/issue/AI-2100/bug-copypaste-keyboard-shortcuts-in-general-captured-by-wfb
Addresses: https://linear.app/n8n/issue/AI-2097/bug-wfb-chat-remove-fade-as-hard-borders-make-it-un-necessary
Screenshots
Review / Merge checklist
release/backport(if the PR is an urgent fix that needs to be backported)