tabby-tabbyspaces 0.0.1 → 0.1.0

package/src/components/workspaceEditor.component.scss

@@ -1,169 +1,254 @@
-.workspace-editor-overlay {
-  position: fixed;
-  top: 0;
-  left: 0;
-  right: 0;
-  bottom: 0;
-  background: rgba(0, 0, 0, 0.6);
-  display: flex;
-  align-items: center;
-  justify-content: center;
-  z-index: 1000;
-}
-
-.workspace-editor-modal {
-  background: var(--theme-bg);
-  border-radius: 12px;
-  width: 90%;
-  max-width: 700px;
-  max-height: 90vh;
-  overflow: hidden;
-  display: flex;
-  flex-direction: column;
-  box-shadow: 0 20px 60px rgba(0, 0, 0, 0.4);
-}
-
-.modal-header {
-  display: flex;
-  justify-content: space-between;
-  align-items: center;
-  padding: 16px 20px;
-  border-bottom: 1px solid var(--theme-border);
+// Workspace editor (inline)
+.workspace-editor-inline {
+  background: var(--theme-bg-more);
+  border-radius: 8px;
+  border: 1px solid var(--theme-border);
+  margin-bottom: 16px;
 
-  h4 {
-    margin: 0;
-    font-size: 1.25rem;
+  .editor-body {
+    padding: 16px;
   }
 
-  .close-btn {
-    font-size: 1.25rem;
-    color: var(--theme-fg-more);
-    padding: 4px 8px;
+  .editor-footer {
+    display: flex;
+    justify-content: space-between;
+    align-items: center;
+    padding: 10px 16px;
+    border-top: 1px solid var(--theme-border);
+    background: var(--theme-bg);
+    border-radius: 0 0 8px 8px;
+
+    .startup-checkbox {
+      display: flex;
+      align-items: center;
+      gap: 6px;
+      font-size: 0.85rem;
+      color: var(--theme-fg-more);
+      cursor: pointer;
+
+      input {
+        margin: 0;
+      }
+    }
 
-    &:hover {
-      color: var(--theme-fg);
+    .editor-actions {
+      display: flex;
+      gap: 8px;
+
+      .btn-success {
+        background: #10b981;
+        border-color: #10b981;
+        color: white;
+
+        &:hover {
+          background: #059669;
+          border-color: #059669;
+        }
+      }
+
+      .unsaved-indicator {
+        color: #f59e0b;
+        font-weight: bold;
+        margin-left: 2px;
+      }
     }
   }
 }
 
-.modal-body {
-  padding: 20px;
-  overflow-y: auto;
-  flex: 1;
-}
-
-.form-row {
+// Shared editor content
+.editor-top-row {
   display: flex;
-  gap: 16px;
+  align-items: flex-start;
+  gap: 12px;
   margin-bottom: 16px;
-
-  &:last-child {
-    margin-bottom: 0;
-  }
 }
 
-.form-group {
-  display: flex;
-  flex-direction: column;
-  gap: 6px;
-
-  &.flex-grow {
-    flex: 1;
-  }
-
-  label {
-    font-size: 0.9rem;
-    color: var(--theme-fg-more);
-  }
+.color-picker {
+  .color-input {
+    width: 40px;
+    height: 40px;
+    padding: 2px;
+    border: 1px solid var(--theme-border);
+    border-radius: 6px;
+    cursor: pointer;
+    background: var(--theme-bg);
+
+    &::-webkit-color-swatch-wrapper {
+      padding: 2px;
+    }
 
-  input[type='checkbox'] {
-    margin-right: 8px;
+    &::-webkit-color-swatch {
+      border-radius: 4px;
+      border: none;
+    }
   }
 }
 
-.form-control {
-  padding: 8px 12px;
+.name-input {
+  flex: 1;
+  min-width: 150px;
+  padding: 10px 12px;
   border-radius: 6px;
   border: 1px solid var(--theme-border);
-  background: var(--theme-bg-more);
+  background: var(--theme-bg);
   color: var(--theme-fg);
-  font-size: 0.95rem;
+  font-size: 1rem;
+  font-weight: 500;
 
   &:focus {
     outline: none;
     border-color: var(--theme-primary);
   }
-}
 
-.color-input {
-  width: 60px;
-  height: 36px;
-  padding: 2px;
-  cursor: pointer;
+  &::placeholder {
+    color: var(--theme-fg-more);
+    font-weight: 400;
+  }
 }
 
-.icon-selector {
-  display: flex;
-  flex-wrap: wrap;
-  gap: 4px;
-}
+.icon-picker {
+  position: relative;
 
-.icon-option {
-  width: 36px;
-  height: 36px;
-  display: flex;
-  align-items: center;
-  justify-content: center;
-  border: 1px solid var(--theme-border);
-  border-radius: 6px;
-  background: var(--theme-bg-more);
-  color: var(--theme-fg-more);
-  cursor: pointer;
-  transition: all 0.2s;
+  .icon-trigger {
+    width: 40px;
+    height: 40px;
+    display: flex;
+    align-items: center;
+    justify-content: center;
+    border: 1px solid var(--theme-border);
+    border-radius: 6px;
+    background: var(--theme-bg);
+    cursor: pointer;
+    font-size: 1.1rem;
+    transition: background 0.15s;
 
-  &:hover {
-    background: var(--theme-bg-more-more);
-    color: var(--theme-fg);
+    &:hover {
+      background: var(--theme-bg-more);
+    }
   }
 
-  &.selected {
-    border-color: var(--theme-primary);
-    background: var(--theme-primary);
-    color: white;
+  .icon-dropdown {
+    position: absolute;
+    top: 100%;
+    right: 0;
+    margin-top: 4px;
+    padding: 8px;
+    background: var(--theme-bg-more);
+    border: 1px solid var(--theme-border);
+    border-radius: 6px;
+    box-shadow: 0 4px 12px rgba(0, 0, 0, 0.15);
+    z-index: 100;
+    display: grid;
+    grid-template-columns: repeat(6, 28px);
+    gap: 4px;
   }
-}
 
-.form-section {
-  margin-top: 20px;
+  .icon-option {
+    width: 28px;
+    height: 28px;
+    display: flex;
+    align-items: center;
+    justify-content: center;
+    border: 1px solid transparent;
+    border-radius: 4px;
+    background: transparent;
+    color: var(--theme-fg-more);
+    cursor: pointer;
+    transition: all 0.15s;
+    font-size: 0.85rem;
+
+    &:hover {
+      background: var(--theme-bg-more-more);
+      color: var(--theme-fg);
+    }
+
+    &.selected {
+      border-color: currentColor;
+      background: var(--theme-bg);
+    }
+  }
+}
 
-  .section-header {
+// Layout section
+.layout-section {
+  .layout-header {
     display: flex;
     justify-content: space-between;
     align-items: center;
-    margin-bottom: 12px;
+    margin-bottom: 8px;
+  }
 
-    h5 {
-      margin: 0;
-      font-size: 1rem;
-    }
+  .layout-title {
+    font-size: 0.8rem;
+    font-weight: 500;
+    color: var(--theme-fg-more);
+    text-transform: uppercase;
+    letter-spacing: 0.5px;
   }
 }
 
-.orientation-toggle {
+// Layout toolbar
+.layout-toolbar {
   display: flex;
-  gap: 8px;
+  align-items: center;
+  gap: 4px;
+  padding: 8px 0;
+  margin-bottom: 8px;
 
-  .btn {
+  .toolbar-group {
     display: flex;
     align-items: center;
-    gap: 6px;
+    gap: 2px;
+  }
+
+  .toolbar-label {
+    font-size: 0.75rem;
+    color: var(--theme-fg-more);
+    margin-right: 4px;
+    text-transform: uppercase;
+    letter-spacing: 0.3px;
+  }
+
+  .toolbar-spacer {
+    flex: 1;
+  }
+
+  .toolbar-divider {
+    width: 1px;
+    height: 20px;
+    background: var(--theme-border);
+    margin: 0 4px;
+  }
+
+  .toolbar-btn {
+    padding: 4px 8px;
+    background: var(--theme-bg);
+    border: 1px solid var(--theme-border);
+    border-radius: 4px;
+    color: var(--theme-fg);
+    cursor: pointer;
+    transition: all 0.15s;
+    font-size: 0.85rem;
+
+    &:hover:not(:disabled) {
+      background: var(--theme-bg-more-more);
+    }
+
+    &:disabled {
+      opacity: 0.4;
+      cursor: not-allowed;
+    }
+
+    &.danger {
+      color: var(--theme-danger, #ef4444);
+    }
   }
 }
 
-.modal-footer {
-  display: flex;
-  justify-content: flex-end;
-  gap: 12px;
-  padding: 16px 20px;
-  border-top: 1px solid var(--theme-border);
+.layout-preview-container {
+  background: var(--theme-bg);
+  border-radius: 6px;
+  padding: 8px;
+  min-height: 120px;
+  max-height: 180px;
 }

package/src/components/workspaceEditor.component.ts

@@ -1,4 +1,4 @@
-import { Component, Input, Output, EventEmitter, OnInit } from '@angular/core'
+import { Component, Input, Output, EventEmitter, OnInit, OnChanges, AfterViewInit, SimpleChanges, HostListener, ElementRef, ViewChild } from '@angular/core'
 import {
   Workspace,
   WorkspacePane,
@@ -15,12 +15,17 @@ import { WorkspaceEditorService } from '../services/workspaceEditor.service'
   template: require('./workspaceEditor.component.pug'),
   styles: [require('./workspaceEditor.component.scss')],
 })
-export class WorkspaceEditorComponent implements OnInit {
+export class WorkspaceEditorComponent implements OnInit, OnChanges, AfterViewInit {
   @Input() workspace!: Workspace
+  @Input() autoFocus = false
+  @Input() hasUnsavedChanges = false
   @Output() save = new EventEmitter<Workspace>()
   @Output() cancel = new EventEmitter<void>()
 
-  selectedPane: WorkspacePane | null = null
+  @ViewChild('nameInput') nameInput!: ElementRef<HTMLInputElement>
+
+  selectedPaneId: string | null = null
+  editingPane: WorkspacePane | null = null
   showPaneEditor = false
   profiles: TabbyProfile[] = []
   availableIcons = [
@@ -28,11 +33,69 @@ export class WorkspaceEditorComponent implements OnInit {
     'cog', 'database', 'server', 'cloud', 'rocket', 'flask',
     'bug', 'wrench', 'cube', 'layer-group', 'sitemap', 'project-diagram'
   ]
+  iconDropdownOpen = false
+
+  constructor(
+    private workspaceService: WorkspaceEditorService,
+    private elementRef: ElementRef
+  ) {}
+
+  @HostListener('document:click', ['$event'])
+  onDocumentClick(event: MouseEvent): void {
+    const iconPicker = this.elementRef.nativeElement.querySelector('.icon-picker')
+    if (iconPicker && !iconPicker.contains(event.target as Node)) {
+      this.iconDropdownOpen = false
+    }
+  }
 
-  constructor(private workspaceService: WorkspaceEditorService) {}
+  toggleIconDropdown(): void {
+    this.iconDropdownOpen = !this.iconDropdownOpen
+  }
+
+  selectIcon(icon: string): void {
+    this.workspace.icon = icon
+    this.iconDropdownOpen = false
+  }
 
   async ngOnInit(): Promise<void> {
     this.profiles = await this.workspaceService.getAvailableProfiles()
+    this.initializeWorkspace()
+  }
+
+  ngAfterViewInit(): void {
+    if (this.autoFocus) {
+      this.focusNameInput()
+    }
+  }
+
+  private focusNameInput(): void {
+    requestAnimationFrame(() => {
+      setTimeout(() => {
+        if (this.nameInput?.nativeElement) {
+          this.nameInput.nativeElement.focus()
+          this.nameInput.nativeElement.select()
+        }
+      }, 0)
+    })
+  }
+
+  ngOnChanges(changes: SimpleChanges): void {
+    if (changes['workspace'] && !changes['workspace'].firstChange) {
+      // Reset component state when workspace input changes
+      this.selectedPaneId = null
+      this.editingPane = null
+      this.showPaneEditor = false
+      this.iconDropdownOpen = false
+      this.initializeWorkspace()
+    }
+
+    // Handle autoFocus change
+    if (changes['autoFocus']?.currentValue) {
+      this.focusNameInput()
+    }
+  }
+
+  private initializeWorkspace(): void {
     if (!this.workspace.root) {
       this.workspace.root = {
         orientation: 'horizontal',
@@ -54,13 +117,31 @@ export class WorkspaceEditorComponent implements OnInit {
   }
 
   selectPane(pane: WorkspacePane): void {
-    this.selectedPane = pane
+    this.selectedPaneId = pane.id
+  }
+
+  deselectPane(): void {
+    this.selectedPaneId = null
+  }
+
+  onPreviewBackgroundClick(): void {
+    this.deselectPane()
+  }
+
+  editPane(pane: WorkspacePane): void {
+    this.editingPane = pane
     this.showPaneEditor = true
   }
 
+  editSelectedPane(): void {
+    if (!this.selectedPaneId) return
+    const pane = this.findPaneById(this.selectedPaneId)
+    if (pane) this.editPane(pane)
+  }
+
   closePaneEditor(): void {
     this.showPaneEditor = false
-    this.selectedPane = null
+    this.editingPane = null
   }
 
   onPaneSave(pane: WorkspacePane): void {
@@ -68,6 +149,33 @@ export class WorkspaceEditorComponent implements OnInit {
     this.closePaneEditor()
   }
 
+  // Helper functions
+  private findPaneById(id: string): WorkspacePane | null {
+    return this.findPaneInNode(this.workspace.root, id)
+  }
+
+  private findPaneInNode(node: WorkspaceSplit, id: string): WorkspacePane | null {
+    for (const child of node.children) {
+      if (isWorkspaceSplit(child)) {
+        const found = this.findPaneInNode(child, id)
+        if (found) return found
+      } else if (child.id === id) {
+        return child
+      }
+    }
+    return null
+  }
+
+  canRemovePane(): boolean {
+    return this.countPanes(this.workspace.root) > 1
+  }
+
+  private countPanes(node: WorkspaceSplit): number {
+    return node.children.reduce((count, child) => {
+      return count + (isWorkspaceSplit(child) ? this.countPanes(child) : 1)
+    }, 0)
+  }
+
   private updatePaneInTree(node: WorkspaceSplit, updatedPane: WorkspacePane): boolean {
     for (let i = 0; i < node.children.length; i++) {
       const child = node.children[i]
@@ -87,6 +195,12 @@ export class WorkspaceEditorComponent implements OnInit {
     this.splitPaneInTree(this.workspace.root, pane, orientation)
   }
 
+  splitSelectedPane(orientation: 'horizontal' | 'vertical'): void {
+    if (!this.selectedPaneId) return
+    const pane = this.findPaneById(this.selectedPaneId)
+    if (pane) this.splitPane(pane, orientation)
+  }
+
   private splitPaneInTree(
     node: WorkspaceSplit,
     targetPane: WorkspacePane,
@@ -115,9 +229,18 @@ export class WorkspaceEditorComponent implements OnInit {
   }
 
   removePane(pane: WorkspacePane): void {
+    if (this.selectedPaneId === pane.id) {
+      this.selectedPaneId = null
+    }
     this.removePaneFromTree(this.workspace.root, pane)
   }
 
+  removeSelectedPane(): void {
+    if (!this.selectedPaneId || !this.canRemovePane()) return
+    const pane = this.findPaneById(this.selectedPaneId)
+    if (pane) this.removePane(pane)
+  }
+
   private removePaneFromTree(node: WorkspaceSplit, targetPane: WorkspacePane): boolean {
     for (let i = 0; i < node.children.length; i++) {
       const child = node.children[i]
@@ -178,4 +301,68 @@ export class WorkspaceEditorComponent implements OnInit {
     this.workspace.root.ratios = ratios
   }
 
+  // Add pane operations
+  addPane(direction: 'left' | 'right' | 'top' | 'bottom'): void {
+    if (!this.selectedPaneId) return
+    const pane = this.findPaneById(this.selectedPaneId)
+    if (!pane) return
+    this.addPaneInTree(this.workspace.root, pane, direction, null)
+  }
+
+  addPaneFromEvent(pane: WorkspacePane, direction: 'left' | 'right' | 'top' | 'bottom'): void {
+    this.addPaneInTree(this.workspace.root, pane, direction, null)
+  }
+
+  private addPaneInTree(
+    node: WorkspaceSplit,
+    targetPane: WorkspacePane,
+    direction: 'left' | 'right' | 'top' | 'bottom',
+    parentNode: WorkspaceSplit | null
+  ): boolean {
+    const isHorizontalAdd = direction === 'left' || direction === 'right'
+    const isBefore = direction === 'left' || direction === 'top'
+    const targetOrientation = isHorizontalAdd ? 'horizontal' : 'vertical'
+
+    for (let i = 0; i < node.children.length; i++) {
+      const child = node.children[i]
+
+      if (isWorkspaceSplit(child)) {
+        if (this.addPaneInTree(child, targetPane, direction, node)) return true
+      } else if (child.id === targetPane.id) {
+        const newPane = createDefaultPane()
+        newPane.profileId = child.profileId
+
+        if (node.orientation === targetOrientation) {
+          // Same orientation: add as sibling
+          const insertIndex = isBefore ? i : i + 1
+          node.children.splice(insertIndex, 0, newPane)
+          this.recalculateRatios(node)
+        } else {
+          // Different orientation: wrap entire node in new split
+          const nodeCopy: WorkspaceSplit = {
+            orientation: node.orientation,
+            ratios: [...node.ratios],
+            children: [...node.children]
+          }
+          const wrapper: WorkspaceSplit = {
+            orientation: targetOrientation,
+            ratios: [0.5, 0.5],
+            children: isBefore ? [newPane, nodeCopy] : [nodeCopy, newPane]
+          }
+
+          if (node === this.workspace.root) {
+            this.workspace.root = wrapper
+          } else if (parentNode) {
+            const nodeIndex = parentNode.children.indexOf(node)
+            if (nodeIndex !== -1) {
+              parentNode.children[nodeIndex] = wrapper
+            }
+          }
+        }
+        return true
+      }
+    }
+    return false
+  }
+
 }