Skip to content

fix(editor): Prevent Ask Assistant button overlapping other buttons #16066

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.

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

Conversation

Cadiac
Copy link
Contributor

@Cadiac Cadiac commented Jun 5, 2025

Summary

On small screens Ask AI assistant button would cover the execute workflow / open chat buttons.

Nudge that button up when we hit the small screen media breakpoint, which is when those other buttons get positioned to the right edge.

ask-assistant-button-small-screens.mp4

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/ADO-3601/bug-ai-assistant-button-is-covering-the-stop-execution-button-on-small

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

@n8n-assistant n8n-assistant bot added the n8n team Authored by the n8n team label Jun 5, 2025
@Cadiac Cadiac requested a review from a team June 5, 2025 13:02
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

cubic found 2 issues across 1 file. Review them in cubic.dev

React with 👍 or 👎 to teach cubic. Tag @cubic-dev-ai to give specific feedback.

@Cadiac Cadiac requested a review from RicardoE105 June 5, 2025 13:02
@Cadiac Cadiac force-pushed the ado-3601-bug-ai-assistant-button-is-covering-the-stop-execution branch from 47e6dee to e582891 Compare June 5, 2025 13:05
Copy link

codecov bot commented Jun 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

@Cadiac Cadiac requested a review from dariacodes June 9, 2025 06:24
Copy link
Contributor

github-actions bot commented Jun 9, 2025

⚠️ Some Cypress E2E specs are failing, please fix them before merging

Copy link
Contributor

github-actions bot commented Jun 9, 2025

✅ All Cypress E2E specs passed

@Cadiac Cadiac merged commit 4426249 into master Jun 9, 2025
33 checks passed
@Cadiac Cadiac deleted the ado-3601-bug-ai-assistant-button-is-covering-the-stop-execution branch June 9, 2025 09:28
@janober
Copy link
Member

janober commented Jun 11, 2025

Got released with [email protected]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
n8n team Authored by the n8n team Released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants