feat: trigger button stays visible and toggles widget open/close - #109
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8e79c0aab3
ℹ️ 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".
| <XMarkIcon className="w-4 h-4 text-muted-foreground" /> | ||
| </button> | ||
| </div> | ||
| <div className="flex items-center gap-1">{user && <UserAvatarPopover user={user} />}</div> |
There was a problem hiding this comment.
trigger: false
This removes the in-widget close button, but the SDK path that skips trigger creation (init/identify with trigger: false) still exists, so some embeds will have no built-in toggle to close the panel. On desktop there is no backdrop-click close, so users can get stuck in the widget unless the host page implemented its own close control or they use Escape.
Useful? React with 👍 / 👎.
| if (isMobile) { | ||
| trigger.style.display = "flex"; |
There was a problem hiding this comment.
In hidePanel, re-showing the trigger is now conditional on isMobile. If the widget is opened on mobile (trigger hidden) and the viewport crosses to desktop before close, isMobile is false at close time, so this branch is skipped and the trigger stays hidden, preventing reopen via the floating button.
Useful? React with 👍 / 👎.
This reverts commit 5f8ccc2.
…revent scroll-to-bottom on post open
…e-sizing overflow
- Replace ScrollArea with plain overflow-y-auto div, eliminating the Radix display:table wrapper that caused horizontal overflow on the feedback list - Add toggleable board filter pills below "Popular ideas" heading; clicking an active pill deselects (returns to all boards) - Add left/right chevron nav buttons with fade gradients for pill row overflow on desktop; scrollbar hidden via new scrollbar-none utility - Add scrollbar-none @Utility to globals.css
- Move liveCommentCountSq subquery inside refreshStaleSummaries to avoid module-level db query construction at import time - Guard logStartupBanner against missing SECRET_KEY during Nitro's initial build evaluation pass - Switch from vite-tsconfig-paths plugin to native resolve.tsconfigPaths; add routeFileIgnorePattern to exclude __tests__ from route discovery
- Replace inline webkit scrollbar variants with scrollbar-thin utility - Guard usePillsScroll setState with functional updater to skip no-op re-renders during smooth scroll - Remove redundant JSX section comments
…mentation - Panel offset 80px → 88px (larger trigger button) - Icon swap now uses opacity/transform transitions on separate iconChat/iconClose elements rather than swapping trigger.innerHTML directly
Search is now hidden by default — a magnifying glass icon sits inline with the "Popular ideas" heading and expands to a full input on click, removing a permanently visible input row from the widget.
1 participant
Summary
Closes #107
Test plan
placement: "left") works correctly