Skip to content

[codex] show devtools only in dev and move trigger bottom-left#43

Open
phenix3443 wants to merge 26 commits intoperfect-panel:mainfrom
phenix3443:codex/devtools-dev-only-bottom-left
Open

[codex] show devtools only in dev and move trigger bottom-left#43
phenix3443 wants to merge 26 commits intoperfect-panel:mainfrom
phenix3443:codex/devtools-dev-only-bottom-left

Conversation

@phenix3443
Copy link
Copy Markdown
Contributor

Summary

  • show TanStack Devtools only in Vite dev mode for admin and user apps
  • move the devtools trigger to the bottom-left so it no longer blocks bottom-right actions
  • document the default dual-PR workflow for origin and upstream remotes

Why

  • the default TanStack floating trigger was appearing in non-dev contexts and overlapping action buttons
  • this repo now has an explicit standing rule to open both origin and upstream PRs when submission is requested

Validation

  • code inspection only
  • unable to run workspace checks in this environment because bun is not installed

# Conflicts:
#	apps/admin/src/sections/auth/index.tsx
chore: adopt dev branch strategy
docs: deduplicate agent instructions
* fix(ci): address PR 38 review feedback

* chore(ci): use actions checkout v6
* docs: update branch strategy

* fix: update web version metadata and title sync
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 17, 2026

Deploy Preview for ppane-docs ready!

Name Link
🔨 Latest commit 6444c65
🔍 Latest deploy log https://app.netlify.com/projects/ppane-docs/deploys/69e249fe9d5086000818a2e8
😎 Deploy Preview https://deploy-preview-43--ppane-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@phenix3443 phenix3443 force-pushed the codex/devtools-dev-only-bottom-left branch from 2c7ec02 to 2879d13 Compare April 17, 2026 10:45
@web-ppanel
Copy link
Copy Markdown
Contributor

Triage review (automated): This PR looks good. CI checks pass. Changes are scoped to frontend dev tooling — limiting TanStack Devtools visibility to dev mode and repositioning the trigger button to avoid UI conflicts. Ready for maintainer merge review.

@web-ppanel
Copy link
Copy Markdown
Contributor

Thanks for this PR! The changes to show TanStack DevTools only in dev mode and move the trigger to bottom-left are a good UX improvement. The PR is ready for maintainer review and merge when the team is ready.

@web-ppanel
Copy link
Copy Markdown
Contributor

Thanks for this improvement! The changes look reasonable — restricting TanStack Devtools to dev mode only is a solid practice, and moving the trigger to bottom-left avoids UI conflicts.

This is a frontend-only change with clear rationale. Keeping open for maintainer review and merge.

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