Skip to content

Frontend improvments - #40

Merged
betterclever merged 2 commits into
mainfrom
betterclever/fix-frontend
Oct 27, 2025
Merged

betterclever merged 2 commits into
mainfrom
betterclever/fix-frontend

Conversation

@betterclever

@betterclever betterclever commented Oct 26, 2025 •

Copy link
Copy Markdown
Contributor

Summary

  • The frontend was basic without clear app entrypoints. Reworked the navigation and layout to a more professional setup.

Testing

  • bun run test
  • bun run lint
  • bun run typecheck
  • Additional notes:

Documentation

  • Updated the relevant doc(s) (see docs/guide.md) or checked that no updates are needed.
  • Recorded contract/architecture changes in both public docs and .ai logs when applicable.

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +32 to +36
// Auto-collapse sidebar when opening workflow builder, expand for other routes
useEffect(() => {
const isWorkflowRoute = location.pathname.startsWith('/workflows') && location.pathname !== '/'
setSidebarOpen(!isWorkflowRoute)
}, [location.pathname])

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Restore navigation access on mobile workflow builder

The effect collapses the global sidebar on every /workflows route, and the AppTopBar (which hosts the toggle) is rendered only for non-workflow pages. On small screens this means the sidebar width becomes w-0 and there is no remaining UI element to reopen it, so users cannot navigate back to the workflow list or secrets once inside the builder. Consider leaving a visible toggle or avoiding auto-collapse on narrow viewports to retain basic navigation.

Useful? React with 👍 / 👎.

@betterclever
betterclever merged commit 661c850 into main Oct 27, 2025
1 check passed
@LuD1161
LuD1161 deleted the betterclever/fix-frontend branch December 19, 2025 13:20
betterclever added a commit that referenced this pull request Mar 6, 2026
fix: simplify node-pty postinstall repair
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