Skip to content
This repository was archived by the owner on Aug 6, 2026. It is now read-only.

chore(desktop): spaces dragging fix + dev flag - #3916

Merged
trunk-io[bot] merged 2 commits into
mainfrom
chore/desktop/fix-dragging-window
Jul 29, 2026
Merged

chore(desktop): spaces dragging fix + dev flag#3916
trunk-io[bot] merged 2 commits into
mainfrom
chore/desktop/fix-dragging-window

Conversation

@tatoalo

@tatoalo tatoalo commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Problem

window dragging + dev flag for spaces layout experimentation

before:

nah.mp4

after:

yay.mp4

@tatoalo tatoalo self-assigned this Jul 29, 2026
@trunk-io

trunk-io Bot commented Jul 29, 2026

Copy link
Copy Markdown

😎 Merged successfully - details.

@tatoalo tatoalo added the Stamphog This will request an autostamp by stamphog on small changes label Jul 29, 2026
@github-actions

github-actions Bot commented Jul 29, 2026

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit 6b590de.

Comment thread packages/ui/src/features/canvas/hooks/useChannelsLayout.ts
@greptile-apps

greptile-apps Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor
Prompt To Fix All With AI
### Issue 1
packages/ui/src/features/canvas/hooks/useChannelsLayout.ts:12-15
**Development fallback contradicts comment**

Both feature-flag checks now use `import.meta.env.DEV` as their fallback, while the adjacent documentation still says there is no development default and that development matches production. This contradiction makes the gate's intended behavior harder to understand and maintain.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Reviews (1): Last reviewed commit: "chore(desktop): spaces dragging fix + de..." | Re-trigger Greptile

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Tiny, contained UI/dev-flag tweak (drag-region class fix + dev-flag fallback) with no risky territory touched; the only outstanding comment is a cosmetic doc-consistency nit, not a functional concern.

  • greptile-apps[bot] reviewed the current head.
  • Greptile inline comment notes the adjacent doc comment now contradicts the DEV fallback behavior — cosmetic, non-blocking, but worth a follow-up tweak.
Gate mechanics and policy version
Gate Result
prerequisites all clear
deny-list no deny categories matched
size 9L, 2F substantive — within ceiling
tier T1-agent / T1a-trivial (9L, 2F, single-area, chore)
stamphog 2.0.0b3 .stamphog/policy.yml @ 7d1d9d7 · reviewed head a4ae51b

@tatoalo
tatoalo enabled auto-merge (squash) July 29, 2026 09:54
@trunk-io
trunk-io Bot merged commit 6a70e24 into main Jul 29, 2026
33 checks passed
@trunk-io
trunk-io Bot deleted the chore/desktop/fix-dragging-window branch July 29, 2026 10:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Stamphog This will request an autostamp by stamphog on small changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants