Skip to content

Set Obsidian plugin to desktop-only to suppress mobile loading errors#1109

Open
nathancashion wants to merge 2 commits into
DiscourseGraphs:mainfrom
nathancashion:fix/disable-mobile-loading
Open

Set Obsidian plugin to desktop-only to suppress mobile loading errors#1109
nathancashion wants to merge 2 commits into
DiscourseGraphs:mainfrom
nathancashion:fix/disable-mobile-loading

Conversation

@nathancashion

@nathancashion nathancashion commented Jun 6, 2026

Copy link
Copy Markdown

Summary

  • Sets isDesktopOnly: true in apps/obsidian/manifest.json to prevent the plugin from loading on mobile devices
  • The plugin currently relies on desktop-specific APIs (CodeMirror 6 editor extensions, tldraw canvas views, DOM event handlers for drag-and-drop) that cause errors when Obsidian attempts to load it on mobile
  • This is intended as a stopgap — iPad support is plausible in the future since most of the plugin's core functionality (CM6 extensions, vault access, Supabase sync) can work on tablet, but a dedicated mobile compatibility effort would be needed for touch interactions and canvas features

Reposted from DiscourseGraphs/discourse-graph-obsidian#6 per @trangdoan982's guidance, since the publish mirror gets overwritten from this monorepo.

Test plan

  • Verify plugin loads normally on desktop Obsidian
  • Verify plugin no longer appears / causes errors on Obsidian mobile

Open in Devin Review

@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

@mdroidian mdroidian requested a review from trangdoan982 June 9, 2026 20:29

@trangdoan982 trangdoan982 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm. Thank you Nathan!

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Jun 9, 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 Jun 9, 2026 9:36pm

Request Review

@trangdoan982

Copy link
Copy Markdown
Member

@nathancashion you can do the honor of merging =D

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.

2 participants