feat(header): add account access - #166
Merged
huyanxius merged 5 commits intoAug 7, 2026
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![fennoai[bot]](https://avatars.githubusercontent.com/in/1924301?s=60&v=4){kind=small}
Route-level tests now render an app shell that consumes the authentication context. A shared guest wrapper supplies the same provider boundary used by production composition. Projects and playtest coverage remains focused on page behavior without duplicating auth setup.
The account dialog had no discoverable entry in the product shell. The header now reflects session state, preserves safe return paths, and exposes local logout. Responsive sizing keeps navigation and account controls usable without horizontal overflow.
Header behavior now depends on bootstrapped guest and authenticated session states. The tests exercise safe login return paths, identity display, logout, and route navigation. A real session provider keeps the assertions aligned with production composition.
The rebased header now consumes the shared auth session in every AppShell render. Add authenticated and guest test wrappers and isolate the closed account panel test from AppShell. Restore the rebased frontend suite without changing production behavior.
huyanxius
force-pushed
the
feat/164-header-account-entry-submit
branch
from
August 7, 2026 08:49
ec2e64d to
713d05a
Compare
Contributor
UI页面 ，登录右上角不太好看。是否可以优化一下？ |
Contributor
|
@fennoai 讲讲你对这个UI组件美化的做法。 |
Contributor
这版 UI 美化的做法核心不是加装饰，而是把信息层级和交互密度收紧了：
整体风格偏“克制的产品工具栏”：少装饰、强层级、明确状态、低噪音。 |
Contributor
|
@fennoai 如果是你，你要如何更改呢？ |
Contributor
如果是我我会做三处收敛：
如果继续往下调，我会优先做的是“减法”而不是继续加视觉元素：保持现在的浅底、细边框、低噪音状态，再把文本密度和分支再压一层。 |
Collaborator
Author
@fennoai 可以，请你直接修改，给我们提供修改后渲染的截图，没有经过同意前不要更改此 PR 的代码 |
Contributor
先说明边界我不会在未获你明确同意前修改这份 PR 的代码，也就不能先给出“修改后渲染截图”。 如果你确认让我改，我会只做这三处收敛：
你确认后，我再直接改代码并补对应截图。 |
Collaborator
Author
@fennoai 好 |
Contributor
已修改按你的确认，我只做了这三处收敛，没有碰其他页面逻辑：
已验证：
|
xiaocheny214
self-requested a review
August 7, 2026 09:16
xyh202131
approved these changes
Aug 7, 2026
Collaborator
Author
|
@xiaocheny214 好了，你看一眼 |
The account entry felt visually heavier than the surrounding navigation. Use a quiet outlined surface and group authenticated controls into one compact unit. The header now keeps account actions clear without competing with page content.
Collaborator
Author
|
@xiaocheny214 UI 问题等我们都做完后统一改吧 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Feature Description
Implementation Approach
logout()，不读取 token、DTO 或持久化实现。Screenshots
以下截图统一为 1280 × 720 横屏完整页面。
访客态
登录态
Testing
git diff --check upstream/main...HEAD：通过。npm run format:check、npm run lint、npm run typecheck、npm run build：通过。typecheck。Scope Boundary
/account、资料查看和修改密码仍归 feat: 实现账号中心（Refs #157） #161。Closes #164
Related