mirror of
https://github.com/nesquena/hermes-webui.git
synced 2026-05-25 11:10:18 +00:00
nesquena-hermes
cf23d47e2d
Opus advisor on stage-371 caught three issues during pre-release review: 1. RTL salvage missed KaTeX math (display equations + inline LaTeX), diff blocks, CSV tables (column order must read left-to-right regardless of chat direction), and .skill-file-path. The first salvage commit only covered pre/code/kbd/samp/tt and tool-call bodies. Added a second force-LTR block covering: .katex, .katex-block, .katex-display, .katex-html, .katex-inline, .diff-block (+children), .csv-table-wrap, .csv-table (+children), .skill-file-path. Severity: KaTeX is the most user-visible gap — any user rendering math under RTL would see flipped equations. 2. Quota chip @media (max-width:1400px) hide rule conflicted at exactly 1400px with the existing @media (min-width:1400px) .messages-inner rule — chip was hidden AT the wide-desktop boundary where it should first appear. Changed to (max-width:1399.98px). Visually verified at 1400px: chip now correctly visible there. 3. Dead .icon-btn.provider-quota-chip selector — chip never has icon-btn class. Removed. Test added: test_rtl_math_and_tables_stay_ltr (pins the 4 new LTR surfaces). Also removed dead code in test_rtl_code_blocks_stay_ltr (unused code_block variable). Per stage-fix protocol: SHIP-with-followup applied on the stage rather than the source PR, since #2409 is already merged-into-stage and nesquena-approved. Stage-371 review-bypass batch path still holds.
