SiteMeasure Pro

v6.870 Installer exports: MAIN values baked via LibreOffice headless recalc before formula strip — calc columns now carry real numbers (fixes blank-columns installer workbook)

v6.854: ROLLER SHADES sheet7 replaced with Excel-authored sample (canonical OOXML child order; ends Replaced-Part/Load-error class)

v6.844: SHADES - OUTSIDE mount + side-channel toggles redesigned as DRAPES-style buttons between title and W/L/Chain; mountLoc field; gaps 30px; nav arrows grouped; new windows seed single shade; OUTSIDE MOUNT shows 2px blue cell border

v6.841: SHADES C3 - stripped drape STEP 2 machinery from shades view (formula/returns/cutouts/center-point/multi-window); shades view now Shades + SIDE CHANNELS only

v6.840: SHADES - CHAIN / MOTOR label tightened to CHAIN/MOTOR (phone title crowding)

v6.839: SHADES un-share C2 cutover - shade fields bind to shadeList (sInp/updateShadePanel/_shP); mirror removed; blank seed; toggleScOn+chain to panel. Room holds drape+shade independently

v6.838: SHADES un-share C1 - added updateShadePanel + sInp panel-bound helpers (additive, no behavior change; cutover in C2)

v6.837: SHADES - CHAIN (LOC) label renamed CHAIN / MOTOR (motorized handled vendor-side; L/R toggle unchanged)

v6.836: SHADES - edit-view DELETE removes current shade panel when multiple (deleteShade); deletes room only on single panel

v6.835: SHADES - removed clone-residue sheer block (Right Sheer + sheer-pair) from shades view; only Shades + SIDE CHANNELS render

v6.834: SHADES nav arrows blue-to-yellow; Add Shade + Add Window buttons yellow bg black text

v6.833: SHADES multi-panel UI - SHADES (n) title + back/forward arrows (setShadeIdx) + Add Window retargeted to + Add Shade (addShadePanel)

v6.832: SHADES multi-panel model - shadeList/shadeIdx + ensure/sync/load/setShadeIdx/addShadePanel + saveSection sync (no UI yet)

v6.831: SIDE CHANNELS scOn decoupled from drapePair (new toggleScOn + sec.data.scOn) - fixes side-channel toggle mutating drape pair/PTAC/clearing fields

v6.830: removed unused yellow + FAB (smartAdd) - never used, avoids new-user confusion

v6.829: SIDE CHANNELS - NO button relabeled (select)

v6.828: SHADES - CHAIN LOCATION renamed CHAIN (LOC); chain toggle text RIGHT/LEFT (default RIGHT); SHADES + SIDE CHANNELS field labels blue #8fbdf7 (scoped .sh-fld class)

v6.781 — Fix OVERVIEW mergeCells corruption (Excel "Removed Records: Merge cells" repair on open). Two coupled defects: (1) PTAC card on wider variants collided with a template block merge at the same column — clone loop now skips a ref whose anchor is already claimed, mirroring the overflow loop guard; (2) v6.704.3 empty-merge strip ran after the count attr was set and postprocess does not recompute it — count now recomputed to actual element total after the strip. Merge set otherwise unchanged; production export opens clean.

v6.740 — Auto-detect Brave/Chrome download-suppression. After Excel export, if browser silently blocks download (multiple-auto-downloads heuristic demotes site permission), modal appears with one-time fix instructions: tap lock icon → set Automatic downloads to Allow. localStorage flag sm_dl_ok tracks first successful download; once set, modal never re-shows. New installer onboarding handled automatically — no manual setup required.

v6.733 — Two-tier property permissions. view_permissions.can_change_settings resurrected. Right-side checkbox per property in User Permissions panel: unchecked = "Read only" (default), checked = "Full access". canEditProperty() helper gates update_property, delete_property, save_section, delete_section. Read endpoints unchanged (any VP row = view access). Closes Benoit Save-Failed loop.

v6.732 — Data-driven orphan strip. _strip_orphan_cols whitelist built from populated treat_cols (slots actually written) instead of full variant declaration. Fixes bordered empty cells under photos on sections underfilling variant capacity (Duvernay 2-card SHEER+DRAPES under 3-treat-col SLR+DPAN).

v6.731 — Universal orphan strip. v6.729/v6.730 only stripped ro==0/2/4 handler branches. Unhandled row offsets (ro==1/3/5/6/7) still cloned wider-variant ghost cells, visible at row 20 of Duvernay sections (between title row and label row) as styleless K/L/N/O/Q/R cells creating cell-border artifacts behind photo. Fix: drop ro==4 guard on pre-append strip so it fires for every row offset. ro==0 and ro==2 handler-branch strips kept (harmless double-call).

v6.730 — Extend v6.729 orphan strip to ro==0 title row. Duvernay export revealed title rows clone styleless ghost cells (K,L,N,O,Q,R) from wider-variant template into narrower-variant section renders, photo anchors land on top. Same _strip_orphan_cols helper called now at end of ro==0 block. ro==2 and ro==4 strips from v6.729 unchanged.

v6.729 — Orphan cell strip in _inject_client_xml ro==2 / ro==4. New nested helpers _blank_cell and _strip_orphan_cols remove wider-variant clone artifacts at T/U/W/X that leaked styled-empty boxes (s=433) and label text (WIDTH/LENGTH) into narrower-variant section renders. Bypasses v6.665 s_attr preservation surgically. Production legacy variant-row path unchanged otherwise. Toggle states unaffected (geometric+horizontal-flow stay False).

v6.724 — Revert USE_GEOMETRIC_COMPOSITION to False. v6.723 geometric output rendered broken in BW Calgary export. Production back on legacy variant-row clone path. _compose_section_block dead code preserved.

v6.723 — USE_GEOMETRIC_COMPOSITION flipped to True. ALL OVERVIEW section emission now driven by _compose_section_block. Legacy variant-row clone path bypassed for every section. Resolves: 38% payload overflow (silent cutout truncation), 8 unmapped p_keys, orphan template labels causing photo overlap (e.g. BW Calgary 1540.3 T-col photo/label collision). Cards render left-to-right sequentially per section data; photo lands past rightmost populated card.

v6.722 — USE_GEOMETRIC_COMPOSITION flag added (default False). When True, _compose_section_block drives all section emission, bypassing legacy variant-row clone path. Flag False = production byte-identical to v6.721. v6.723 will flip to True.

v6.721 — OVERVIEW geometric composition function added as dead code. _compose_section_block(sec, photo_count, style_catalog, row_anchor) defined module-level. Smoke test invokable via `python3 inject.py --smoke-v6721` — passes against locked BW Calgary payload (narrow + wide + photo + anchor + catalog). Zero callers in production path; _inject_client_xml + main() byte-identical to v6.720. Chapter v6.721-v6.726 first commit.

v6.720 — Hyperlink photo layer card-col scan widened from literal {E,H,K,N,Q} to stride-aware predicate (col-5) % 3 == 0. Matches canonical card-position grid (E, H, K, N, Q, T, W, Z, AC...) per v6.706.1 stride lock. Unblocks cutout column runtime extension in v6.721. Render byte-identical pre/post. ✅ Elements locked.

v6.718 — CT-stripped fallback at _no_ct_key (line ~780) now logs loud when it fires. Fallback path preserved — zero render-output change. Surfaces exact (room, p_key) pairs where template ships no CT-variant, feeding the v6.719 variant-widening inventory. Conservative Option C: kept the safety net, made it visible.

v6.717 — Fixed photo position stride drift in _reposition_photos. Marker comment now lands at rightmost-card+3 (v6.706.1 stride lock) instead of +2, matching the Path 3 drawing-anchor emitter. Docstring refreshed with canonical card sequence. Inert XML comment fix — no visual export change, closes a lying breadcrumb for future consumers.

v6.704.4 [SR]: SESSION_RESUME.md append documenting v6.704 chapter close (v6.704 / v6.704.1 / v6.704.2 / v6.704.3 — legacy photo system stripped to clean canvas, dark fills cleared, leftover 2x2 PHOTO grid merges removed). CANONICAL CARD SEQUENCE relocked in SR.md (prior chat sessions lost the full per-layer cutout mapping): ROOM/TAG, VALANCE, SHEER, DRAPES, WIN 2/3/4, VALANCE/DRAPE/SHEER RETURNS, VALANCE/DRAPE/SHEER (L+R) TOP|BOTTOM CUTOUTS, PTAC SIZES (future), PTAC SKETCH, PHOTOS. Cards push right; no empty placeholders. v6.705 design brief carried forward. NEXT_SESSION_PROMPT.md written with mandatory session-open recon block + working style locks. Three carry-forward discoveries: SST to inlineStr conversion location unknown, CUTOUTS data never populates on 28 rooms (pre-existing), v6.700-v6.703 session log entries missing.

v6.704.3: inject.py — remove leftover PHOTO grid merges + clear styleIds on inner cells. After v6.704.2 stripped PHOTO label text, 88 leftover 2-col x 2-row mergeCell entries persisted (87 at col N, 1 at col K row 11), creating visible empty bordered shells on every PHOTO position row. Inner cells inside those merges retained template styleIds (s=569/570/571/572 etc) drawing the border styling. Fix: walk all mergeCell entries, identify 2x2 merges where top-left cell has no <v> or <is> content (= PHOTO grid leftover, not data card), remove the merge entry, strip styleIds from all 4 inner cells. Discriminator validated: 919 populated 2x2 merges (legitimate data cards at E/H/K/N/Q/T/W) untouched; 88 empty 2x2 merges (PHOTO leftovers at N + K11) cleaned. 39-line addition inside _inject_client_xml after v6.704.2 strip pass.

v6.704.2: inject.py — PHOTO label strip drops ALL cell attributes (no styleId carry-over). v6.704 strip preserved s="N" on emptied PHOTO cells, which inherited the dark PHOTO label fill (s=862/863/864) — rendered as visible dark rectangles down every PHOTO column. v6.704.2 emits self-closing <c r="REF"/> with no style/type attrs, rendering as plain empty cells. Two-line surgical change (comment + lambda) inside the strip regex. AST OK, same line count, same byte count. Clean canvas verified: zero PHOTO labels, zero residual fills, zero hyperlinks. Ready for v6.705 image deploy.

v6.704: inject.py — strip legacy photo system (clean canvas before v6.705 image deploy). Delete suppressed v6.666 text-label emit if-False block, suppressed Step D rels build, _path3_photo_anchors collection, Path 3 emit+splice+hyperlinks rebuild, caller-side _path3_payload consumption (image bytes / image rels / hyperlink rels). Collapse _inject_client_xml return 4-tuple to 3-tuple. Add PHOTO 1/2/3/4 label strip pass at end of _inject_client_xml — inlineStr cells with PHOTO [1-4] text replaced with self-closing empty cells preserving styleId. _emit_photo_thumbnails + _get_photo_thumb_bytes helpers retained on disk for v6.705 reuse. .thumb-cache/ untouched (gitignored in v6.706). Net -160 lines / -9741 bytes.

v6.667 [SR]: session continuation block — v6.665 _write empty-cell styleId preservation (closed 3 backlog items: C13/C149 borders, AA-end T/U/W/X, 14 AA-end with-returns), v6.666 hyperlink emission placement-aware partial fix (covers ~80% of rooms; K-column and AA/Z-column edges deferred to Path 3 supersession). Path 3 architectural pivot identified and design-briefed: embedded compressed thumbnails on OVERVIEW replace text-label hyperlink system entirely. v6.666 is FINAL text-label hyperlink commit. Path 3 scoped as v6.700-era commit chain.

v6.666: inject.py — photo hyperlink emission now placement-aware. Root cause of partial/missing hyperlinks: v6.622 emitter hardcoded N/O but _reposition_photos places PHOTO labels at (rightmost CARD_COL E/H/K/N/Q in the room row) + 3, so rooms without DRAPES had photos at H or K and got zero hyperlinks. Also only the first link emitted per room (single-element check on _room_rids instead of full 4-position loop). Fix: replicate _reposition_photos column math inside the emitter (scan inlineStr cells, find max CARD_COL per row, target = +3), loop all 4 photo positions stamping links 1..N based on actual photo count from payload. Matches placement function exactly so emission tracks placement on every section variant. Note: v6.700-era candidate — embedded thumbnails on OVERVIEW (Path 3) would supersede text-label hyperlinks entirely. v6.666 is bridge for current text-label workflow.

v6.665: inject.py _write — preserve styleId on empty-value cells. Root cause of C13/C149 perimeter border strips post-v6.663: line 398-399 returned before+after on empty val, deleting the <c> element entirely and discarding the template styleId (and with it every border v6.663 wrote into the template). Fix: emit self-closing <c r="REF" s="N"/> when styleId existed, so template borders/fills survive on every inject-cleared cell. Likely silently closes AA-end T/U/W/X bordered cells bug + 14 AA-end with-returns backlog items (same root mechanism). Two-line surgical change inside _write.

v6.660: inject.py + inject_view.py — drop dead BW Calgary lookup, activeTab="0" opens to OVERVIEW. Collapsed 3-line client-tab lookup (property_name first-attempt always returned None, fallback to literal 'Best Western - Calgary') to single direct 'OVERVIEW' lookup. activeTab block flipped "1" (MAIN) to "0" (OVERVIEW). 2 stale comments updated. inject_view.py mirrors inject.py changes. 9 mutations total, zero active residue. Line 935 docstring archaeology preserved. Pairs with v6.659 template rename.

v6.657: app.html — orphan cleanup sweep. Removed dlPostForm form (lines 414-418, dead form from abandoned POST-based download approach) + dl-toast (HTML 328-332 + CSS 315-318, dead toast UI from abandoned async-download approach) + line 1480 consumer (var dlForm=getElementById('dlPostForm') reference). Recon-confirmed zero consumers across app.html JS/CSS and api.php/dl.php/inject.py. 15 lines deleted total. Closes carried dead-architecture thread from v6.654. Pre-edit Python sweep aborted on first attempt — Block 3 had strict line-range assertions (line 417 expected blank, was actually the submit button line — recon display indentation misread as blank line) contradicting a defensive -scan in the same block. Assertion fired before file mutation, no damage. Fixed by removing strict asserts, keeping defensive scan. Lesson: strict line-range checks and defensive content-scans should not coexist in one assertion block — strict asserts short-circuit before defensive scan runs. Pick one approach per block.

v6.654: app.html — Excel download single-fire fix. Root cause: window.location.href=jdata.url navigated the top browser window to the download URL on every export, corrupting the JavaScript context — _exportRunning flag, currentProperty, sections array, button event bindings entered undefined state after navigation. First click worked, subsequent clicks failed until page refresh restored fresh JS context. Fix: replaced top-window navigation with standard anchor-click pattern — dynamic <a> element with download attribute appended to body, .click() fires browser download handler WITHOUT navigating, then element removed. JS context preserved, button stays live, clicks N times = downloads N times. Single button, no second-button workaround. Orphan dlPostForm + dl-toast elements (lines 414, 328) noted as dead architecture from abandoned download approaches; deferred to separate cleanup commit.

v6.653: api.php — save_section endpoint hardened. Wrapped UPDATE + INSERT branches in try/catch. Added rowCount() audit: UPDATE returning 0 affected rows now logs UPDATE_NOOP (catches silent-success-but-nothing-changed class — symptom of v6.631 prove-out intermittent "save failed"). INSERT returning lastInsertId 0 logs INSERT_NOID. PDOException catch logs full context (user, property_id, section_id, sqlstate, exception message, input key list — NOT values, to avoid PII leak) to LOG_DIR/save_section_debug.log, returns http 500 + structured {success:false,error:"Save failed",detail:"logged"} response. Root cause architectural defect addressed: prior endpoint had zero error handling, PDO throw = raw 500 with no forensic capture. Existing success echo paths preserved. Matches existing log convention (prop_debug.log, vp_debug.log).

v6.652: app.html — final dead CSS sweep. Removed 3 orphan CSS rules (.user-list, .user-row, .user-row.inactive) carried from the deleted #usersView block in v6.644. Recon confirmed zero consumers: no HTML class= usage, no JS innerHTML/template literal injection, no getElementsByClassName/querySelector references, no api.php server-side use. Cosmetic cleanup, zero functional impact. Closes carried cleanup thread from v6.650.

v6.650: app.html — dead CSS sweep. Removed 4 orphan CSS rules (.users-wrap, .users-header, .users-title, .users-back) left behind when v6.644 deleted #usersView block. .user-add-form retained (still in use by Add User form inside User Permissions accordion). Also removed 3 orphan comments: /* ADMIN GEAR */, /* USER MANAGEMENT */, . Cosmetic cleanup, zero functional impact. Closes carried cleanup thread from v6.644.

v6.649: api.php + app.html — new change_password endpoint + Change Password button in User Permissions accordion. api.php: admin-only endpoint, validates user_id + new_password (min 6 chars), confirms user exists, password_hash(PASSWORD_DEFAULT), UPDATE users SET password_hash + auth_token=NULL (forces re-login). app.html: vpChangePwWrap div inserted between vpUserSelect and vpPropList; loadViewPerms now shows/hides the wrap based on uid; showChangePwForm renders inline password input + Save/Cancel; submitChangePw posts to api, flashSave on result. Button surfaces only when a user is selected from the dropdown.

v6.647: app.html + api.php — retire all can_change_settings reads and writes ahead of v6.648 DDL drop. api.php: check_auth no longer injects field into user payload; 3 CREATE TABLE statements stripped of column; get_view_permissions drops ALTER + SELECT col + response field + log mention; save_view_permissions drops $canChange + ALTER + 3-col INSERT. app.html: _canChangeSettings global removed at both write sites (line 849 full delete, line 874 inline strip); save_view_permissions JS payload drops can_change_settings:1. Column is now structurally orphan. Permission model unchanged (admin = full edit everywhere, non-admin = full edit on own + shared).

v6.645: app.html — fix ReferenceError in loadViewPerms blocking property checkbox list render. Latent v6.631 bug exposed by v6.644: when v6.631 deleted the mockup Can Change Settings button, the JS line consuming btn (var declaration removed, if(btn){...} block left behind) became a dead reference. Legacy loadViewPerms masked it until v6.644 promoted mockup to canonical. Surgical deletion of the orphan if(btn){...} line. Property checkbox list now renders for Benoit + Michael as designed.

v6.644: app.html — mockup → canonical rename-commit. Dropped -mockup suffix on all IDs/functions/classes/localStorage keys. Deleted legacy: .admin-gear CSS + button, #usersView block (Manage Users view), #adminLoginPanel, #adminPanel, legacy #viewPermsSection block, legacy initViewPerms/loadViewPerms/toggleVpCanChange/saveAllViewPerms/showAdminLogin/adminAuth/adminLogout/loadUsers/renderUsers/toggleUser/deleteUser/addUser functions, showView usersView line, isSuper var, ADMIN USER MANAGEMENT comment. End state: single User Permissions section in Settings, no mockup label, no gear button, no separate Manage Users view. Add User form lives inside User Permissions accordion.

v6.641: app.html — lighten dropped-down section header backgrounds by 15%. Line 142 adds background:#292E3E override (was inherited var(--surface2) #242836). Scoped to .room-group-body .section-card .section-header so only expanded-room child rows (MEETING.1-4) change; standalone rooms (OFFICE) and parent expanded headers (MEETING) keep original surface2. Yellow text contrast preserved. Pairs with v6.640 border lighten for full visual separation pass.

v6.640: app.html — lighten border on dropped-down section headers (rgba alpha 0.08 -> 0.15). Visual separation between expanded-room child rows (e.g. MEETING.1-4 under MEETING) without altering container background. Path B from session: surgical border-only change, container untouched to preserve yellow text contrast.

v6.639: formalize photos/Best_Western_-_Calgary_201.jpg deletion. File removed pre-session, room not needed. Per HARD_RULES rule 23 post-amendment (photos = runtime media). Closes orphan delete tracked since session open.

v6.638: untrack __pycache__/inject.cpython-312.pyc from git index. File remains on disk, regenerated by Python normally. Was tracked pre-v6.636 before __pycache__/ landed in .gitignore. Explicit git rm --cached required.

v6.637: HARD_RULES rule 23 amendment — photos are runtime media, not source. Aligns rule text with v6.636 .gitignore reality. 320 existing tracked photos retained in git history; new uploads gitignored. Server-level backup is the safety net.

v6.636: .gitignore overhaul — photos/ + .bak* + __pycache__/ + .pyc gitignored. B1 decision: photos become runtime media. Backup workflow artifacts no longer pollute git status. Lowercase .bak patterns added (existing pattern only covered uppercase .BAK).

v6.635 [GOV]: HARD_RULES.md — Rule E REVIEW GATE appended. Multi-file/multi-commit sequences require explicit STOP between draft and commit. 3-step protocol: draft block, review block, commit-only block.

v6.634 [GOV]: GIT_WORKFLOW.md seeded — pre-edit/edit/verify/commit/bust discipline. Idempotency markers reference body not filenames (root cause from 9578b0a). Schema changes own commit boundary.

v6.633 [GOV]: CONVENTIONS.md seeded — repo/file scope/backend/data flow/permission model/deploy verbs. Retroactive landing (chronologically last but version-line sequential).

v6.632 [GOV]: HARD_RULES.md seeded — locked rules + Rules A-D (served-path verification, credential exposure ban, recon silence, photo lifecycle). Idempotency marker rule. Banned words. Cross-system boundary.

v6.631: app.html — remove view-only concept from mockup permission UI. Badge VIEW ONLY -> SHARED (provenance, no permission claim). Mockup Can Change Settings toggle button + toggleVpCanChangeMockup deleted. saveAllViewPerms hardcodes can_change_settings:1 (backend payload preserved, dead column). Admin retains oversight visibility; non-admin = full edit on own + shared.

v6.630: app.html — Removed early-return gate in openSettings() that blocked Settings overlay from opening for non-admin users. Root cause: line conflated "can open Settings" with "can change settings values" — installers need Settings access for Manage Check Boxes & Buttons workflow. Per-field admin-only enforcement deferred to v6.631 (visibility checkboxes). _canChangeSettings variable now set but unread; full removal in v6.631 when replaced by per-property view_permissions.can_change_settings lookup. Bridge: verbal "don't touch installation cost" instruction for Benoit demo until v6.631 ships.

v6.629: app.html — Manage Users relocated into Settings > User Permissions accordion as mockup-suffix partial. New section #viewPermsSection contains Add User collapser (default closed) + verbatim copy of existing perms UI. Four new JS funcs (initViewPerms, loadViewPerms, toggleVpCanChangeMockup, saveAllViewPerms, addUser) wired to -mockup IDs. Original viewPermsSection + initViewPerms etc untouched. Gear button + #usersView + showAdminLogin/adminAuth/adminLogout removal deferred to rename-commit.

v6.627: app.html — photo baseUrl migrated from measure.hotelhospitalcurtains.com to app.sitemeasurespro.com (lines 1733, 2151). Old hostname stays live for permanent 301 redirect — historical xlsx exports have absolute photo URLs baked in.

v6.626: app.html — clientSections payload widened: added per-layer L/R cutout fields (slctW/slctD/srctW/srctD/dlctW/dlctD/drctW/drctD) and cutDiag passthrough. Unblocks inject.py reading TOP/BOTTOM direction and per-layer cutout dimensions for BW page rendering. Inject.py read logic ships v6.627-v6.629.

v6.625: app.html — SR markers added (NEW CHAT PROMPT auto-bumps to last system version; VERSION HISTORY list seeded). Additive only. Unblocks automated SR updates on every future commit.

v6.624: inject.py — CT-suffix selector for cutout sections. Disambiguates cutout/non-cutout template variants at key level; dead tmpl_lct_lookup removed; fallback chain tries with-CT then without-CT.