Skip to content

ENG-2060 Remove discourse context overlay button from blocks on canvas - #1287

Open
sid597 wants to merge 1 commit into
mainfrom
eng-2060-remove-discourse-context-overlay-button-from-blocks-on
Open

ENG-2060 Remove discourse context overlay button from blocks on canvas#1287
sid597 wants to merge 1 commit into
mainfrom
eng-2060-remove-discourse-context-overlay-button-from-blocks-on

Conversation

@sid597

@sid597 sid597 commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator
eng-2060.mp4

Blocks on a tldraw canvas were showing the discourse context overlay button, but discourse context does not apply to blocks. This skips rendering the overlay for blck-node shapes, using the same check the component already uses for block detection.


Open in Devin Review

@linear-code

linear-code Bot commented Aug 10, 2026

Copy link
Copy Markdown

ENG-2060

@vercel

vercel Bot commented Aug 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
discourse-graph Skipped Skipped Aug 10, 2026 10:08am

Request Review

@supabase

supabase Bot commented Aug 10, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project zytfjzqyijgagqxrzbmz because there are no changes detected in packages/database/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@sid597
sid597 requested a review from mdroidian August 10, 2026 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant