Skip to content

fix(web): center project monograms in settled rows - #16841

Merged
Yash-Singh1 merged 1 commit into
pingdotgg:mainfrom
Aforno:t3/fix-project-icon-vertical-alignment
Oct 7, 2026
Merged

Yash-Singh1 merged 1 commit into
pingdotgg:mainfrom
Aforno:t3/fix-project-icon-vertical-alignment

Conversation

@Aforno

@Aforno Aforno commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Projects without a favicon get a fallback monogram that sits 2 CSS pixels above the vertical center of a settled sidebar row. The plain span around it creates an inline line box with baseline space below the icon.

Make that existing wrapper a flex container with centered items. The monogram now shares the row's vertical center. This also applies to snoozed slim rows and the desktop app's shared web sidebar. Mobile already centers its monograms in a native View.

This is a small, focused fix for an obvious visual bug under CONTRIBUTING.md's exception. Searched existing issues and PRs for icon alignment, monogram, favicon, settled icons, and vertical centering. The related merged #11806 fixes horizontal alignment in menus, not this settled-row baseline offset.

Verified in T3's Browser panel against an isolated worktree dev database: the same 36px settled row's 16px fallback monogram changed from a -2px center offset to 0px. A browser layout probe using the same wrapper also measured 0px for image, emoji, and Lucide icon markup. No native mobile or packaged Electron run was performed.

Checks: web typecheck passed; targeted Sidebar.tsx lint passed with existing warnings; targeted formatting and git diff --check passed. No test added for this CSS-only change; browser geometry verifies the observable defect.

Before After
Before: fallback monogram is 2px above center After: fallback monogram is centered

Built with GPT-6.1-Sol through the Codex harness in T3 Code.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Oct 7, 2026
@macroscopeapp

macroscopeapp Bot commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 9b2eb0b

Macroscope's review found this PR approvable — This is a one-line, localized CSS fix that vertically centers existing project icons in settled sidebar rows. It changes only presentation and has no apparent impact on application logic, data, APIs, or production configuration.

You can add or adjust custom eligibility rules. Learn more.

@coderabbitai

coderabbitai Bot commented Oct 7, 2026

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Path: .coderabbit.config.ts
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 8355c910-1e9c-49e2-8e1a-37789be2e06b
📥 Commits

Reviewing files that changed from the base of the PR and between 611132c and 9b2eb0b.

📒 Files selected for processing (1)
  • apps/web/src/components/Sidebar.tsx

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 8 remain after this review.


📝 Walkthrough

Walkthrough

The slim sidebar thread row adds the flex class to the project favicon wrapper. Its existing shrink and transition classes remain.

Changes

Slim sidebar layout

Layer / File(s) Summary
Project favicon layout
apps/web/src/components/Sidebar.tsx
The slim-row project favicon wrapper adds the flex class. Its existing shrink and transition classes remain.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Bug fix

Suggested reviewers: juliusmarminge

Merge Risk: ⚪ Minimal · up to 9b2eb

The slim-row project monogram wrapper gains flex layout, with its existing styling preserved. No concrete regression is indicated, so the change is ready to merge after normal checks.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the primary change: centering project monograms in settled web sidebar rows.
Description check ✅ Passed The description covers the problem, implementation, scope rationale, verification steps, observed results, limitations, UI screenshots, and agent attribution. Its content maps to all required template…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Approvability ✅ Passed The pull request changes only apps/web/src/components/Sidebar.tsx. It adds flex items-center to the existing favicon wrapper in the slim sidebar row. This is a focused layout bug fix. It does not …
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@Yash-Singh1
Yash-Singh1 merged commit 46dd094 into pingdotgg:main Oct 7, 2026
30 checks passed
github-actions Bot added a commit to omarcresp/t3code-flake that referenced this pull request Oct 8, 2026
## What's Changed
* feat(models): add Claude Haiku 5.5 and retire Sonnet 5 and Opus 5 to legacy by @juliusmarminge in pingdotgg/t3code#16903
* fix(web): iPhone Duo folds animate, center on the hinge, and keep the phone's orientation by @gabrielelpidio in pingdotgg/t3code#16885
* fix(server): Claude 5-series models always run 1M context by @juliusmarminge in pingdotgg/t3code#16908
* fix(desktop): setup prompts name a t3 that runs on desktop installs by @juliusmarminge in pingdotgg/t3code#16676
* feat(desktop): install the t3 command from Settings by @juliusmarminge in pingdotgg/t3code#16683
* fix(web): show live names in thread-read activity by @Bil0000 in pingdotgg/t3code#13140
* feat(mobile): adopt v5 navigation and native iPad columns by @juliusmarminge in pingdotgg/t3code#16733
* fix(mobile): Android composer picker scrolls past the first four rows by @shivamhwp in pingdotgg/t3code#15856
* fix(desktop): sign-in and captchas work again in desktop browser tabs by @juliusmarminge in pingdotgg/t3code#16939
* fix(server): missing project folders no longer log favicon warnings by @yordis in pingdotgg/t3code#16757
* fix(server): unload Codex threads left idle on the shared app-server by @RhysSullivan in pingdotgg/t3code#16917
* fix(web): fast typing no longer scrambles text when type-to-focus kicks in by @otavio in pingdotgg/t3code#14595
* fix(web): simplify workspace card rows by @Bil0000 in pingdotgg/t3code#16823
* fix(web): Copy MCP URL shows up for environments reached over plain http by @SunkenInTime in pingdotgg/t3code#16909
* fix(web): C#, Java, PHP and 11 other languages get file icons by @juliusmarminge in pingdotgg/t3code#16974
* feat(clients): live row shows the agent's latest thought by @t3dotgg in pingdotgg/t3code#16284
* feat(web): block-level Markdown in the rich text composer by @chrisdeeming in pingdotgg/t3code#14677
* fix(web): center project monograms in settled rows by @Aforno in pingdotgg/t3code#16841
* fix(web): cancelling a new citation no longer leaves a stray space by @Aforno in pingdotgg/t3code#16828
* fix(settings): provider updates show live progress instead of a bare spinner by @shivamhwp in pingdotgg/t3code#16958
* feat(web): find in diffs with Cmd+F by @juliusmarminge in pingdotgg/t3code#14623
* refactor(server): GitHub services are named for the API they call, not gh by @juliusmarminge in pingdotgg/t3code#16967
* refactor(server): GitHub GraphQL batches use variables and share one pager by @juliusmarminge in pingdotgg/t3code#16960
* refactor(server): GitHub source control reads GitHubApi directly by @juliusmarminge in pingdotgg/t3code#16982
* refactor(server): GitHub rate limits read the response headers by @juliusmarminge in pingdotgg/t3code#16986

## New Contributors
* @RhysSullivan made their first contribution in pingdotgg/t3code#16917
* @Aforno made their first contribution in pingdotgg/t3code#16841

**Full Changelog**: pingdotgg/t3code@v0.0.46-nightly.20261007.2787...v0.0.46-nightly.20261008.2801

Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.46-nightly.20261008.2801
github-actions Bot added a commit to davidvanderklay/t3code-flake that referenced this pull request Oct 8, 2026
## What's Changed
* feat(models): add Claude Haiku 5.5 and retire Sonnet 5 and Opus 5 to legacy by @juliusmarminge in pingdotgg/t3code#16903
* fix(web): iPhone Duo folds animate, center on the hinge, and keep the phone's orientation by @gabrielelpidio in pingdotgg/t3code#16885
* fix(server): Claude 5-series models always run 1M context by @juliusmarminge in pingdotgg/t3code#16908
* fix(desktop): setup prompts name a t3 that runs on desktop installs by @juliusmarminge in pingdotgg/t3code#16676
* feat(desktop): install the t3 command from Settings by @juliusmarminge in pingdotgg/t3code#16683
* fix(web): show live names in thread-read activity by @Bil0000 in pingdotgg/t3code#13140
* feat(mobile): adopt v5 navigation and native iPad columns by @juliusmarminge in pingdotgg/t3code#16733
* fix(mobile): Android composer picker scrolls past the first four rows by @shivamhwp in pingdotgg/t3code#15856
* fix(desktop): sign-in and captchas work again in desktop browser tabs by @juliusmarminge in pingdotgg/t3code#16939
* fix(server): missing project folders no longer log favicon warnings by @yordis in pingdotgg/t3code#16757
* fix(server): unload Codex threads left idle on the shared app-server by @RhysSullivan in pingdotgg/t3code#16917
* fix(web): fast typing no longer scrambles text when type-to-focus kicks in by @otavio in pingdotgg/t3code#14595
* fix(web): simplify workspace card rows by @Bil0000 in pingdotgg/t3code#16823
* fix(web): Copy MCP URL shows up for environments reached over plain http by @SunkenInTime in pingdotgg/t3code#16909
* fix(web): C#, Java, PHP and 11 other languages get file icons by @juliusmarminge in pingdotgg/t3code#16974
* feat(clients): live row shows the agent's latest thought by @t3dotgg in pingdotgg/t3code#16284
* feat(web): block-level Markdown in the rich text composer by @chrisdeeming in pingdotgg/t3code#14677
* fix(web): center project monograms in settled rows by @Aforno in pingdotgg/t3code#16841
* fix(web): cancelling a new citation no longer leaves a stray space by @Aforno in pingdotgg/t3code#16828
* fix(settings): provider updates show live progress instead of a bare spinner by @shivamhwp in pingdotgg/t3code#16958
* feat(web): find in diffs with Cmd+F by @juliusmarminge in pingdotgg/t3code#14623
* refactor(server): GitHub services are named for the API they call, not gh by @juliusmarminge in pingdotgg/t3code#16967
* refactor(server): GitHub GraphQL batches use variables and share one pager by @juliusmarminge in pingdotgg/t3code#16960
* refactor(server): GitHub source control reads GitHubApi directly by @juliusmarminge in pingdotgg/t3code#16982
* refactor(server): GitHub rate limits read the response headers by @juliusmarminge in pingdotgg/t3code#16986

## New Contributors
* @RhysSullivan made their first contribution in pingdotgg/t3code#16917
* @Aforno made their first contribution in pingdotgg/t3code#16841

**Full Changelog**: pingdotgg/t3code@v0.0.46-nightly.20261007.2787...v0.0.46-nightly.20261008.2801

Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.46-nightly.20261008.2801
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS 0-9 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants