Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

feat(frontend): move current user info into secondary sidebar#507

Merged
hmans merged 3 commits into
mainchattocorp/chatto:mainfrom
hmans/user-info-secondary-sidebarchattocorp/chatto:hmans/user-info-secondary-sidebarCopy head branch name to clipboard
May 19, 2026
Merged

feat(frontend): move current user info into secondary sidebar#507
hmans merged 3 commits into
mainchattocorp/chatto:mainfrom
hmans/user-info-secondary-sidebarchattocorp/chatto:hmans/user-info-secondary-sidebarCopy head branch name to clipboard

Conversation

@hmans

@hmans hmans commented May 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Move the current user info (avatar + display name + @login + settings cog) out of the cross-server SpaceList and into a new CurrentUserBar rendered at the bottom of the per-server SecondarySidebar, since identity is server-scoped.
  • Pin the Add Server "+" button to the bottom of the SpaceList in its own bordered row, so its separator and footprint line up horizontally with the new user bar.
  • Add cursor-pointer to the Add Server button (the space-list-item utility doesn't include it under Tailwind v4).

Test plan

  • mise test-frontend passes (svelte-check 0 errors, 755 unit tests green locally).
  • Visually verify on https://memphis-v1.orb.local/ that the two bottom separator lines align across the primary and secondary sidebars, and that the user bar shows both display name and @login.
  • Clicking the user bar navigates to /chat/[serverId]/settings; clicking "+" still opens the Add Server dialog.

hmans added 3 commits May 19, 2026 16:51
Identity is server-scoped, so the current user belongs next to the
server's room list rather than at the bottom of the cross-server
SpaceList.

- Add CurrentUserBar (avatar + display name + settings cog) inside
  SecondarySidebar, visible across both the chrome routes and the
  per-server settings layout
- Remove the avatar block from the bottom of SpaceList
…dary-sidebar

* origin/main:
  chore(dev): replace docker compose with mise dev for Conductor (#505)
- Pin the Add Server button to a bottom row with its own `border-t`,
  so its separator aligns horizontally with the current-user bar's.
- Make both bottom cells the same 64px height and add the
  `cursor-pointer` that `space-list-item` doesn't provide on its own.
- Show `@login` under the display name in the current-user bar
  (suppressed when it would duplicate the display name).
@hmans
hmans merged commit de6b298 into main May 19, 2026
4 checks passed
@hmans
hmans deleted the hmans/user-info-secondary-sidebar branch May 19, 2026 15:24
hmans added a commit that referenced this pull request May 19, 2026
…y-research

* origin/main:
  0.0.178
  fix(frontend): restore per-server last-room memory on server re-entry (#511)
  style(admin): give panels a tinted header band, ridge, and lift (#510)
  fix(frontend): use base font size for username change notice (#508)
  feat(frontend): add server overview entries to Cmd-K quick switcher (#506)
  feat(frontend): move current user info into secondary sidebar (#507)
  refactor(core): store verified emails as per-email proto entries (#504)

# Conflicts:
#	docs/fdr/INDEX.md
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.