🎨 Palette: Improve clarity of statistical inputs with concrete tooltip examples#83
🎨 Palette: Improve clarity of statistical inputs with concrete tooltip examples#83
Conversation
…ters Added `help` text to `adv_pct`, `vol_q_high`, and `bt_cost` inputs in the sidebar. This provides users with concrete numerical examples for abstract units (like basis points and quantiles) to reduce cognitive load and improve usability. Added missing `matplotlib` dependency required for rendering background gradients in the dataframe UI. Co-authored-by: aarjava <218419324+aarjava@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
Fixed formatting issues in `src/dashboard.py` specifically for lines modified in the previous commit (`adv_pct`, `vol_q_high`, `bt_cost`). Adjusted trailing commas and parentheses to align with Black's output while intentionally not running a global file format to respect PR size constraints. Co-authored-by: aarjava <218419324+aarjava@users.noreply.github.com>
Reverted accidental, out-of-scope Black formatting changes in `src/dashboard.py` (lines 670-723) and removed the unrelated `matplotlib` dependency addition from `requirements.txt`. Ensured that only the lines specifically related to the `adv_pct`, `vol_q_high`, and `bt_cost` tooltips are modified and properly formatted according to Black rules, respecting strict PR size limits. Co-authored-by: aarjava <218419324+aarjava@users.noreply.github.com>
Reverted accidental out-of-scope Black formatting changes in `src/dashboard.py` (lines 670-723) and removed the unrelated `matplotlib` dependency addition from `requirements.txt`. Ensured that only the lines specifically related to the `adv_pct`, `vol_q_high`, and `bt_cost` tooltips are modified and properly formatted according to Black rules, respecting strict PR size limits. Co-authored-by: aarjava <218419324+aarjava@users.noreply.github.com>
1 participant
💡 What: Added descriptive
helptooltips with concrete numerical examples to statistical and financial input widgets (adv_pct,vol_q_high,bt_cost) in the sidebar. Also addedmatplotlibtorequirements.txtto fix a runtime rendering error withpandas.style.🎯 Why: Users in financial applications often face cognitive load when parsing abstract parameters like "bps" (basis points) or scaled quantiles. Providing concrete examples directly in the UI (e.g.,
10 bps = 0.10%) immediately anchors their understanding.♿ Accessibility: Improves cognitive accessibility by explicitly defining technical domain terminology in the context where it is used.
📸 Before/After: No visual change to the main layout; small '?' tooltip icons now appear next to the enhanced labels.
PR created automatically by Jules for task 5753560273878920205 started by @aarjava