feat(office): table detail panes and Alt+1/2/3 layout toggles
Add spreadsheet-style list, split detail pane with forms and document preview, and Alt+1/2/3 to show/hide panes with even width distribution. Co-authored-by: Cursor <cursoragent@cursor.com>
This commit is contained in:
Cursor
+12
-12
@@ -5,6 +5,7 @@ type ActionID string
|
||||
|
||||
const (
|
||||
ActionView ActionID = "view"
|
||||
ActionSave ActionID = "save"
|
||||
ActionDelete ActionID = "delete"
|
||||
ActionRefresh ActionID = "refresh"
|
||||
ActionDownload ActionID = "download"
|
||||
@@ -17,48 +18,47 @@ type ItemAction struct {
|
||||
Danger bool
|
||||
}
|
||||
|
||||
// ActionsFor returns available operations for an item kind.
|
||||
// ActionsFor returns available operations for an item kind (detail pane action bar).
|
||||
func ActionsFor(kind Kind) []ItemAction {
|
||||
switch kind {
|
||||
case KindProject:
|
||||
return []ItemAction{
|
||||
{ID: ActionView, Label: "View details"},
|
||||
{ID: ActionDelete, Label: "Delete project", Danger: true},
|
||||
{ID: ActionSave, Label: "Save"},
|
||||
{ID: ActionDelete, Label: "Delete", Danger: true},
|
||||
}
|
||||
case KindTask, KindCRMTask:
|
||||
case KindTask:
|
||||
return []ItemAction{
|
||||
{ID: ActionView, Label: "View details"},
|
||||
{ID: ActionSave, Label: "Save"},
|
||||
{ID: ActionDelete, Label: "Delete task", Danger: true},
|
||||
}
|
||||
case KindCRMTask:
|
||||
return []ItemAction{
|
||||
{ID: ActionDelete, Label: "Delete", Danger: true},
|
||||
}
|
||||
case KindContact:
|
||||
return []ItemAction{
|
||||
{ID: ActionView, Label: "View details"},
|
||||
{ID: ActionDelete, Label: "Delete contact", Danger: true},
|
||||
}
|
||||
case KindOpportunity:
|
||||
return []ItemAction{
|
||||
{ID: ActionView, Label: "View details"},
|
||||
{ID: ActionDelete, Label: "Delete deal", Danger: true},
|
||||
}
|
||||
case KindCase:
|
||||
return []ItemAction{
|
||||
{ID: ActionView, Label: "View details"},
|
||||
{ID: ActionDelete, Label: "Delete case", Danger: true},
|
||||
}
|
||||
case KindMail:
|
||||
return []ItemAction{
|
||||
{ID: ActionView, Label: "Read message"},
|
||||
{ID: ActionDelete, Label: "Delete message", Danger: true},
|
||||
}
|
||||
case KindFile:
|
||||
return []ItemAction{
|
||||
{ID: ActionView, Label: "Preview file"},
|
||||
{ID: ActionDownload, Label: "Download"},
|
||||
{ID: ActionDelete, Label: "Delete file", Danger: true},
|
||||
}
|
||||
case KindEvent, KindCalendar, KindUser:
|
||||
return []ItemAction{{ID: ActionView, Label: "View details"}}
|
||||
return nil
|
||||
default:
|
||||
return []ItemAction{{ID: ActionView, Label: "View details"}}
|
||||
return nil
|
||||
}
|
||||
}
|
||||
|
||||
@@ -0,0 +1,287 @@
|
||||
package model
|
||||
|
||||
import (
|
||||
"fmt"
|
||||
"sort"
|
||||
"strconv"
|
||||
"strings"
|
||||
)
|
||||
|
||||
// TableColumn is one column in the center data table.
|
||||
type TableColumn struct {
|
||||
Key string
|
||||
Title string
|
||||
Width int
|
||||
}
|
||||
|
||||
var subjectExtraKeys = map[Subject][]string{
|
||||
SubjectTasks: {"status", "deadline", "responsible"},
|
||||
SubjectCalendars: {"description"},
|
||||
SubjectEvents: {"start", "end"},
|
||||
SubjectContacts: {"primaryEmail", "displayName"},
|
||||
SubjectPersons: {"primaryEmail", "displayName"},
|
||||
SubjectCompanies: {"primaryEmail", "displayName"},
|
||||
SubjectOpportunities: {"stage", "bidValue", "bidCurrency"},
|
||||
SubjectCases: {"status"},
|
||||
SubjectCRMTasks: {"status", "deadline"},
|
||||
SubjectMailInbox: {"from", "date"},
|
||||
SubjectMailSent: {"to", "date"},
|
||||
SubjectMailDrafts: {"to", "date"},
|
||||
SubjectMailTrash: {"from", "date"},
|
||||
SubjectMailSpam: {"from", "date"},
|
||||
SubjectUsers: {"email", "displayName"},
|
||||
}
|
||||
|
||||
// BuildColumns derives table columns from the list subject and item payloads.
|
||||
func BuildColumns(subject Subject, items []Item) []TableColumn {
|
||||
if subject == SubjectProjects {
|
||||
return buildProjectColumns(items)
|
||||
}
|
||||
cols := []TableColumn{
|
||||
{Key: "_sel", Title: "✓", Width: 3},
|
||||
{Key: "id", Title: "ID", Width: 10},
|
||||
{Key: "title", Title: "Title", Width: 28},
|
||||
}
|
||||
if hasSubtitle(items) {
|
||||
cols = append(cols, TableColumn{Key: "subtitle", Title: "Subtitle", Width: 22})
|
||||
}
|
||||
|
||||
seen := map[string]bool{"_sel": true, "id": true, "title": true, "subtitle": true}
|
||||
for _, key := range subjectExtraKeys[subject] {
|
||||
if seen[key] || !columnHasData(items, key) {
|
||||
continue
|
||||
}
|
||||
seen[key] = true
|
||||
cols = append(cols, TableColumn{Key: key, Title: titleLabel(key), Width: defaultWidth(key)})
|
||||
}
|
||||
|
||||
discovered := discoverRawKeys(items, seen, 8)
|
||||
for _, key := range discovered {
|
||||
cols = append(cols, TableColumn{Key: key, Title: titleLabel(key), Width: defaultWidth(key)})
|
||||
}
|
||||
|
||||
sizeColumns(cols, items)
|
||||
return cols
|
||||
}
|
||||
|
||||
func buildProjectColumns(items []Item) []TableColumn {
|
||||
cols := []TableColumn{
|
||||
{Key: "_sel", Title: "✓", Width: 3},
|
||||
{Key: "id", Title: "ID", Width: 8},
|
||||
{Key: "title", Title: "Title", Width: 28},
|
||||
{Key: "tasks", Title: "Tasks (open/closed)", Width: 18},
|
||||
{Key: "documents", Title: "Documents", Width: 11},
|
||||
{Key: "users", Title: "Users", Width: 8},
|
||||
}
|
||||
sizeColumns(cols, items)
|
||||
return cols
|
||||
}
|
||||
|
||||
// CellText returns the display string for one table cell.
|
||||
func CellText(it Item, key string) string {
|
||||
switch key {
|
||||
case "_sel":
|
||||
if it.Selected {
|
||||
return "●"
|
||||
}
|
||||
return "○"
|
||||
case "id":
|
||||
return it.ID
|
||||
case "title":
|
||||
return it.Title
|
||||
case "subtitle":
|
||||
return it.Subtitle
|
||||
case "tasks":
|
||||
return formatProjectTasks(it.Raw)
|
||||
case "documents":
|
||||
return intRaw(it.Raw, "documentsCount")
|
||||
case "users":
|
||||
return intRaw(it.Raw, "participantCount")
|
||||
case "kind":
|
||||
return string(it.Kind)
|
||||
default:
|
||||
if it.Raw == nil {
|
||||
return ""
|
||||
}
|
||||
return formatAny(it.Raw[key])
|
||||
}
|
||||
}
|
||||
|
||||
func formatProjectTasks(raw map[string]any) string {
|
||||
if raw == nil {
|
||||
return "0/0"
|
||||
}
|
||||
open := intRawVal(raw, "taskCount")
|
||||
total := intRawVal(raw, "taskCountTotal")
|
||||
closed := total - open
|
||||
if closed < 0 {
|
||||
closed = 0
|
||||
}
|
||||
return fmt.Sprintf("%d/%d", open, closed)
|
||||
}
|
||||
|
||||
func intRaw(raw map[string]any, key string) string {
|
||||
if raw == nil {
|
||||
return ""
|
||||
}
|
||||
v := intRawVal(raw, key)
|
||||
if v == 0 {
|
||||
if _, ok := raw[key]; !ok {
|
||||
return ""
|
||||
}
|
||||
}
|
||||
return fmt.Sprintf("%d", v)
|
||||
}
|
||||
|
||||
func intRawVal(raw map[string]any, key string) int {
|
||||
if raw == nil {
|
||||
return 0
|
||||
}
|
||||
switch v := raw[key].(type) {
|
||||
case int:
|
||||
return v
|
||||
case int64:
|
||||
return int(v)
|
||||
case float64:
|
||||
return int(v)
|
||||
case string:
|
||||
n, _ := strconv.Atoi(v)
|
||||
return n
|
||||
default:
|
||||
if raw[key] == nil {
|
||||
return 0
|
||||
}
|
||||
n, _ := strconv.Atoi(fmt.Sprint(raw[key]))
|
||||
return n
|
||||
}
|
||||
}
|
||||
|
||||
func hasSubtitle(items []Item) bool {
|
||||
for _, it := range items {
|
||||
if it.Subtitle != "" {
|
||||
return true
|
||||
}
|
||||
}
|
||||
return false
|
||||
}
|
||||
|
||||
func columnHasData(items []Item, key string) bool {
|
||||
for _, it := range items {
|
||||
if CellText(it, key) != "" {
|
||||
return true
|
||||
}
|
||||
}
|
||||
return false
|
||||
}
|
||||
|
||||
func discoverRawKeys(items []Item, seen map[string]bool, limit int) []string {
|
||||
counts := map[string]int{}
|
||||
for _, it := range items {
|
||||
if it.Raw == nil {
|
||||
continue
|
||||
}
|
||||
for k := range it.Raw {
|
||||
if seen[k] || k == "id" || k == "title" {
|
||||
continue
|
||||
}
|
||||
counts[k]++
|
||||
}
|
||||
}
|
||||
type kv struct {
|
||||
k string
|
||||
n int
|
||||
}
|
||||
var ranked []kv
|
||||
for k, n := range counts {
|
||||
ranked = append(ranked, kv{k, n})
|
||||
}
|
||||
sort.Slice(ranked, func(i, j int) bool {
|
||||
if ranked[i].n == ranked[j].n {
|
||||
return ranked[i].k < ranked[j].k
|
||||
}
|
||||
return ranked[i].n > ranked[j].n
|
||||
})
|
||||
out := make([]string, 0, limit)
|
||||
for _, r := range ranked {
|
||||
if len(out) >= limit {
|
||||
break
|
||||
}
|
||||
out = append(out, r.k)
|
||||
}
|
||||
return out
|
||||
}
|
||||
|
||||
func titleLabel(key string) string {
|
||||
if key == "" {
|
||||
return ""
|
||||
}
|
||||
parts := strings.Split(key, "_")
|
||||
for i, p := range parts {
|
||||
if p == "" {
|
||||
continue
|
||||
}
|
||||
parts[i] = strings.ToUpper(p[:1]) + p[1:]
|
||||
}
|
||||
return strings.Join(parts, " ")
|
||||
}
|
||||
|
||||
func defaultWidth(key string) int {
|
||||
switch key {
|
||||
case "id", "status", "stage":
|
||||
return 10
|
||||
case "start", "end", "deadline", "date":
|
||||
return 18
|
||||
case "from", "to", "email", "primaryEmail":
|
||||
return 22
|
||||
case "description", "displayName":
|
||||
return 24
|
||||
default:
|
||||
return 14
|
||||
}
|
||||
}
|
||||
|
||||
func sizeColumns(cols []TableColumn, items []Item) {
|
||||
for i := range cols {
|
||||
maxW := runeLen(cols[i].Title)
|
||||
for _, it := range items {
|
||||
n := runeLen(CellText(it, cols[i].Key))
|
||||
if n > maxW {
|
||||
maxW = n
|
||||
}
|
||||
}
|
||||
maxW += 2
|
||||
if maxW < 4 {
|
||||
maxW = 4
|
||||
}
|
||||
if maxW > 36 {
|
||||
maxW = 36
|
||||
}
|
||||
cols[i].Width = maxW
|
||||
}
|
||||
}
|
||||
|
||||
func runeLen(s string) int {
|
||||
return len([]rune(s))
|
||||
}
|
||||
|
||||
func formatAny(v any) string {
|
||||
if v == nil {
|
||||
return ""
|
||||
}
|
||||
switch x := v.(type) {
|
||||
case string:
|
||||
return x
|
||||
case bool:
|
||||
if x {
|
||||
return "yes"
|
||||
}
|
||||
return "no"
|
||||
case float64:
|
||||
if x == float64(int64(x)) {
|
||||
return fmt.Sprintf("%d", int64(x))
|
||||
}
|
||||
return fmt.Sprintf("%g", x)
|
||||
default:
|
||||
return fmt.Sprint(v)
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,55 @@
|
||||
package model
|
||||
|
||||
import "testing"
|
||||
|
||||
func TestBuildProjectColumns(t *testing.T) {
|
||||
items := []Item{{
|
||||
ID: "1", Title: "Alpha", Kind: KindProject,
|
||||
Raw: map[string]any{
|
||||
"taskCount": 3, "taskCountTotal": 5,
|
||||
"documentsCount": 2, "participantCount": 4,
|
||||
},
|
||||
}}
|
||||
cols := BuildColumns(SubjectProjects, items)
|
||||
want := []string{"_sel", "id", "title", "tasks", "documents", "users"}
|
||||
if len(cols) != len(want) {
|
||||
t.Fatalf("got %d columns, want %d", len(cols), len(want))
|
||||
}
|
||||
for i, key := range want {
|
||||
if cols[i].Key != key {
|
||||
t.Fatalf("col[%d]=%q want %q", i, cols[i].Key, key)
|
||||
}
|
||||
}
|
||||
for _, c := range cols {
|
||||
if c.Key == "status" || c.Key == "subtitle" {
|
||||
t.Fatalf("unexpected column %q", c.Key)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
func TestFormatProjectTasksOpenClosed(t *testing.T) {
|
||||
raw := map[string]any{"taskCount": 3, "taskCountTotal": 8}
|
||||
if got := formatProjectTasks(raw); got != "3/5" {
|
||||
t.Fatalf("got %q want 3/5", got)
|
||||
}
|
||||
}
|
||||
|
||||
func TestCellTextProjectDocumentsUsers(t *testing.T) {
|
||||
it := Item{Raw: map[string]any{"documentsCount": 7, "participantCount": 2}}
|
||||
if got := CellText(it, "documents"); got != "7" {
|
||||
t.Fatalf("documents: %q", got)
|
||||
}
|
||||
if got := CellText(it, "users"); got != "2" {
|
||||
t.Fatalf("users: %q", got)
|
||||
}
|
||||
}
|
||||
|
||||
func TestBuildColumnsIncludesSelectionAndCoreFields(t *testing.T) {
|
||||
items := []Item{
|
||||
{ID: "1", Title: "Alpha", Kind: KindTask, Raw: map[string]any{"status": "Open"}},
|
||||
}
|
||||
cols := BuildColumns(SubjectTasks, items)
|
||||
if cols[0].Key != "_sel" {
|
||||
t.Fatalf("first column should be selection, got %q", cols[0].Key)
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,115 @@
|
||||
package model
|
||||
|
||||
import "fmt"
|
||||
|
||||
// FormFields holds editable/display fields loaded from API detail.
|
||||
type FormFields struct {
|
||||
PrimaryLabel string
|
||||
SecondaryLabel string
|
||||
Primary string
|
||||
Secondary string
|
||||
ReadOnly bool
|
||||
}
|
||||
|
||||
// KindHeading returns a short label for the detail pane header.
|
||||
func KindHeading(kind Kind, id string) string {
|
||||
return fmt.Sprintf("%s %s", kindLabel(kind), id)
|
||||
}
|
||||
|
||||
func kindLabel(kind Kind) string {
|
||||
switch kind {
|
||||
case KindProject:
|
||||
return "Project"
|
||||
case KindTask:
|
||||
return "Task"
|
||||
case KindCRMTask:
|
||||
return "CRM task"
|
||||
case KindContact:
|
||||
return "Contact"
|
||||
case KindOpportunity:
|
||||
return "Deal"
|
||||
case KindCase:
|
||||
return "Case"
|
||||
case KindMail:
|
||||
return "Mail"
|
||||
case KindEvent:
|
||||
return "Event"
|
||||
case KindCalendar:
|
||||
return "Calendar"
|
||||
case KindFile:
|
||||
return "Document"
|
||||
case KindUser:
|
||||
return "User"
|
||||
default:
|
||||
return string(kind)
|
||||
}
|
||||
}
|
||||
|
||||
// FormFieldsFromRaw maps API detail JSON to form fields for the right pane.
|
||||
func FormFieldsFromRaw(kind Kind, raw map[string]any) FormFields {
|
||||
switch kind {
|
||||
case KindMail:
|
||||
body := strRaw(raw, "body")
|
||||
if body == "" {
|
||||
body = strRaw(raw, "htmlBody")
|
||||
}
|
||||
return FormFields{
|
||||
PrimaryLabel: "Subject", SecondaryLabel: "Body",
|
||||
Primary: strRaw(raw, "subject"), Secondary: body,
|
||||
ReadOnly: true,
|
||||
}
|
||||
case KindUser:
|
||||
return FormFields{
|
||||
PrimaryLabel: "Name", SecondaryLabel: "Email",
|
||||
Primary: strRaw(raw, "displayName"), Secondary: strRaw(raw, "email"),
|
||||
ReadOnly: true,
|
||||
}
|
||||
case KindContact:
|
||||
name := strRaw(raw, "displayName")
|
||||
if name == "" {
|
||||
name = strRaw(raw, "title")
|
||||
}
|
||||
return FormFields{
|
||||
PrimaryLabel: "Name", SecondaryLabel: "About",
|
||||
Primary: name, Secondary: strRaw(raw, "about"),
|
||||
ReadOnly: true,
|
||||
}
|
||||
case KindEvent, KindCalendar:
|
||||
return FormFields{
|
||||
PrimaryLabel: "Title", SecondaryLabel: "Description",
|
||||
Primary: strRaw(raw, "title"), Secondary: strRaw(raw, "description"),
|
||||
ReadOnly: true,
|
||||
}
|
||||
default:
|
||||
title := strRaw(raw, "title")
|
||||
if title == "" {
|
||||
title = strRaw(raw, "name")
|
||||
}
|
||||
if title == "" {
|
||||
title = strRaw(raw, "subject")
|
||||
}
|
||||
return FormFields{
|
||||
PrimaryLabel: "Title", SecondaryLabel: "Description",
|
||||
Primary: title, Secondary: strRaw(raw, "description"),
|
||||
ReadOnly: kind != KindTask && kind != KindProject,
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
func strRaw(m map[string]any, key string) string {
|
||||
if m == nil {
|
||||
return ""
|
||||
}
|
||||
if v, ok := m[key].(string); ok {
|
||||
return v
|
||||
}
|
||||
if m[key] == nil {
|
||||
return ""
|
||||
}
|
||||
return fmt.Sprint(m[key])
|
||||
}
|
||||
|
||||
// IsDocumentKind is true when the right pane should show file content, not a form.
|
||||
func IsDocumentKind(kind Kind) bool {
|
||||
return kind == KindFile
|
||||
}
|
||||
@@ -0,0 +1,20 @@
|
||||
package model
|
||||
|
||||
import "testing"
|
||||
|
||||
func TestFormFieldsFromRawMail(t *testing.T) {
|
||||
raw := map[string]any{"subject": "Hi", "body": "Hello"}
|
||||
f := FormFieldsFromRaw(KindMail, raw)
|
||||
if f.Primary != "Hi" || f.Secondary != "Hello" || !f.ReadOnly {
|
||||
t.Fatalf("unexpected mail fields: %+v", f)
|
||||
}
|
||||
}
|
||||
|
||||
func TestIsDocumentKind(t *testing.T) {
|
||||
if !IsDocumentKind(KindFile) {
|
||||
t.Fatal("file should be document")
|
||||
}
|
||||
if IsDocumentKind(KindTask) {
|
||||
t.Fatal("task should be form")
|
||||
}
|
||||
}
|
||||
@@ -69,7 +69,7 @@ func TestPrevFocusPane(t *testing.T) {
|
||||
|
||||
func TestActionsForContact(t *testing.T) {
|
||||
acts := model.ActionsFor(model.KindContact)
|
||||
if len(acts) < 2 {
|
||||
t.Fatalf("expected view+delete, got %d", len(acts))
|
||||
if len(acts) != 1 || acts[0].ID != model.ActionDelete {
|
||||
t.Fatalf("expected delete only, got %v", acts)
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user