Update ui/okr_ui_generator.py

ui/okr_ui_generator.py

@@ -19,7 +19,7 @@ def create_enhanced_okr_tab():
     okr_custom_css = """
     <style>
     /* ----------------------------------------- */
-    /* --- THEME & COLOR VARIABLES (NEW) --- */
+    /* --- LIGHT MODE THEME & COLOR VARIABLES --- */
     /* ----------------------------------------- */
     :root {
         --okr-bg-start: #667eea;
@@ -36,8 +36,8 @@ def create_enhanced_okr_tab():
         --objective-card-border: #3b82f6;
         --objective-shadow: 0 4px 16px rgba(0,0,0,0.05);
         --objective-shadow-hover: 0 8px 24px rgba(0,0,0,0.1);
-        --objective-header-bg: transparent; /* Cleaner header background */
-        --objective-title-color: #1e40af; /* Make title the colored element */
+        --objective-header-bg: transparent;
+        --objective-title-color: #1e40af;
         --objective-meta-text-color: #475569;
         --key-result-bg: white;
         --key-result-border: #e5e7eb;
@@ -68,50 +68,9 @@ def create_enhanced_okr_tab():
         --priority-low-border: #86efac;
     }
 
-    /* --- DARK MODE OVERRIDES (IMPROVED) --- */
-    html.dark {
-        --header-text-color: #e5e7eb;
-        --stat-card-bg: rgba(28, 28, 28, 0.7);
-        --stat-card-bg-hover: rgba(40, 40, 40, 0.8);
-        --stat-card-border: rgba(50, 50, 50, 0.8);
-        --content-bg: #0d1117; /* Black background for content */
-        --content-shadow: rgba(0,0,0,0.6);
-        --objective-card-bg: #161b22; /* Blackish card background */
-        --objective-card-border: #3b82f6;
-        --objective-shadow: 0 8px 24px rgba(0,0,0,0.5);
-        --objective-shadow-hover: 0 10px 28px rgba(0,0,0,0.6);
-        --objective-title-color: #58a6ff; /* White-ish blue for title */
-        --objective-meta-text-color: #8b949e; /* White-ish grey for meta */
-        --key-result-bg: #161b22;
-        --key-result-border: #30363d;
-        --key-result-border-hover: #58a6ff;
-        --kr-header-bg: #161b22;
-        --kr-title-color: #c9d1d9; /* White text */
-        --kr-metric-bg: rgba(56, 139, 253, 0.15);
-        --kr-metric-color: #58a6ff;
-        --kr-metric-border: rgba(56, 139, 253, 0.4);
-        --task-item-bg: #21262d;
-        --task-item-bg-hover: #30363d;
-        --task-item-border: #30363d;
-        --task-item-border-hover: #8b949e;
-        --task-title-color: #c9d1d9; /* White text */
-        --task-detail-label-color: #8b949e;
-        --task-detail-text-color: #8b949e;
-        --task-description-bg: #0d1117;
-        --task-description-border: #30363d;
-        --task-description-color: #8b949e;
-        --priority-high-bg: rgba(248, 81, 73, 0.15);
-        --priority-high-color: #ff7b72;
-        --priority-high-border: rgba(248, 81, 73, 0.4);
-        --priority-medium-bg: rgba(210, 153, 34, 0.15);
-        --priority-medium-color: #d29922;
-        --priority-medium-border: rgba(210, 153, 34, 0.4);
-        --priority-low-bg: rgba(63, 185, 80, 0.15);
-        --priority-low-color: #56d364;
-        --priority-low-border: rgba(63, 185, 80, 0.4);
-    }
-
-    /* Overall Container */
+    /* ----------------------------------------- */
+    /* --- BASE STYLES (Uses Variables) --- */
+    /* ----------------------------------------- */
     .okr-container {
         font-family: 'Inter', sans-serif;
         background: linear-gradient(135deg, var(--okr-bg-start) 0%, var(--okr-bg-end) 100%);
@@ -122,385 +81,98 @@ def create_enhanced_okr_tab():
         overflow-y: auto;
     }
 
-    /* Header Section */
-    .okr-header {
-        text-align: center;
-        margin-bottom: 3rem;
-        color: var(--header-text-color);
-    }
-
-    .okr-title {
-        font-size: 2.5rem;
-        font-weight: 700;
-        margin-bottom: 0.5rem;
-        text-shadow: 0 2px 4px var(--header-text-shadow);
-        display: flex;
-        justify-content: center;
-        align-items: center;
-        gap: 0.75rem;
-    }
-
-    .okr-title-content {
-        background: linear-gradient(45deg, #ffffff, #e0e7ff);
-        -webkit-background-clip: text;
-        -webkit-text-fill-color: transparent;
-        background-clip: text;
-    }
+    .okr-header { text-align: center; margin-bottom: 3rem; color: var(--header-text-color); }
+    .okr-title { font-size: 2.5rem; font-weight: 700; margin-bottom: 0.5rem; text-shadow: 0 2px 4px var(--header-text-shadow); display: flex; justify-content: center; align-items: center; gap: 0.75rem; }
+    .okr-title-content { background: linear-gradient(45deg, #ffffff, #e0e7ff); -webkit-background-clip: text; -webkit-text-fill-color: transparent; background-clip: text; }
+    .okr-title-emoji { font-size: 2.5rem; -webkit-text-fill-color: initial; }
+    .okr-subtitle { font-size: 1.2rem; opacity: 0.9; font-weight: 300; letter-spacing: 0.5px; }
+    .okr-stats-bar { display: flex; justify-content: center; gap: 2rem; margin: 2rem 0; flex-wrap: wrap; }
+    .stat-card { background: var(--stat-card-bg); backdrop-filter: blur(10px); border: 1px solid var(--stat-card-border); border-radius: 16px; padding: 1.5rem; text-align: center; color: var(--header-text-color); min-width: 140px; transition: all 0.3s ease; }
+    .stat-card:hover { transform: translateY(-2px); background: var(--stat-card-bg-hover); box-shadow: 0 8px 32px var(--content-shadow); }
+    .stat-number { font-size: 2rem; font-weight: 700; margin-bottom: 0.25rem; color: var(--stat-number-color); }
+    .stat-label { font-size: 0.9rem; opacity: 0.9; text-transform: uppercase; letter-spacing: 1px; }
+    .okr-content { background: var(--content-bg); border-radius: 24px; padding: 0; box-shadow: 0 20px 40px var(--content-shadow); overflow: hidden; margin-top: 2rem; }
+    .okr-objective { background: var(--objective-card-bg); border-left: 6px solid var(--objective-card-border); margin: 2rem; border-radius: 16px; overflow: hidden; box-shadow: var(--objective-shadow); transition: all 0.3s ease; }
+    .okr-objective:hover { transform: translateY(-2px); box-shadow: var(--objective-shadow-hover); }
+    .objective-header { padding: 2rem; background: var(--objective-header-bg); border-bottom: 1px solid var(--key-result-border); position: relative; }
+    .objective-title { font-size: 1.5rem; font-weight: 700; margin-bottom: 0.75rem; position: relative; z-index: 1; color: var(--objective-title-color); }
+    .objective-meta { display: flex; gap: 2rem; margin-top: 1rem; flex-wrap: wrap; position: relative; z-index: 1; }
+    .meta-item { display: flex; align-items: center; gap: 0.5rem; font-size: 0.9rem; color: var(--objective-meta-text-color); }
+    .meta-icon { width: 16px; height: 16px; opacity: 0.8; }
+    .key-results-container { padding: 2rem; }
+    .key-result { background: var(--key-result-bg); border: 2px solid var(--key-result-border); border-radius: 12px; margin: 1.5rem 0; overflow: hidden; transition: all 0.3s ease; }
+    .key-result:hover { border-color: var(--key-result-border-hover); box-shadow: 0 4px 12px rgba(59, 130, 246, 0.1); }
+    .kr-header { background: var(--kr-header-bg); padding: 1.5rem; border-bottom: 1px solid var(--key-result-border); }
+    .kr-title { font-size: 1.2rem; font-weight: 600; color: var(--kr-title-color); margin-bottom: 0.75rem; }
+    .kr-metrics { display: flex; gap: 1.5rem; flex-wrap: wrap; margin-top: 1rem; }
+    .kr-metric { background: var(--kr-metric-bg); color: var(--kr-metric-color); padding: 0.5rem 1rem; border-radius: 8px; font-size: 0.85rem; font-weight: 500; border: 1px solid var(--kr-metric-border); }
+    .tasks-section { padding: 1.5rem; }
+    .tasks-title { font-size: 1rem; font-weight: 600; color: var(--task-detail-label-color); margin-bottom: 1rem; display: flex; align-items: center; gap: 0.5rem; }
+    .task-item { background: var(--task-item-bg); border: 1px solid var(--task-item-border); border-radius: 8px; padding: 1.25rem; margin: 1rem 0; transition: all 0.2s ease; }
+    .task-item:hover { background: var(--task-item-bg-hover); border-color: var(--task-item-border-hover); }
+    .task-header { display: flex; justify-content: space-between; align-items: flex-start; margin-bottom: 1rem; gap: 1rem; }
+    .task-title { font-weight: 600; color: var(--task-title-color); flex: 1; line-height: 1.4; }
+    .task-priority { padding: 0.25rem 0.75rem; border-radius: 12px; font-size: 0.75rem; font-weight: 600; text-transform: uppercase; letter-spacing: 0.5px; white-space: nowrap; }
+    .priority-high { background: var(--priority-high-bg); color: var(--priority-high-color); border: 1px solid var(--priority-high-border); }
+    .priority-medium { background: var(--priority-medium-bg); color: var(--priority-medium-color); border: 1px solid var(--priority-medium-border); }
+    .priority-low { background: var(--priority-low-bg); color: var(--priority-low-color); border: 1px solid var(--priority-low-border); }
+    .task-details { display: grid; grid-template-columns: repeat(auto-fit, minmax(200px, 1fr)); gap: 1rem; margin-top: 1rem; }
+    .task-detail-item { display: flex; align-items: center; gap: 0.5rem; font-size: 0.875rem; color: var(--task-detail-text-color); }
+    .task-detail-label { font-weight: 500; color: var(--task-detail-label-color); min-width: 80px; }
+    .task-description { margin-top: 1rem; padding: 1rem; background: var(--task-description-bg); border-radius: 6px; border-left: 3px solid var(--task-description-border); font-size: 0.9rem; line-height: 1.5; color: var(--task-description-color); }
+    .empty-state { text-align: center; padding: 4rem 2rem; color: var(--task-detail-text-color); }
+    .empty-state-icon { font-size: 3rem; margin-bottom: 1rem; }
+    .empty-state-title { font-size: 1.5rem; font-weight: 600; margin-bottom: 0.5rem; color: var(--objective-title-color); }
+    .loading-spinner { display: inline-block; width: 20px; height: 20px; border: 3px solid var(--task-item-bg-hover); border-radius: 50%; border-top-color: var(--objective-card-border); animation: spin 1s ease-in-out infinite; }
+    @keyframes spin { to { transform: rotate(360deg); } }
     
-    html.dark .okr-title-content {
-        background: linear-gradient(45deg, #e5e7eb, #9ca3af);
-        -webkit-background-clip: text;
-        -webkit-text-fill-color: transparent;
-        background-clip: text;
-    }
-
-    .okr-title-emoji {
-        font-size: 2.5rem;
-        -webkit-text-fill-color: initial;
-    }
-
-    .okr-subtitle {
-        font-size: 1.2rem;
-        opacity: 0.9;
-        font-weight: 300;
-        letter-spacing: 0.5px;
-    }
-
-    /* Stats Bar */
-    .okr-stats-bar {
-        display: flex;
-        justify-content: center;
-        gap: 2rem;
-        margin: 2rem 0;
-        flex-wrap: wrap;
-    }
-
-    .stat-card {
-        background: var(--stat-card-bg);
-        backdrop-filter: blur(10px);
-        border: 1px solid var(--stat-card-border);
-        border-radius: 16px;
-        padding: 1.5rem;
-        text-align: center;
-        color: var(--header-text-color);
-        min-width: 140px;
-        transition: all 0.3s ease;
-    }
-
-    .stat-card:hover {
-        transform: translateY(-2px);
-        background: var(--stat-card-bg-hover);
-        box-shadow: 0 8px 32px var(--content-shadow);
-    }
-
-    .stat-number {
-        font-size: 2rem;
-        font-weight: 700;
-        margin-bottom: 0.25rem;
-        color: var(--stat-number-color);
-    }
-
-    .stat-label {
-        font-size: 0.9rem;
-        opacity: 0.9;
-        text-transform: uppercase;
-        letter-spacing: 1px;
-    }
-
-    /* Main Content Area */
-    .okr-content {
-        background: var(--content-bg);
-        border-radius: 24px;
-        padding: 0;
-        box-shadow: 0 20px 40px var(--content-shadow);
-        overflow: hidden;
-        margin-top: 2rem;
-    }
-
-    /* Objective Card */
-    .okr-objective {
-        background: var(--objective-card-bg);
-        border-left: 6px solid var(--objective-card-border);
-        margin: 2rem;
-        border-radius: 16px;
-        overflow: hidden;
-        box-shadow: var(--objective-shadow);
-        transition: all 0.3s ease;
-    }
-
-    .okr-objective:hover {
-        transform: translateY(-2px);
-        box-shadow: var(--objective-shadow-hover);
-    }
-    
-    /* --- Objective Header (REDESIGNED) --- */
-    .objective-header {
-        padding: 2rem;
-        background: var(--objective-header-bg);
-        border-bottom: 1px solid var(--key-result-border); /* Use variable for consistency */
-        position: relative;
-    }
-    
-    .objective-title {
-        font-size: 1.5rem;
-        font-weight: 700;
-        margin-bottom: 0.75rem;
-        position: relative;
-        z-index: 1;
-        color: var(--objective-title-color); /* Use variable */
-    }
-
-    .objective-meta {
-        display: flex;
-        gap: 2rem;
-        margin-top: 1rem;
-        flex-wrap: wrap;
-        position: relative;
-        z-index: 1;
-    }
-
-    .meta-item {
-        display: flex;
-        align-items: center;
-        gap: 0.5rem;
-        font-size: 0.9rem;
-        color: var(--objective-meta-text-color);
-    }
-
-    .meta-icon {
-        width: 16px;
-        height: 16px;
-        opacity: 0.8;
-    }
-
-    /* Key Results Container */
-    .key-results-container {
-        padding: 2rem;
-    }
-
-    /* Key Result Card */
-    .key-result {
-        background: var(--key-result-bg);
-        border: 2px solid var(--key-result-border);
-        border-radius: 12px;
-        margin: 1.5rem 0;
-        overflow: hidden;
-        transition: all 0.3s ease;
-    }
-
-    .key-result:hover {
-        border-color: var(--key-result-border-hover);
-        box-shadow: 0 4px 12px rgba(59, 130, 246, 0.1);
-    }
-    
-    html.dark .key-result:hover {
-        box-shadow: 0 4px 12px rgba(59, 130, 246, 0.2);
-    }
-
-    .kr-header {
-        background: var(--kr-header-bg);
-        padding: 1.5rem;
-        border-bottom: 1px solid var(--key-result-border);
-    }
-
-    .kr-title {
-        font-size: 1.2rem;
-        font-weight: 600;
-        color: var(--kr-title-color);
-        margin-bottom: 0.75rem;
-    }
-
-    .kr-metrics {
-        display: flex;
-        gap: 1.5rem;
-        flex-wrap: wrap;
-        margin-top: 1rem;
-    }
-
-    .kr-metric {
-        background: var(--kr-metric-bg);
-        color: var(--kr-metric-color);
-        padding: 0.5rem 1rem;
-        border-radius: 8px;
-        font-size: 0.85rem;
-        font-weight: 500;
-        border: 1px solid var(--kr-metric-border);
-    }
-
-    /* Tasks Section */
-    .tasks-section {
-        padding: 1.5rem;
-    }
-
-    .tasks-title {
-        font-size: 1rem;
-        font-weight: 600;
-        color: var(--task-detail-label-color);
-        margin-bottom: 1rem;
-        display: flex;
-        align-items: center;
-        gap: 0.5rem;
-    }
-
-    .task-item {
-        background: var(--task-item-bg);
-        border: 1px solid var(--task-item-border);
-        border-radius: 8px;
-        padding: 1.25rem;
-        margin: 1rem 0;
-        transition: all 0.2s ease;
-    }
-
-    .task-item:hover {
-        background: var(--task-item-bg-hover);
-        border-color: var(--task-item-border-hover);
-    }
-
-    .task-header {
-        display: flex;
-        justify-content: space-between;
-        align-items: flex-start;
-        margin-bottom: 1rem;
-        gap: 1rem;
-    }
-
-    .task-title {
-        font-weight: 600;
-        color: var(--task-title-color);
-        flex: 1;
-        line-height: 1.4;
-    }
-
-    .task-priority {
-        padding: 0.25rem 0.75rem;
-        border-radius: 12px;
-        font-size: 0.75rem;
-        font-weight: 600;
-        text-transform: uppercase;
-        letter-spacing: 0.5px;
-        white-space: nowrap;
-    }
-
-    /* Priority Colors */
-    .priority-high {
-        background: var(--priority-high-bg);
-        color: var(--priority-high-color);
-        border: 1px solid var(--priority-high-border);
-    }
-
-    .priority-medium {
-        background: var(--priority-medium-bg);
-        color: var(--priority-medium-color);
-        border: 1px solid var(--priority-medium-border);
-    }
-
-    .priority-low {
-        background: var(--priority-low-bg);
-        color: var(--priority-low-color);
-        border: 1px solid var(--priority-low-border);
-    }
-
-    /* Task Details Grid */
-    .task-details {
-        display: grid;
-        grid-template-columns: repeat(auto-fit, minmax(200px, 1fr));
-        gap: 1rem;
-        margin-top: 1rem;
-    }
-
-    .task-detail-item {
-        display: flex;
-        align-items: center;
-        gap: 0.5rem;
-        font-size: 0.875rem;
-        color: var(--task-detail-text-color);
-    }
-
-    .task-detail-label {
-        font-weight: 500;
-        color: var(--task-detail-label-color);
-        min-width: 80px;
-    }
-
-    /* Task Description/Rationale */
-    .task-description {
-        margin-top: 1rem;
-        padding: 1rem;
-        background: var(--task-description-bg);
-        border-radius: 6px;
-        border-left: 3px solid var(--task-description-border);
-        font-size: 0.9rem;
-        line-height: 1.5;
-        color: var(--task-description-color);
-    }
-
-    /* Empty State */
-    .empty-state {
-        text-align: center;
-        padding: 4rem 2rem;
-        color: var(--task-detail-text-color);
-    }
-
-    .empty-state-icon {
-        font-size: 3rem;
-        margin-bottom: 1rem;
-    }
-
-    .empty-state-title {
-        font-size: 1.5rem;
-        font-weight: 600;
-        margin-bottom: 0.5rem;
-        color: var(--objective-title-color);
-    }
-
-    /* Loading Spinner */
-    .loading-spinner {
-        display: inline-block;
-        width: 20px;
-        height: 20px;
-        border: 3px solid var(--task-item-bg-hover);
-        border-radius: 50%;
-        border-top-color: var(--objective-card-border);
-        animation: spin 1s ease-in-out infinite;
-    }
-
-    @keyframes spin {
-        to { transform: rotate(360deg); }
-    }
+    /* ----------------------------------------- */
+    /* --- DARK MODE OVERRIDES (DIRECT) --- */
+    /* ----------------------------------------- */
+    html.dark .okr-title-content { background: linear-gradient(45deg, #e5e7eb, #9ca3af); -webkit-background-clip: text; -webkit-text-fill-color: transparent; background-clip: text; }
+    html.dark .okr-header { color: #e5e7eb; }
+    html.dark .stat-card { background: rgba(28, 28, 28, 0.7); border-color: rgba(50, 50, 50, 0.8); box-shadow: 0 8px 32px rgba(0,0,0,0.6); }
+    html.dark .okr-content { background: #0d1117; box-shadow: 0 20px 40px rgba(0,0,0,0.6); }
+    html.dark .okr-objective { background: #161b22; border-left-color: #3b82f6; box-shadow: 0 8px 24px rgba(0,0,0,0.5); }
+    html.dark .okr-objective:hover { box-shadow: 0 10px 28px rgba(0,0,0,0.6); }
+    html.dark .objective-header { border-bottom-color: #30363d; }
+    html.dark .objective-title { color: #58a6ff; }
+    html.dark .meta-item { color: #8b949e; }
+    html.dark .key-result { background: #161b22; border-color: #30363d; }
+    html.dark .key-result:hover { border-color: #58a6ff; box-shadow: 0 4px 12px rgba(59, 130, 246, 0.2); }
+    html.dark .kr-header { background: #161b22; border-bottom-color: #30363d; }
+    html.dark .kr-title { color: #c9d1d9; }
+    html.dark .kr-metric { background: rgba(56, 139, 253, 0.15); color: #58a6ff; border-color: rgba(56, 139, 253, 0.4); }
+    html.dark .tasks-title { color: #8b949e; }
+    html.dark .task-item { background: #21262d; border-color: #30363d; }
+    html.dark .task-item:hover { background: #30363d; border-color: #8b949e; }
+    html.dark .task-title { color: #c9d1d9; }
+    html.dark .task-detail-label { color: #8b949e; }
+    html.dark .task-detail-item { color: #8b949e; }
+    html.dark .task-description { background: #0d1117; color: #8b949e; border-left-color: #30363d; }
+    html.dark .priority-high { background: rgba(248, 81, 73, 0.15); color: #ff7b72; border-color: rgba(248, 81, 73, 0.4); }
+    html.dark .priority-medium { background: rgba(210, 153, 34, 0.15); color: #d29922; border-color: rgba(210, 153, 34, 0.4); }
+    html.dark .priority-low { background: rgba(63, 185, 80, 0.15); color: #56d364; border-color: rgba(63, 185, 80, 0.4); }
+    html.dark .empty-state { color: #8b949e; }
+    html.dark .empty-state-title { color: #58a6ff; }
+    html.dark .loading-spinner { border-color: #30363d; border-top-color: #58a6ff; }
 
     /* Responsive Adjustments */
     @media (max-width: 768px) {
-        .okr-container {
-            padding: 1rem;
-        }
-        .okr-title {
-            font-size: 2rem;
-        }
-        .okr-stats-bar {
-            gap: 1rem;
-        }
-        .stat-card {
-            min-width: 120px;
-            padding: 1rem;
-        }
-        .objective-meta {
-            flex-direction: column;
-            gap: 1rem;
-        }
-        .task-details {
-            grid-template-columns: 1fr;
-        }
-        .task-header {
-            flex-direction: column;
-            align-items: flex-start;
-        }
+        .okr-container { padding: 1rem; }
+        .okr-title { font-size: 2rem; }
+        .okr-stats-bar { gap: 1rem; }
+        .stat-card { min-width: 120px; padding: 1rem; }
+        .objective-meta { flex-direction: column; gap: 1rem; }
+        .task-details { grid-template-columns: 1fr; }
+        .task-header { flex-direction: column; align-items: flex-start; }
     }
     </style>
     <script>
         function syncThemeWithParent() {
             // This script runs inside the Gradio iframe. It looks at the parent page's
             // <html> tag to see if the 'dark' class is present and applies it to the
-            // iframe's own <html> tag, ensuring the CSS variables for dark mode are activated.
+            // iframe's own <html> tag, ensuring the CSS rules for dark mode are activated.
             try {
                 const parentHtml = window.parent.document.querySelector('html');
                 if (parentHtml) {
@@ -527,8 +199,13 @@ def create_enhanced_okr_tab():
             console.error("Could not set up theme MutationObserver:", e);
         }
 
-        // Also run the sync function once when the iframe's content has loaded.
-        document.addEventListener('DOMContentLoaded', syncThemeWithParent);
+        // Also run the sync function once when the iframe's content has loaded,
+        // and a few times after as a fallback for timing issues.
+        document.addEventListener('DOMContentLoaded', () => {
+            syncThemeWithParent();
+            setTimeout(syncThemeWithParent, 50);
+            setTimeout(syncThemeWithParent, 250);
+        });
     </script>
     """