jens/brewpi
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix fire gap: restore flame positions, increase clip topBuffer to 20px

Browse Source 
Flames back at y=177 (tips touching pot bottom at y=174), viewBox
back to 255. topBuffer raised from 10 to 20px to cover flicker
overshoot without clipping the tips.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DGKe8JzTM2Gr1fdx9wYXTi
This commit is contained in:
jens
2026-06-26 23:02:11 +02:00
co-authored by Claude Sonnet 4.6
parent 7809cb1938
commit c92c41230d
2 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
web/app.js
+3 -3
View File
@@ -294,8 +294,8 @@ function updateFireVisualization() {
const g = document.getElementById('fire-group');
const clipRect = document.getElementById('fire-clip-rect');
if (!g) return;
const fireBase = 256; // y of flame base (all paths end here)
const fireTip = 185; // y of tallest flame tip
const fireBase = 248; // y of flame base (all paths end here)
const fireTip = 177; // y of tallest flame tip
const fireH = fireBase - fireTip; // 71px total flame height
if (heaterPowerIst <= 0) {
g.style.opacity = '0';
@@ -304,7 +304,7 @@ function updateFireVisualization() {
const intensity = Math.min(1, heaterPowerIst / heaterMaxPower);
// Always show the lower 35% (base), scale upper 65% with intensity
const visibleH = (0.35 + 0.65 * intensity) * fireH;
const topBuffer = 10; // headroom for flicker scaleY overshoot
const topBuffer = 20; // headroom for flicker scaleY overshoot (~12% of height)
clipRect.setAttribute('y', (fireBase - visibleH - topBuffer).toFixed(1));
clipRect.setAttribute('height', (visibleH + topBuffer + 2).toFixed(1));
g.style.opacity = String((0.6 + 0.4 * intensity).toFixed(2));