From 1a20f0a15baa37ac0e611fcecb3ed7bfb818423c Mon Sep 17 00:00:00 2001
From: Giorgio Gilestro <giorgio@gilest.ro>
Date: Thu, 28 May 2026 19:13:52 +0200
Subject: [PATCH] mobile: tag Qty/Avg cells in JS-rendered portfolio table
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The portfolio table is rendered client-side in portfolio.js (not by
the partials/portfolio.html Jinja template, which is unused for this
view). The previous commit's mobile-hide class made it into the
template but never reached the actual DOM. Adding the class to the
JS-emitted <th> and <td> strings so .dense .mobile-hide { display:
none } actually picks them up at ≤480px.
---
 app/static/js/portfolio.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/static/js/portfolio.js b/app/static/js/portfolio.js
index ac2c175..b7f820f 100644
--- a/app/static/js/portfolio.js
+++ b/app/static/js/portfolio.js
@@ -303,8 +303,8 @@
       return '<tr>' +
         '<td class="label">' + esc(p.yahoo_ticker) + '</td>' +
         '<td>' + esc(p.name || '') + '</td>' +
-        '<td class="num">' + fmt(p.qty, { maximumFractionDigits: 6 }) + '</td>' +
-        '<td class="num neu">' + fmt(p.avg_cost) + '</td>' +
+        '<td class="num mobile-hide">' + fmt(p.qty, { maximumFractionDigits: 6 }) + '</td>' +
+        '<td class="num neu mobile-hide">' + fmt(p.avg_cost) + '</td>' +
         '<td class="num">' + lastDisplay + fxBadge + '</td>' +
         '<td class="num ' + cls(p._ppl) + '">' + signed(p._ppl) + '</td>' +
         '<td class="num ' + cls(p._ppl_pct) + '">' + pct(p._ppl_pct) + '</td>' +
@@ -365,7 +365,7 @@
       '<table class="dense">' +
         '<thead><tr>' +
           '<th>Ticker</th><th>Name</th>' +
-          '<th class="num">Qty</th><th class="num">Avg</th>' +
+          '<th class="num mobile-hide">Qty</th><th class="num mobile-hide">Avg</th>' +
           '<th class="num">Last</th><th class="num">P/L</th>' +
           '<th class="num">%</th>' +
           '<th></th>' +