New SQL tab: focus the editor (Ctrl+T, palette) and label completion kinds
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 55/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- react, tauri, typescript
Hướng nghiên cứu
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.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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")
- Ngôn ngữ chính
- TypeScript
- Star
- 0
- Fork
- 0
- Merge trung bình
- 1 giờ 19 phút
- Pull request đã merge (30 ngày)
- 28
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của Cemoche/tablemax
-
area:frontend size:S status:ready type:bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 83/100
Maintainer thường phản hồi trong vòng 1 ngày
-
area:backend size:S status:ready type:bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
Maintainer thường phản hồi trong vòng 1 ngày
-
area:infra good first issue size:S status:ready type:bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 83/100
Maintainer thường phản hồi trong vòng 1 ngày
-
area:docs good first issue size:S status:ready type:task
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
area:infra size:S status:ready type:task
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của Cemoche/tablemax
Issue tương tự
-
skillopt: success-reflect prompt omits the edit schema, so every success-only edit is droppedĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Workspace secrets/constants containing "$" are corrupted when resolved in data source optionsĐang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
mobile-next/mobile-mcp#526 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
e2e-failure ready-to-code
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 73/100
redhat-developer/rhdh-plugin-export-overlays#4470 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày