Address code review

Author: pablogsal

Lib/profiling/sampling/flamegraph.css

@@ -228,7 +228,7 @@ body {
 }
 
 .thread-filter-wrapper {
-  display: inline-flex;
+  display: none;
   align-items: center;
   margin-left: 16px;
   background: white;

Lib/profiling/sampling/flamegraph.js

@@ -536,10 +536,6 @@ function initThreadFilter(data) {
   const threadWrapper = document.querySelector('.thread-filter-wrapper');
 
   if (!threadFilter || !data.threads) {
-    // Hide thread filter if no thread data
-    if (threadWrapper) {
-      threadWrapper.style.display = 'none';
-    }
     return;
   }
 
@@ -555,9 +551,9 @@ function initThreadFilter(data) {
     threadFilter.appendChild(option);
   });
 
-  // Hide filter if only one thread or no threads
-  if (threads.length <= 1 && threadWrapper) {
-    threadWrapper.style.display = 'none';
+  // Show filter if more than one thread
+  if (threads.length > 1 && threadWrapper) {
+    threadWrapper.style.display = 'inline-flex';
   }
 }
 

Lib/profiling/sampling/sample.py

@@ -754,7 +754,7 @@ def main():
         "--mode",
         choices=["wall", "cpu", "gil"],
         default="wall",
-        help="Sampling mode: wall (all threads), cpu (only CPU-running threads), gil (only GIL-holding threads)",
+        help="Sampling mode: wall (all threads), cpu (only CPU-running threads), gil (only GIL-holding threads) (default: wall)",
     )
 
     # Output format selection