fix(web): center icons in thread details icon buttons - #15669
Conversation
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — This is a one-line styling adjustment that centers icons in existing thread-detail action buttons. It has no evident impact on application logic, data, APIs, defaults, or infrastructure. You can add or adjust custom eligibility rules. Learn more. |
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 8 remain after this review. 📝 WalkthroughWalkthroughThe thread details panel icon action class now includes ChangesThread details panel actions
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Bug fix Suggested reviewers: Merge Risk: ⚪ Minimal · up to This change centers existing icon actions. No concrete merge-blocking risk is established, so it is ready for normal checks. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
## What's Changed * fix(cli): reject accidental server launches by @maria-rcks in pingdotgg/t3code#15795 * feat(clients): reach one environment over several routes by @juliusmarminge in pingdotgg/t3code#15467 * feat(clients): learn an environment's LAN and tailnet addresses by @juliusmarminge in pingdotgg/t3code#15468 * fix(server): share MCP tool presentation across providers by @juliusmarminge in pingdotgg/t3code#15475 * revert(chat): remove automatic file-link repair by @maria-rcks in pingdotgg/t3code#15824 * perf(web): validate monospace fonts when selected by @maria-rcks in pingdotgg/t3code#15642 * fix(server): expand home-relative media paths by @maria-rcks in pingdotgg/t3code#15618 * fix(server): recover Linux runtime directory for device hub by @maria-rcks in pingdotgg/t3code#12402 * fix(web): center icons in thread details icon buttons by @RakshithBhat03 in pingdotgg/t3code#15669 * fix(mobile): back from an agent's thread returns to its parent by @AKolenda in pingdotgg/t3code#15068 * fix(dev): worktree setup never deletes a real env file by @juliusmarminge in pingdotgg/t3code#15845 * fix(server): drop the duplicate Option import that breaks main CI by @juliusmarminge in pingdotgg/t3code#15847 * fix(dev): write bootstrap warnings directly to stderr by @maria-rcks in pingdotgg/t3code#15865 * fix(mobile): a message that fails to send now says why in the thread by @shivamhwp in pingdotgg/t3code#15807 * fix(server): queue background notifications during active tools by @Yash-Singh1 in pingdotgg/t3code#15892 * refactor(server): share one keyed lock that releases idle keys by @juliusmarminge in pingdotgg/t3code#15577 **Full Changelog**: pingdotgg/t3code@v0.0.46-nightly.20261004.2657...v0.0.46-nightly.20261005.2667 Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.46-nightly.20261005.2667
## What's Changed * fix(cli): reject accidental server launches by @maria-rcks in pingdotgg/t3code#15795 * feat(clients): reach one environment over several routes by @juliusmarminge in pingdotgg/t3code#15467 * feat(clients): learn an environment's LAN and tailnet addresses by @juliusmarminge in pingdotgg/t3code#15468 * fix(server): share MCP tool presentation across providers by @juliusmarminge in pingdotgg/t3code#15475 * revert(chat): remove automatic file-link repair by @maria-rcks in pingdotgg/t3code#15824 * perf(web): validate monospace fonts when selected by @maria-rcks in pingdotgg/t3code#15642 * fix(server): expand home-relative media paths by @maria-rcks in pingdotgg/t3code#15618 * fix(server): recover Linux runtime directory for device hub by @maria-rcks in pingdotgg/t3code#12402 * fix(web): center icons in thread details icon buttons by @RakshithBhat03 in pingdotgg/t3code#15669 * fix(mobile): back from an agent's thread returns to its parent by @AKolenda in pingdotgg/t3code#15068 * fix(dev): worktree setup never deletes a real env file by @juliusmarminge in pingdotgg/t3code#15845 * fix(server): drop the duplicate Option import that breaks main CI by @juliusmarminge in pingdotgg/t3code#15847 * fix(dev): write bootstrap warnings directly to stderr by @maria-rcks in pingdotgg/t3code#15865 * fix(mobile): a message that fails to send now says why in the thread by @shivamhwp in pingdotgg/t3code#15807 * fix(server): queue background notifications during active tools by @Yash-Singh1 in pingdotgg/t3code#15892 * refactor(server): share one keyed lock that releases idle keys by @juliusmarminge in pingdotgg/t3code#15577 **Full Changelog**: pingdotgg/t3code@v0.0.46-nightly.20261004.2657...v0.0.46-nightly.20261005.2667 Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.46-nightly.20261005.2667
The
⋯menu next to Lineage drew its hover and open highlight with the dots pushed to the left. The square was 24px wide, but the 14px icon started at its left edge, so it sat about 4px off center.The icon-button style shared by the thread details panel never set
justify-content, so its contents aligned to the start. It now centers them. The same style is used for the Automations settings, edit, and run-now buttons, which had the same offset and are fixed by this change too.Verified with lint on the changed file and by opening the Lineage menu on a real thread in a local dev build. Did not check the Automations buttons in the browser.
Made by Claude Opus 5.5 in T3 Code through the Claude Code harness.