New SQL tab: focus the editor (Ctrl+T, palette) and label completion kinds
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 55/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- react, tauri, typescript
調査の方向性
Start in src/features/sql/SqlEditor.tsx and reproduce focus after opening a tab, including after the password prompt closes; write a test for the focus call and check the null viewRef case described in the issue. Read src/features/sql/completion.ts and src/lib/strings.ts for completion detail labels, then inspect src/features/workspace/WorkspaceView.tsx for disconnected workspace handling. Done when the listed acceptance criteria pass, including the specified test and lint commands.
索引モデルが issue の本文から書いたものです。
説明
Context
Marked FIX-BEFORE-MERGE by the plan 4 review but left open: Ctrl+T and the palette's "New SQL tab" do not focus the new editor, so typing goes nowhere until a click. The smoke run saw it again with Ctrl+T right after a password prompt closed (not re-checked in isolation). The SQL completion detail also shows the raw kind enum (table, view).
To do
- Focus the CodeMirror editor of a newly opened SQL tab, including after a password prompt closes; re-check in isolation first and write a test for the focus call (the editor
focuscommand returns true even whenviewRefis null, so the fallback is skipped). - Show a
strings.tslabel instead of the raw kind in the completion detail (completion.ts). - Also:
Mod+T/ new tab on a disconnected workspace should go throughrequestConnect(re-check).
Acceptance criteria
- Ctrl+T and palette "New SQL tab": typing immediately goes into the editor, also right after a connect password prompt.
- Completion popup shows localized labels for table and view.
-
pnpm test && pnpm lintgreen (pluscargo clippy --all-targets -- -D warningsandpnpm test:rustwhen Rust changes)
Likely files touched
src/features/sql/SqlEditor.tsx,src/features/sql/completion.tssrc/features/workspace/WorkspaceView.tsxsrc/lib/strings.ts
Linked docs
docs/notes/plan-04-followups.md(M1, Manual #4, Task 6)docs/notes/integration-v1-smoke.md("Seen, not fixed")
- 主要言語
- TypeScript
- スター
- 0
- フォーク
- 0
- 平均マージ
- 1時間 2分
- マージ済み PR（30日）
- 41
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
Cemoche/tablemax のほかの issue
-
accessibility area:frontend size:S status:ready type:bug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
area:backend size:S status:ready type:bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
area:infra good first issue size:S status:ready type:bug
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
メンテナーはふだん 1 日以内に返信
-
area:docs good first issue size:S status:ready type:task
難易度 1/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
area:infra size:S status:ready type:task
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
メンテナーはふだん 1 日以内に返信
Cemoche/tablemax の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
db-ux-design-system/core-web#8613 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
bug:new
難易度 1/5 1時間未満 初心者へのやさしさ 78/100
callstackincubator/simlock#934 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
gchq/LD-Explorer#286 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
foxglove/mcap#1907 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信