tabby-tabbyspaces 0.2.4 → 0.2.6

package/src/components/workspaceList.component.pug

@@ -1,6 +1,6 @@
 .workspace-list-container
   //- Tab bar navigation
-  .tab-bar
+  .tab-bar(*ngIf='workspaces.length > 0 || editingWorkspace')
     .tab(
       *ngFor='let tab of displayTabs; trackBy: trackByTab',
       [class.active]='isTabSelected(tab)',
@@ -30,8 +30,11 @@
     )
 
   //- Empty state (when no workspaces)
-  .tab-content.empty-state(*ngIf='!editingWorkspace && workspaces.length === 0')
-    p No workspaces configured yet.
-    p Click
-      strong +
-      | to create your first workspace.
+  .empty-state(*ngIf='!editingWorkspace && workspaces.length === 0')
+    .empty-state-icon
+      i.fas.fa-columns
+    h2.empty-state-title TabbySpaces
+    p.empty-state-description Create custom split layouts with profiles, working directories, and startup commands.
+    button.empty-state-cta((click)='createWorkspace()')
+      i.fas.fa-plus
+      | Create Workspace

package/src/components/workspaceList.component.scss

@@ -1,7 +1,18 @@
 @use '../styles/index' as *;
 
+:host {
+  display: flex;
+  flex-direction: column;
+  flex: 1;
+  min-height: 0;
+}
+
 .workspace-list-container {
   padding: $spacing-2xl;
+  display: flex;
+  flex-direction: column;
+  flex: 1;
+  min-height: 0;
 }
 
 // Tab bar
@@ -109,19 +120,43 @@
   border: 1px solid var(--theme-border, $fallback-border);
   border-top: none;
   padding: $spacing-xl;
+  flex: 1;
+  min-height: 0;
+  display: flex;
+  flex-direction: column;
+}
+
+// Empty state
+.empty-state {
+  @include flex-center;
+  flex-direction: column;
+  padding: 80px $spacing-2xl;
+  text-align: center;
+  flex: 1;
+}
 
-  &.empty-state {
-    text-align: center;
-    padding: $spacing-2xl;
-    color: var(--theme-fg-more);
+.empty-state-icon {
+  font-size: 2.5rem;
+  color: var(--theme-fg-more, $fallback-fg-more);
+  margin-bottom: $spacing-xl;
+  opacity: 0.5;
+}
 
-    p {
-      margin: $spacing-md 0;
-    }
+.empty-state-title {
+  font-size: 1.2rem;
+  font-weight: 600;
+  color: var(--theme-fg);
+  margin: 0 0 $spacing-md;
+}
 
-    strong {
-      color: var(--theme-primary);
-      padding: 0 $spacing-xs;
-    }
-  }
+.empty-state-description {
+  font-size: $font-sm;
+  color: var(--theme-fg-more, $fallback-fg-more);
+  margin: 0 0 $spacing-2xl;
+  max-width: 320px;
+  line-height: 1.5;
+}
+
+.empty-state-cta {
+  @include btn-primary;
 }

package/src/components/workspaceList.component.ts

@@ -16,7 +16,7 @@ import {
   isWorkspaceSplit,
 } from '../models/workspace.model'
 
-const SETTINGS_MAX_WIDTH = '876px'
+const SETTINGS_MAX_WIDTH = 'none'
 
 @Component({
   selector: 'workspace-list',
@@ -53,12 +53,22 @@ export class WorkspaceListComponent implements OnInit, OnDestroy, AfterViewInit
     })
   }
 
+  private originalStyles: { element: HTMLElement; maxWidth: string; height: string }[] = []
+
   ngAfterViewInit(): void {
-    // Hack: Override Tabby's settings-tab-body max-width restriction
+    // Hack: Override Tabby's settings-tab-body to allow full-height layout.
+    // Only touches settingsTabBody — host styles are in SCSS :host block.
+    // Parent elements (like .tab-pane) are NOT modified to avoid affecting other tabs.
     setTimeout(() => {
-      const parent = this.elementRef.nativeElement.closest('settings-tab-body') as HTMLElement
-      if (parent) {
-        parent.style.maxWidth = SETTINGS_MAX_WIDTH
+      const settingsTabBody = this.elementRef.nativeElement.closest('settings-tab-body') as HTMLElement
+      if (settingsTabBody) {
+        this.originalStyles.push({
+          element: settingsTabBody,
+          maxWidth: settingsTabBody.style.maxWidth,
+          height: settingsTabBody.style.height,
+        })
+        settingsTabBody.style.maxWidth = SETTINGS_MAX_WIDTH
+        settingsTabBody.style.height = '100%'
       }
     }, 0)
   }
@@ -82,6 +92,12 @@ export class WorkspaceListComponent implements OnInit, OnDestroy, AfterViewInit
 
   ngOnDestroy(): void {
     this.configSubscription?.unsubscribe()
+    // Restore original styles on parent elements to avoid affecting other settings tabs
+    for (const entry of this.originalStyles) {
+      entry.element.style.maxWidth = entry.maxWidth
+      entry.element.style.height = entry.height
+    }
+    this.originalStyles = []
   }
 
   loadWorkspaces(): void {

package/src/models/workspace.model.ts

@@ -58,6 +58,7 @@ export interface TabbySplitLayoutProfile {
 // Workspace interfaces
 export interface WorkspacePane {
   id: string
+  name?: string
   profileId: string
   cwd?: string
   startupCommand?: string

package/src/styles/_mixins.scss

@@ -173,7 +173,7 @@
 // ======================
 
 @mixin dropdown-panel {
-  background: var(--theme-bg-more);
+  background: var(--theme-bg-more, $fallback-bg-more);
   border: 1px solid var(--theme-border, $fallback-border);
   border-radius: $radius-md;
   box-shadow: $shadow-dropdown;

package/src/styles/_variables.scss

@@ -28,6 +28,7 @@ $color-danger: #ef4444;
 // Fallback values for Tabby theme variables that may not be defined
 $fallback-border: rgba(255, 255, 255, 0.1);
 $fallback-fg-more: rgba(255, 255, 255, 0.3);
+$fallback-bg-more: #1e1e1e;
 
 // ======================
 // SHADOWS (S1: Flat - no shadows)

package/.github/workflows/claude-code-review.yml

@@ -1,44 +0,0 @@
-name: Claude Code Review
-
-on:
-  pull_request:
-    types: [opened, synchronize, ready_for_review, reopened]
-    # Optional: Only run on specific file changes
-    # paths:
-    #   - "src/**/*.ts"
-    #   - "src/**/*.tsx"
-    #   - "src/**/*.js"
-    #   - "src/**/*.jsx"
-
-jobs:
-  claude-review:
-    # Only run for repository members to control costs
-    if: |
-      github.event.pull_request.author_association == 'OWNER' ||
-      github.event.pull_request.author_association == 'MEMBER' ||
-      github.event.pull_request.author_association == 'COLLABORATOR'
-
-    runs-on: ubuntu-latest
-    permissions:
-      contents: write
-      pull-requests: write
-      issues: read
-      id-token: write
-
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-
-      - name: Run Claude Code Review
-        id: claude-review
-        uses: anthropics/claude-code-action@v1.0.0
-        with:
-          claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
-          prompt: >
-            Please review the changes in this pull request and leave a concise summary comment with any issues, suggestions, and potential bugs:
-            https://github.com/${{ github.repository }}/pull/${{ github.event.pull_request.number }}
-          # See https://github.com/anthropics/claude-code-action/blob/main/docs/usage.md
-          # or https://code.claude.com/docs/en/cli-reference for available options
-