Merge pull request #38 from aldrin-labs/copilot/fix-37

Fix broken data structure visualization in orderbook-visualizer.html

Author: 0xrinegade

docs/orderbook-visualizer.html

@@ -539,15 +539,15 @@
         .execution-stats {
             display: grid;
             grid-template-columns: repeat(4, 1fr);
-            gap: 0.2rem;
-            margin-bottom: 0.3rem;
+            gap: 0.15rem;
+            margin-bottom: 0.2rem;
         }
 
         .stat-card {
             background: rgba(255, 255, 255, 0.7);
             border: 1px solid rgba(0, 0, 0, 0.1);
             border-radius: 6px;
-            padding: 0.2rem;
+            padding: 0.15rem;
             text-align: center;
             transition: all 0.3s ease;
             position: relative;
@@ -575,10 +575,10 @@
         }
 
         .stat-value {
-            font-size: 0.85rem;
+            font-size: 0.75rem;
             font-weight: bold;
             color: var(--accent-blue);
-            margin-bottom: 0.05rem;
+            margin-bottom: 0.03rem;
             transition: all 0.3s ease;
         }
 
@@ -593,43 +593,48 @@
         }
 
         .stat-label {
-            font-size: 0.6rem;
+            font-size: 0.55rem;
             color: var(--ink-medium);
             text-transform: uppercase;
             letter-spacing: 0.5px;
             line-height: 1.1;
         }
 
         .algorithm-steps {
-            margin: 0.3rem 0;
-            flex: 1;
+            margin: 0.2rem 0;
+            flex: 0 0 auto;
             min-height: 0;
+            max-height: 80px;
+            overflow: hidden;
+            display: flex;
+            flex-direction: column;
         }
 
         .steps-container {
             background: rgba(255, 255, 255, 0.5);
             border: 1px solid rgba(0, 0, 0, 0.1);
             border-radius: 6px;
-            padding: 0.3rem;
-            min-height: 120px;
-            max-height: 200px;
+            padding: 0.15rem;
+            min-height: 50px;
+            max-height: 60px;
             overflow-y: auto;
-            font-size: 0.7rem;
-            line-height: 1.2;
+            font-size: 0.58rem;
+            line-height: 1.0;
+            flex: 1;
         }
 
         .execution-step {
             display: flex;
             align-items: center;
-            padding: 0.15rem 0.2rem;
-            margin: 0.05rem 0;
+            padding: 0.08rem 0.1rem;
+            margin: 0.01rem 0;
             background: rgba(255, 255, 255, 0.8);
-            border-radius: 3px;
+            border-radius: 2px;
             border-left: 2px solid var(--accent-blue);
             animation: stepAppear 0.5s ease-out;
-            font-size: 0.65rem;
-            line-height: 1.1;
-            min-height: 18px;
+            font-size: 0.55rem;
+            line-height: 1.0;
+            min-height: 14px;
             transition: all 0.3s ease;
         }
 
@@ -696,17 +701,20 @@
         }
 
         .step-time {
-            font-size: 0.5rem;
+            font-size: 0.45rem;
             color: var(--ink-light);
-            margin-left: 0.2rem;
+            margin-left: 0.15rem;
             flex-shrink: 0;
             white-space: nowrap;
         }
 
         .skip-list-visualization {
-            margin: 0.3rem 0;
-            flex: 1;
+            margin: 0.2rem 0;
+            flex: 3;
             min-height: 0;
+            overflow: hidden;
+            display: flex;
+            flex-direction: column;
         }
 
         .skip-list-container {
@@ -988,7 +996,7 @@
 
         /* Data Structure Specific Visualizations */
         .data-structure-viz {
-            margin: 0.3rem 0;
+            margin: 0.2rem 0;
             flex: 1;
             min-height: 0;
         }
@@ -997,9 +1005,9 @@
             background: rgba(255, 255, 255, 0.5);
             border: 1px solid rgba(0, 0, 0, 0.1);
             border-radius: 6px;
-            padding: 0.3rem;
-            min-height: 60px;
-            max-height: 120px;
+            padding: 0.4rem;
+            min-height: 100px;
+            max-height: 200px;
             overflow: auto;
         }
 
@@ -1632,7 +1640,14 @@ <h3>📈 Recent Trades
                     ].sort((a, b) => a - b);
 
                     if (allPrices.length === 0) {
+                        // If no prices, clear the data structure and show empty state
+                        this.skipListNodes = [];
+                        this.btreeNodes = [];
+                        this.hashmapBuckets = [];
+                        this.sortedArray = [];
+                        this.rbTreeNodes = [];
                         this._generatingDataStructure = false;
+                        this.updateDataStructureDisplay();
                         return;
                     }
 
@@ -1876,11 +1891,6 @@ <h3>📈 Recent Trades
                 this.generateDataStructureFromOrderbook();
             }
 
-            generateSkipListFromOrderbook() {
-                // Use the new unified method
-                this.generateDataStructureFromOrderbook();
-            }
-
             updateSkipListDisplay() {
                 // Use the new unified method
                 this.updateDataStructureDisplay();