Repository navigation
feat: move notes tools into studio-style content cards - #216
Merged
Merged
Conversation
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
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.
What does this PR do?
将最右侧笔记面板的四个创建入口从顶栏移入内容区,采用 NotebookLM Studio 风格的紧凑 2×2 工具卡片:创建笔记、思维导图、生成题目、Anki 卡片。
Why?
让入口具有可见文字,减少顶栏拥挤,并让工具和已有产物在同一内容区连续呈现。
What changed?
DESIGN.md,不复用图表色、不放宽其他区域的配色规则。How was this tested?
npm run check:design— passed.npm run typecheck— passed.npm test— 496 tests passed.npm run build— passed.react-hooks/exhaustive-depswarning in unchanged editor save logic.git diff --check— passed.NotePanelwith mocked IPC/store at 260px panel width in a short 300px window, across light/dark, English/Chinese, empty/populated states. Verified four cards, no horizontal overflow, label wrapping and shared scrolling. Verified hover retains opaque fill, visible keyboard Tab focus ring, and click dispatch for mocked create-note/mind-map callbacks.Screenshots
Actual component with mocked local items; intentionally narrow/short to show wrapping. Temporary harness/scripts are not included in this PR.
Checklist
npm run typecheckpasses.npm run buildpasses.