feat(settings): move theme and appearance editing into a dedicated dialog #21

New Issue

2026-10-11T12:36:14Z

KofTwentyTwo commented

2026-10-11 12:36:14 +00:00

(Migrated from github.com)

Add a dedicated Settings dialog. Theme editing/management and related configuration are currently crowded into View; move configuration into Settings while leaving everyday reading controls in View.

Acceptance:

Provide a discoverable native Settings command and keyboard access.
Organize theme editing/typography/colors/CSS, theme management/order/enable/delete/restore, import/export and global reading appearance in Settings. Include relevant existing configuration without duplicating save logic.
Keep theme selection, zoom, source/outline visibility and other immediate reading actions in View; remove the crowded editing entries after providing equivalent Settings access.
Preserve owned dialogs, validation, cancellation, saved preferences, sibling broadcasts and editor/theme import conflict/newer-schema confirmation behavior. No overlapping dialogs or lost copied drafts.
Use a comfortably sized responsive, accessible native dialog with readable controls at supported window/DPI sizes.
Verify actual entry/navigation/edit-save/cancel and theme/appearance behavior, update documentation and affected UI automation, and retain coverage/build/style gates.

Related: #7, #1. Owner request; implementation and acceptance pending.

Substantial AI assistance: Codex (GPT-6).

Add a dedicated Settings dialog. Theme editing/management and related configuration are currently crowded into View; move configuration into Settings while leaving everyday reading controls in View. Acceptance: - Provide a discoverable native Settings command and keyboard access. - Organize theme editing/typography/colors/CSS, theme management/order/enable/delete/restore, import/export and global reading appearance in Settings. Include relevant existing configuration without duplicating save logic. - Keep theme selection, zoom, source/outline visibility and other immediate reading actions in View; remove the crowded editing entries after providing equivalent Settings access. - Preserve owned dialogs, validation, cancellation, saved preferences, sibling broadcasts and editor/theme import conflict/newer-schema confirmation behavior. No overlapping dialogs or lost copied drafts. - Use a comfortably sized responsive, accessible native dialog with readable controls at supported window/DPI sizes. - Verify actual entry/navigation/edit-save/cancel and theme/appearance behavior, update documentation and affected UI automation, and retain coverage/build/style gates. Related: #7, #1. Owner request; implementation and acceptance pending. Substantial AI assistance: Codex (GPT-6).

KofTwentyTwo commented

2026-10-11 13:23:06 +00:00

(Migrated from github.com)

Implemented locally: File → Settings has native Appearance, Themes and Editors tabs. Reader width/global CSS use a copied Save/Cancel draft, merge against the latest store and broadcast to open readers. Theme/editor operations use existing save logic, close the parent before secondary dialogs and return to the retained parent draft. Six configuration entries were removed from View; editor configuration also moved into Settings. Everyday reading actions stay in View. File/Settings access keys provide keyboard entry; each tab scrolls within available space.

Existing actual editor/theme/transfer/appearance acceptance now uses Settings. A new native case checks retained appearance input after secondary-editor Cancel, then unchanged saved preferences after parent Cancel.

Verified: full Release build has zero warnings/errors;538 core tests pass at100% measured line coverage; format passes. UI-project discovery finds71 cases (58 desktop and13 filesystem). No new native Settings acceptance ran, and the running Debug process predates this change. All work remains uncommitted/unpushed while the existing1Password signing authentication reply is pending. Keep this issue open for actual distributed save/cancel/navigation acceptance.

Substantial AI assistance: Codex (GPT-6).

Implemented locally: File → Settings has native Appearance, Themes and Editors tabs. Reader width/global CSS use a copied Save/Cancel draft, merge against the latest store and broadcast to open readers. Theme/editor operations use existing save logic, close the parent before secondary dialogs and return to the retained parent draft. Six configuration entries were removed from View; editor configuration also moved into Settings. Everyday reading actions stay in View. File/Settings access keys provide keyboard entry; each tab scrolls within available space. Existing actual editor/theme/transfer/appearance acceptance now uses Settings. A new native case checks retained appearance input after secondary-editor Cancel, then unchanged saved preferences after parent Cancel. Verified: full Release build has zero warnings/errors;538 core tests pass at100% measured line coverage; format passes. UI-project discovery finds71 cases (58 desktop and13 filesystem). No new native Settings acceptance ran, and the running Debug process predates this change. All work remains uncommitted/unpushed while the existing1Password signing authentication reply is pending. Keep this issue open for actual distributed save/cancel/navigation acceptance. Substantial AI assistance: Codex (GPT-6).

KofTwentyTwo commented

2026-10-11 14:13:26 +00:00

(Migrated from github.com)

Owner review feedback: the tabbed Settings ContentDialog feels nonstandard. Replace it with a separate native WinUI Settings window using left NavigationView categories and normal Windows 11 controls, headings, spacing and resizing. Routine preferences should be accessible directly; remove the hide/reopen parent-dialog interaction. Preserve checked saves, copied drafts for advanced/theme edits and sibling updates. This supersedes the earlier Pivot/modal Settings acceptance.

KofTwentyTwo commented

2026-10-11 14:40:17 +00:00

(Migrated from github.com)

Owner-rejected modal Pivot Settings is replaced locally with a normal resizable WinUI window: adaptive left NavigationView, independently scrolling native pages, width ToggleSwitch/NumberBox immediate checked persistence and explicit Apply custom CSS. Closing discards unapplied CSS. Saved appearance broadcasts update Settings baselines without overwriting its CSS draft. Theme/editor dialogs and native file pickers belong to the settings window; category changes detach/reuse the real native pages. Existing automation was migrated; a new real native window/resize/navigation/immediate-save/CSS-discard case compiles. Zero-warning Release build passes; no desktop acceptance run. Review-latest.md is open for owner feedback.

KofTwentyTwo commented

2026-10-11 15:05:36 +00:00

(Migrated from github.com)

Added Advanced navigation with native startup splash/duration/update controls, checked immediate saves and shared reader baselines. The standard resizable Settings window remains modeless, with CSS explicitly applied and separate owned theme/editor dialogs. New actual startup/sibling persistence case compiles, not run.

Fresh owner preview: Review-toolbar-splash.md — MarkdownViewr, PID103040, output artifacts/splash-toolbar-review/bin/x64/Release/net10.0-windows10.0.19041.0/MarkdownViewr.Windows.exe. Its separate review profile enables a5000ms splash and Source, and disables startup update checks. Other owner windows were preserved.

Verification: zero-warning/error full Release build;570 core tests pass,100% measured line coverage; formatting and changed-doc lint pass.79 UI-project cases discovered (66desktop/13filesystem; expected78 per installed channel); actual new desktop/distributed acceptance has not run. Launch/document opening succeeded with no warnings in the profile log. Passive Computer Use inspection was unavailable because its native pipe was not found; no mouse/keyboard input occurred.

The original four staged corrections remain unchanged. Signed commits/push still await the pending1Password ready reply; no signing retry, merge, tag, release or Azure change. System Print handoff, PDF open actions, remaining full PRODUCT parity, standards and exact production-signature proof remain open.

Added Advanced navigation with native startup splash/duration/update controls, checked immediate saves and shared reader baselines. The standard resizable Settings window remains modeless, with CSS explicitly applied and separate owned theme/editor dialogs. New actual startup/sibling persistence case compiles, not run. Fresh owner preview: `Review-toolbar-splash.md — MarkdownViewr`, PID103040, output `artifacts/splash-toolbar-review/bin/x64/Release/net10.0-windows10.0.19041.0/MarkdownViewr.Windows.exe`. Its separate review profile enables a5000ms splash and Source, and disables startup update checks. Other owner windows were preserved. Verification: zero-warning/error full Release build;570 core tests pass,100% measured line coverage; formatting and changed-doc lint pass.79 UI-project cases discovered (66desktop/13filesystem; expected78 per installed channel); actual new desktop/distributed acceptance has not run. Launch/document opening succeeded with no warnings in the profile log. Passive Computer Use inspection was unavailable because its native pipe was not found; no mouse/keyboard input occurred. The original four staged corrections remain unchanged. Signed commits/push still await the pending1Password ready reply; no signing retry, merge, tag, release or Azure change. System Print handoff, PDF open actions, remaining full PRODUCT parity, standards and exact production-signature proof remain open.

KofTwentyTwo commented

2026-10-11 16:33:32 +00:00

(Migrated from github.com)

Owner review follow-up — 2026-10-11

The owner still rejects the inner theme-management/settings dialogs, even though they use native WinUI controls. The supplied screenshot shows a crowded scrolling modal containing a list and several rows of actions.

Proposed revision: use Settings pages for sustained theme/editor management and theme editing, with an adaptive list/detail layout, clear action grouping and one main scroll region. Reserve ContentDialog for short naming/confirmation/error decisions and native file pickers for import/export. Retain draft/apply/discard behavior for theme/CSS edits and immediate saves for routine preferences; preserve every existing import conflict, newer-schema, restore and validation behavior.

Microsoft references:

This page-based workflow is a design recommendation informed by the guidance, not a claim that Microsoft mandates a specific settings library.

Additional owner requirement: new windows start with Contents off and Source off. Add both default-visibility controls to Settings; current-window toolbar toggles must not silently rewrite the selected startup defaults. The review fixture's forced Source-on setting is not the desired default. Verify initial install, saved preference, restart/new-window and independent-window behavior.

Recorded as pending during paused review. The current outer Settings window remains locally implemented; the inner-dialog layout is not accepted.

## Owner review follow-up — 2026-10-11 The owner still rejects the inner theme-management/settings dialogs, even though they use native WinUI controls. The supplied screenshot shows a crowded scrolling modal containing a list and several rows of actions. Proposed revision: use Settings pages for sustained theme/editor management and theme editing, with an adaptive list/detail layout, clear action grouping and one main scroll region. Reserve ContentDialog for short naming/confirmation/error decisions and native file pickers for import/export. Retain draft/apply/discard behavior for theme/CSS edits and immediate saves for routine preferences; preserve every existing import conflict, newer-schema, restore and validation behavior. Microsoft references: - https://learn.microsoft.com/en-us/windows/apps/design/controls/dialogs-and-flyouts/dialogs - https://learn.microsoft.com/en-us/windows/apps/develop/ui/controls/navigationview This page-based workflow is a design recommendation informed by the guidance, not a claim that Microsoft mandates a specific settings library. Additional owner requirement: new windows start with Contents off and Source off. Add both default-visibility controls to Settings; current-window toolbar toggles must not silently rewrite the selected startup defaults. The review fixture's forced Source-on setting is not the desired default. Verify initial install, saved preference, restart/new-window and independent-window behavior. Recorded as pending during paused review. The current outer Settings window remains locally implemented; the inner-dialog layout is not accepted.

KofTwentyTwo commented

2026-10-11 19:21:50 +00:00

(Migrated from github.com)

Signed/DCO commit05bf612 is pushed to feat/windows-viewer/PR#3. It fixes the full-page outer body width reset, aligns Contents/Print with the other48px toolbar hit targets, uses the observed exact Open picker filename field1148 and tests the actual outline-depth control. Pixel checks sample the owned WebView; native width/scroll/font assertions are unchanged and now retain precise failure evidence. No cases are skipped.

Local gates: zero-warning Release build,586 core tests,100% measured core line coverage, full formatting and all commit/security/workflow hooks pass. Exact prior CI38166230860: source-hover passes; full native74/97,23failures,0skips; published suite exceeds600seconds before installed outcomes. A focused hosted reader-regressions job now covers these current paths alongside the full suite. Native/distributed/signature/review/release acceptance is still pending; no production release published.

Owner preview PID55472 remains responsive and untouched. Its portable output predates the latest candidate fixes. Existing Azure setup is preserved; precise repository-specific verification, immutable OIDC trust and profile-scoped role commands are in docs/azure-signing-admin.md, with no Azure mutations performed.

Signed/DCO commit05bf612 is pushed to feat/windows-viewer/PR#3. It fixes the full-page outer body width reset, aligns Contents/Print with the other48px toolbar hit targets, uses the observed exact Open picker filename field1148 and tests the actual outline-depth control. Pixel checks sample the owned WebView; native width/scroll/font assertions are unchanged and now retain precise failure evidence. No cases are skipped. Local gates: zero-warning Release build,586 core tests,100% measured core line coverage, full formatting and all commit/security/workflow hooks pass. Exact prior CI38166230860: source-hover passes; full native74/97,23failures,0skips; published suite exceeds600seconds before installed outcomes. A focused hosted reader-regressions job now covers these current paths alongside the full suite. Native/distributed/signature/review/release acceptance is still pending; no production release published. Owner preview PID55472 remains responsive and untouched. Its portable output predates the latest candidate fixes. Existing Azure setup is preserved; precise repository-specific verification, immutable OIDC trust and profile-scoped role commands are in docs/azure-signing-admin.md, with no Azure mutations performed.

KofTwentyTwo commented

2026-10-11 23:13:45 +00:00

(Migrated from github.com)

Reviewed the eight owner-provided Mac screenshots. The settings design reference is clearer: General/Editors/Themes/Markdown/Printing categories, grouped related controls, short explanations, and aligned values. We should carry those features and organization into native Windows settings controls.

Verified gaps in the current Windows code: effective default-viewer status/action is not visible in General; Contents wrapping/truncation and optional heading-depth markers are missing; theme management needs palette previews and clearer Enabled/default selection; print/PDF defaults need a dedicated Settings category as well as the print flow. Audit separate default-theme semantics before changing them.

Keep the owner's requested Windows toolbar behavior: visible labeled Print and Contents, no duplicate heading-depth dropdown, Contents/Source off by default and50/50 resizable source with persistent highlighting. The private screenshots/document content are not attached or reproduced in this public issue. This comparison is design evidence, not a completed redesign or release-readiness claim.

Reviewed the eight owner-provided Mac screenshots. The settings design reference is clearer: General/Editors/Themes/Markdown/Printing categories, grouped related controls, short explanations, and aligned values. We should carry those features and organization into native Windows settings controls. Verified gaps in the current Windows code: effective default-viewer status/action is not visible in General; Contents wrapping/truncation and optional heading-depth markers are missing; theme management needs palette previews and clearer Enabled/default selection; print/PDF defaults need a dedicated Settings category as well as the print flow. Audit separate default-theme semantics before changing them. Keep the owner's requested Windows toolbar behavior: visible labeled Print and Contents, no duplicate heading-depth dropdown, Contents/Source off by default and50/50 resizable source with persistent highlighting. The private screenshots/document content are not attached or reproduced in this public issue. This comparison is design evidence, not a completed redesign or release-readiness claim.

KofTwentyTwo commented

2026-10-11 23:28:21 +00:00

(Migrated from github.com)

Signed/DCO aeeb28d is pushed in head b5e519e; it adds native General settings for full Contents wrapping or one-line ellipsis/full-title tooltips, plus optional heading-depth markers. New-window Contents/Source defaults and Default Apps guidance move to General; splash/startup-update controls remain Advanced. Checked persistence precedes live sibling updates and baseline changes, preserving per-window visibility/depth and CSS drafts. Category changes reset normal scrolling.

All593 core tests pass with100% measured line coverage; Release build has zero warnings/errors; format and commit/security/workflow hooks pass. The new real native long-title/sibling/persistence/restoration case compiles and is discovered; no local input occurred. Updated default/file-type acceptance uses General. Actual hosted runtime and Settings screenshots remain pending on exact pushed b5e519e.

Exact previous53755ab CI38182128299 is now terminal success: focused21/21, native/published99/99 and installed stable/dev98/98 each, zero skips. Actual registration lifecycle and simultaneous profiles/isolated updates/uninstall preservation pass. Head b5e519e was pushed only after this full acceptance finished. Full theme/default-status/Printing settings design, owner acceptance and final signing/review remain open; this issue is not closed.

Signed/DCO aeeb28d is pushed in head b5e519e; it adds native General settings for full Contents wrapping or one-line ellipsis/full-title tooltips, plus optional heading-depth markers. New-window Contents/Source defaults and Default Apps guidance move to General; splash/startup-update controls remain Advanced. Checked persistence precedes live sibling updates and baseline changes, preserving per-window visibility/depth and CSS drafts. Category changes reset normal scrolling. All593 core tests pass with100% measured line coverage; Release build has zero warnings/errors; format and commit/security/workflow hooks pass. The new real native long-title/sibling/persistence/restoration case compiles and is discovered; no local input occurred. Updated default/file-type acceptance uses General. Actual hosted runtime and Settings screenshots remain pending on exact pushed b5e519e. Exact previous53755ab CI38182128299 is now terminal success: focused21/21, native/published99/99 and installed stable/dev98/98 each, zero skips. Actual registration lifecycle and simultaneous profiles/isolated updates/uninstall preservation pass. Head b5e519e was pushed only after this full acceptance finished. Full theme/default-status/Printing settings design, owner acceptance and final signing/review remain open; this issue is not closed.

KofTwentyTwo commented

2026-10-11 23:43:14 +00:00

(Migrated from github.com)

Signed/DCO local7fa9f77 adds a dedicated native Printing page with a shared PrintOptionsEditor used by the output dialog: themes, backgrounds/images, font zoom, width, borders and bounded padding. Apply saves validated defaults; Restore uses the latest saved baseline; closing discards drafts. PDF destinations and Open PDF/Open folder remain deliberate export actions. Checked saves precede shared reader-baseline broadcasts, preserving unsubmitted sibling Settings drafts. Theme changes refresh available named print choices; native navigation labels stay visible at normal width.

Zero-warning/error Release build and formatting pass; all598 core tests/100% measured lines pass.102 native cases compile/discover, including new actual Apply/Restore/Print-cancel/persistence and concurrent sibling-output/draft/Restore checks, with owned controls and a real Settings screenshot. Existing Print/PDF/pixel checks and every deadline remain unchanged. New native assertions are not yet executed; no local input occurred.

Exact pushedb5e519e focused19/22 and full native98/100 fail, zero skips. The new Contents options case passes in both. Localec78b50 corrects actual raw-inline/HTML-block closing boundaries, tooltip click interception and an observed exact picker lookup problem;598 core cases prove the reproduced parser correction. Installed114603470178 is still live, so these commits are held to preserve the full run. Retained focused/full logs and actual TRX/Settings pixels under artifacts/reader-regressions-38184378762. Full theme/default-status design, fresh acceptance, owner review and production signing remain open.

Signed/DCO local7fa9f77 adds a dedicated native Printing page with a shared PrintOptionsEditor used by the output dialog: themes, backgrounds/images, font zoom, width, borders and bounded padding. Apply saves validated defaults; Restore uses the latest saved baseline; closing discards drafts. PDF destinations and Open PDF/Open folder remain deliberate export actions. Checked saves precede shared reader-baseline broadcasts, preserving unsubmitted sibling Settings drafts. Theme changes refresh available named print choices; native navigation labels stay visible at normal width. Zero-warning/error Release build and formatting pass; all598 core tests/100% measured lines pass.102 native cases compile/discover, including new actual Apply/Restore/Print-cancel/persistence and concurrent sibling-output/draft/Restore checks, with owned controls and a real Settings screenshot. Existing Print/PDF/pixel checks and every deadline remain unchanged. New native assertions are not yet executed; no local input occurred. Exact pushedb5e519e focused19/22 and full native98/100 fail, zero skips. The new Contents options case passes in both. Localec78b50 corrects actual raw-inline/HTML-block closing boundaries, tooltip click interception and an observed exact picker lookup problem;598 core cases prove the reproduced parser correction. Installed114603470178 is still live, so these commits are held to preserve the full run. Retained focused/full logs and actual TRX/Settings pixels under artifacts/reader-regressions-38184378762. Full theme/default-status design, fresh acceptance, owner review and production signing remain open.

KofTwentyTwo commented

2026-10-11 23:44:45 +00:00

(Migrated from github.com)

Signed/DCO headb2ecc8c is pushed: ec78b50 corrects complete inline title syntax across Markdig HTML blocks plus the observed tooltip/picker input problems;7fa9f77 adds native Printing settings sharing the output editor and checked default saves. All598 core tests pass with100% measured line coverage; Release build has zero warnings/errors; formatting and commit/security/workflow hooks pass.102 native cases compile/discover, including actual Apply/Restore/dialog-cancel and concurrent sibling output/draft preservation checks. Fresh hosted native/distributed execution is required; no local input occurred.

Exact previousb5e519e / CI38184378762 is terminal: focused19/22, native98/100 and published98/100, zero skips. General/Contents options case passes in focused/full. Published actual TRX confirms the same raw-inline fold and physical source-click/alignment failures; installed lifecycle phase is not reached. Retain current-installed-b5e519e.log and actual artifacts/TRX under artifacts/current-installed-evidence-38184378762. Previous53755ab remains the fully green distributed/registration/coexistence checkpoint, with unsigned fixtures.

Headb2ecc8c was pushed only after preserving terminal evidence. Existing assertions/cases/deadlines remain; no failed check disabled, fabricated human review, shared merge, Azure retry, tag or production release. Remaining: fresh exact-head acceptance and Settings screenshots, effective default-viewer status, theme palette/default clarity, owner acceptance, actual Windows chooser/Explorer and reviewed/licensed/signed production release requirements.

Signed/DCO headb2ecc8c is pushed: ec78b50 corrects complete inline title syntax across Markdig HTML blocks plus the observed tooltip/picker input problems;7fa9f77 adds native Printing settings sharing the output editor and checked default saves. All598 core tests pass with100% measured line coverage; Release build has zero warnings/errors; formatting and commit/security/workflow hooks pass.102 native cases compile/discover, including actual Apply/Restore/dialog-cancel and concurrent sibling output/draft preservation checks. Fresh hosted native/distributed execution is required; no local input occurred. Exact previousb5e519e / CI38184378762 is terminal: focused19/22, native98/100 and published98/100, zero skips. General/Contents options case passes in focused/full. Published actual TRX confirms the same raw-inline fold and physical source-click/alignment failures; installed lifecycle phase is not reached. Retain current-installed-b5e519e.log and actual artifacts/TRX under artifacts/current-installed-evidence-38184378762. Previous53755ab remains the fully green distributed/registration/coexistence checkpoint, with unsigned fixtures. Headb2ecc8c was pushed only after preserving terminal evidence. Existing assertions/cases/deadlines remain; no failed check disabled, fabricated human review, shared merge, Azure retry, tag or production release. Remaining: fresh exact-head acceptance and Settings screenshots, effective default-viewer status, theme palette/default clarity, owner acceptance, actual Windows chooser/Explorer and reviewed/licensed/signed production release requirements.

KofTwentyTwo commented

2026-10-11 23:55:57 +00:00

(Migrated from github.com)

Signed/DCO local24a464e adds effective .md default-viewer status to General. Windows' read-only AssocQueryStringW/ASSOCSTR_PROGID query is bounded to1024 Unicode characters; the exact edition/install must retain its owned registration. Registration alone is not Default, another edition/retargeted command is not Default, and unavailable results remain Unknown. Refresh runs when returning from Windows Settings or explicitly. No defaults/UserChoice/hash writes or competing-app paths are read/displayed. ADR0005/threat model document the boundary.

Seven actual read-only API/isolated registry tests pass; zero-warning Release build/format/hooks pass.105 native cases compile/discover; updated General status/refresh and new true-truncation tooltip/physical-heading tests remain unexecuted. The commit is held to preserve currently live full native114606533948 and installed114606533825.

Exact pushedb2ecc8c / CI38185403701 focused23/24 fails, zero skips: actual TRX proves both new Printing apply/restore/cancel/concurrent-draft cases, raw-inline folding, picker cancellation and Contents options pass. Remaining physical source/reader alignment still fails; retained actual evidence includes a short-title tooltip. The new refinement limits full-title tips to genuinely trimmed native labels with an explicit placement target and preserves mouse invocation diagnostics, without changing any existing assertion/deadline. Fresh native/distributed acceptance is required before claiming resolution. Retain current-focused-b2ecc8c.log and artifacts/reader-regressions-38185403701.

Actual Windows user-selected default/Explorer, theme previews/default clarity, owner acceptance, genuine review and production signing remain required; no release is published.

Signed/DCO local24a464e adds effective `.md` default-viewer status to General. Windows' read-only AssocQueryStringW/ASSOCSTR_PROGID query is bounded to1024 Unicode characters; the exact edition/install must retain its owned registration. Registration alone is not Default, another edition/retargeted command is not Default, and unavailable results remain Unknown. Refresh runs when returning from Windows Settings or explicitly. No defaults/UserChoice/hash writes or competing-app paths are read/displayed. ADR0005/threat model document the boundary. Seven actual read-only API/isolated registry tests pass; zero-warning Release build/format/hooks pass.105 native cases compile/discover; updated General status/refresh and new true-truncation tooltip/physical-heading tests remain unexecuted. The commit is held to preserve currently live full native114606533948 and installed114606533825. Exact pushedb2ecc8c / CI38185403701 focused23/24 fails, zero skips: actual TRX proves both new Printing apply/restore/cancel/concurrent-draft cases, raw-inline folding, picker cancellation and Contents options pass. Remaining physical source/reader alignment still fails; retained actual evidence includes a short-title tooltip. The new refinement limits full-title tips to genuinely trimmed native labels with an explicit placement target and preserves mouse invocation diagnostics, without changing any existing assertion/deadline. Fresh native/distributed acceptance is required before claiming resolution. Retain current-focused-b2ecc8c.log and artifacts/reader-regressions-38185403701. Actual Windows user-selected default/Explorer, theme previews/default clarity, owner acceptance, genuine review and production signing remain required; no release is published.

KofTwentyTwo commented

2026-10-12 00:13:34 +00:00

(Migrated from github.com)

Signed/DCO head1bfab22 is pushed after exactb2ecc8c CI38185403701 became terminal. It includes24a464e effective default-viewer status and1bfab22 native theme palette/state/default controls plus fixture cleanup. General reports the actual effective .md selection with bounded read-only Windows ProgID querying, exact installed/edition ownership and manual/activation refresh. Seven actual non-GUI API/isolated registry tests pass.

Theme rows show five validated color previews with hexadecimal accessible names/tooltips and textual Built-in/edited/Custom, Enabled/Disabled and Default labels. Native default radio selection enables the selected choice inside the copied draft; Save applies the existing saved DocumentTheme to open/new readers, Cancel discards it. Existing edit/copy/import/restore behaviors remain. The new actual native case checks painted Nord swatch pixels, default/enabled states, save/reopen and retains a Settings screenshot.106 native cases compile/discover; zero-warning Release build, format and hooks pass. Core unchanged598/100% measured lines.

Previousb2ecc8c focused23/24 and native101/102 fail, zero skips, only source physical alignment. Both Printing cases/raw-inline folds/picker/Contents options pass in completed suites. Published process reaches unchanged600seconds without final TRX; its process log records alignment failure and ten failed exits. No installed lifecycle outcome. Exact pinned FlaUI source uses CloseMainWindow plus5-second wait; fixture teardown now closes every owned PID window before the unchanged close/kill fallback. No cases/assertions/deadlines removed or expanded. Retain current-native/current-installed-b2ecc8c logs and actual artifact bundle38185403701.

Fresh exact1bfab22 CI38187115859 is live; pr-gates/security pass, scripts/CodeQL live. Actual default/status/palette/trim-only tooltip and full published/installed runtime remain pending. Owner design acceptance, actual Windows chooser/Explorer, shared/license and genuine head review, deployed Azure profile/identity and shipped signature trust remain required. No local GUI input, preview55472 changes, Azure retry, shared merge, tag or production publication.

Signed/DCO head1bfab22 is pushed after exactb2ecc8c CI38185403701 became terminal. It includes24a464e effective default-viewer status and1bfab22 native theme palette/state/default controls plus fixture cleanup. General reports the actual effective .md selection with bounded read-only Windows ProgID querying, exact installed/edition ownership and manual/activation refresh. Seven actual non-GUI API/isolated registry tests pass. Theme rows show five validated color previews with hexadecimal accessible names/tooltips and textual Built-in/edited/Custom, Enabled/Disabled and Default labels. Native default radio selection enables the selected choice inside the copied draft; Save applies the existing saved DocumentTheme to open/new readers, Cancel discards it. Existing edit/copy/import/restore behaviors remain. The new actual native case checks painted Nord swatch pixels, default/enabled states, save/reopen and retains a Settings screenshot.106 native cases compile/discover; zero-warning Release build, format and hooks pass. Core unchanged598/100% measured lines. Previousb2ecc8c focused23/24 and native101/102 fail, zero skips, only source physical alignment. Both Printing cases/raw-inline folds/picker/Contents options pass in completed suites. Published process reaches unchanged600seconds without final TRX; its process log records alignment failure and ten failed exits. No installed lifecycle outcome. Exact pinned FlaUI source uses CloseMainWindow plus5-second wait; fixture teardown now closes every owned PID window before the unchanged close/kill fallback. No cases/assertions/deadlines removed or expanded. Retain current-native/current-installed-b2ecc8c logs and actual artifact bundle38185403701. Fresh exact1bfab22 CI38187115859 is live; pr-gates/security pass, scripts/CodeQL live. Actual default/status/palette/trim-only tooltip and full published/installed runtime remain pending. Owner design acceptance, actual Windows chooser/Explorer, shared/license and genuine head review, deployed Azure profile/identity and shipped signature trust remain required. No local GUI input, preview55472 changes, Azure retry, shared merge, tag or production publication.

KofTwentyTwo commented

2026-10-12 00:24:43 +00:00

(Migrated from github.com)

Exact pushed1bfab22 / CI38187115859 focused23/26 fails, zero skips. The new palette/default case cannot find ThemeChoice:Nord; existing future-schema/export case cannot find ThemeChoice:Owned future. The taller native rows leave these choices virtualized offscreen. Signed/DCO localb1f8af7 scrolls the owned ThemeList through its native Scroll pattern until the same exact automation ID is realized, then uses the same native selection. All value assertions/cases and10-second deadline remain; Release/format/hooks pass. Fresh runtime is required.

The long-title tooltip/physical heading test passes, as do both Printing cases, raw-inline folding, picker cancellation and Contents options. Physical source/reader alignment still fails on1bfab22; signed local49f0556 restores the trusted heading Tag on the Contents content Grid for repeat ItemClick. Client pointer diagnostics were not screen-space coordinates; the earlier tooltip-interception inference was not established. Retain current-focused-1bfab22.log and actual focused TRX/artifacts38187115859.

Full native114611732252 and installed114611732224 remain live; local49f0556/b1f8af7 are held until terminal. No installed/signature outcome inferred and no checks/assertions/deadlines changed. Reference ContentView/SettingsView/ThemeManager cached text matches the immutable6d08bc9 blobs after newline/BOM normalization. No local input, owner preview change, Azure retry, shared merge, tag or publication.

Exact pushed1bfab22 / CI38187115859 focused23/26 fails, zero skips. The new palette/default case cannot find ThemeChoice:Nord; existing future-schema/export case cannot find ThemeChoice:Owned future. The taller native rows leave these choices virtualized offscreen. Signed/DCO localb1f8af7 scrolls the owned ThemeList through its native Scroll pattern until the same exact automation ID is realized, then uses the same native selection. All value assertions/cases and10-second deadline remain; Release/format/hooks pass. Fresh runtime is required. The long-title tooltip/physical heading test passes, as do both Printing cases, raw-inline folding, picker cancellation and Contents options. Physical source/reader alignment still fails on1bfab22; signed local49f0556 restores the trusted heading Tag on the Contents content Grid for repeat ItemClick. Client pointer diagnostics were not screen-space coordinates; the earlier tooltip-interception inference was not established. Retain current-focused-1bfab22.log and actual focused TRX/artifacts38187115859. Full native114611732252 and installed114611732224 remain live; local49f0556/b1f8af7 are held until terminal. No installed/signature outcome inferred and no checks/assertions/deadlines changed. Reference ContentView/SettingsView/ThemeManager cached text matches the immutable6d08bc9 blobs after newline/BOM normalization. No local input, owner preview change, Azure retry, shared merge, tag or publication.

KofTwentyTwo commented

2026-10-12 00:34:06 +00:00

(Migrated from github.com)

Signed/DCO head9d5c0d1 is pushed after preserving terminal1bfab22 evidence. It includes49f0556 trusted heading identity on the native Contents content Grid, enabling repeat ItemClick when selection has not changed; andb1f8af7 native scrolling of the owned virtualized theme list until the same exact ID is realized before unchanged selection/assertions. Zero-warning Release build, formatting and commit hooks pass.106 native cases remain; no assertion, case or deadline is removed/expanded.

Exact prior1bfab22 CI38187115859 focused23/26/native102/106 fails, zero skips. Published process hits600seconds without final TRX, with the same four failures: Nord palette/default and copied-draft lookups, Owned future import/export lookup, and source repeat activation/alignment. Installed lifecycle is not reached. Retain current-installed-1bfab22.log and actual artifacts/current-installed-evidence-38187115859 plus focused/native logs/TRX. Long-title tooltip physical activation, both Printing cases, raw-inline fold, picker cancellation and Contents options pass in completed suites.

Fresh exact9d5c0d1 CI38188145397 is live; pr-gates/security/scripts pass, CodeQL live. Shared signing/license/inventory/AppKit Help PRs31/32/36/29 remain unmerged drafts with no reviews; app PR3 remains draft with no genuine reviews. Protected release environment has no signing configuration variables. Actual default/Explorer choice, owner design acceptance, reviewed shared/license requirements, genuine head acknowledgement and deployed Azure/shipped signing remain required. No local GUI input, preview55472 changes, Azure retry, shared merge, tag or production release.

Signed/DCO head9d5c0d1 is pushed after preserving terminal1bfab22 evidence. It includes49f0556 trusted heading identity on the native Contents content Grid, enabling repeat ItemClick when selection has not changed; andb1f8af7 native scrolling of the owned virtualized theme list until the same exact ID is realized before unchanged selection/assertions. Zero-warning Release build, formatting and commit hooks pass.106 native cases remain; no assertion, case or deadline is removed/expanded. Exact prior1bfab22 CI38187115859 focused23/26/native102/106 fails, zero skips. Published process hits600seconds without final TRX, with the same four failures: Nord palette/default and copied-draft lookups, Owned future import/export lookup, and source repeat activation/alignment. Installed lifecycle is not reached. Retain current-installed-1bfab22.log and actual artifacts/current-installed-evidence-38187115859 plus focused/native logs/TRX. Long-title tooltip physical activation, both Printing cases, raw-inline fold, picker cancellation and Contents options pass in completed suites. Fresh exact9d5c0d1 CI38188145397 is live; pr-gates/security/scripts pass, CodeQL live. Shared signing/license/inventory/AppKit Help PRs31/32/36/29 remain unmerged drafts with no reviews; app PR3 remains draft with no genuine reviews. Protected release environment has no signing configuration variables. Actual default/Explorer choice, owner design acceptance, reviewed shared/license requirements, genuine head acknowledgement and deployed Azure/shipped signing remain required. No local GUI input, preview55472 changes, Azure retry, shared merge, tag or production release.

KofTwentyTwo commented

2026-10-12 01:11:29 +00:00

(Migrated from github.com)

The terminal CI38188145397 is successful at 9d5c0d1:

Focused desktop26/26; full native/published106/106; installed stable/dev105/105 each, zero failures/skips.
Actual palette/default/save/reopen, Printing defaults and sibling draft, Contents wrapping/markers/full-title hover, repeated physical source synchronization and raw-inline folding all pass.
Both real AppKit updates/restarts pass. Actual simultaneous stable/dev windows, separate profiles, isolated updates, five-extension registration and sibling-preserving uninstall pass, preserving existing defaults.

Comparing the eight private Mac screenshots identified Settings gaps addressed with native General/Printing controls and palette/default previews. No private screenshots or document contents are published. Hosted Settings screenshots additionally revealed that its default cascading position clipped the window beyond the monitor. Signed c3c735e centers it inside the actual monitor work area and adds containment to the existing layout regression. Fresh CI38190828814 is running; complete screenshots still need inspection.

Fixtures are unsigned. Actual user-selected Default Apps/Explorer behavior, final owner design feedback, required current-head maintainer review, shared signing-builder/license decisions and deployed Azure/final-artifact signing remain outstanding. No release or shared merge performed. Substantial AI assistance: Codex.

The terminal [CI38188145397](https://github.com/KofTwentyTwo/MarkdownViewr.Windows/actions/runs/38188145397) is successful at `9d5c0d1`: - Focused desktop26/26; full native/published106/106; installed stable/dev105/105 each, zero failures/skips. - Actual palette/default/save/reopen, Printing defaults and sibling draft, Contents wrapping/markers/full-title hover, repeated physical source synchronization and raw-inline folding all pass. - Both real AppKit updates/restarts pass. Actual simultaneous stable/dev windows, separate profiles, isolated updates, five-extension registration and sibling-preserving uninstall pass, preserving existing defaults. Comparing the eight private Mac screenshots identified Settings gaps addressed with native General/Printing controls and palette/default previews. No private screenshots or document contents are published. Hosted Settings screenshots additionally revealed that its default cascading position clipped the window beyond the monitor. Signed `c3c735e` centers it inside the actual monitor work area and adds containment to the existing layout regression. [Fresh CI38190828814](https://github.com/KofTwentyTwo/MarkdownViewr.Windows/actions/runs/38190828814) is running; complete screenshots still need inspection. Fixtures are unsigned. Actual user-selected Default Apps/Explorer behavior, final owner design feedback, required current-head maintainer review, shared signing-builder/license decisions and deployed Azure/final-artifact signing remain outstanding. No release or shared merge performed. Substantial AI assistance: Codex.

KofTwentyTwo commented

2026-10-12 03:52:02 +00:00

(Migrated from github.com)

Reviewed source confirms that theme management, theme editing and external-editor configuration are hosted inside the existing Settings window through ShowEditorAsync; the historical Dialog class names do not imply separate modal windows. This corrects the earlier screenshot-comparison summary. Short confirmations and native file pickers remain modal.

All eight Mac reference screenshots were inspected privately; no screenshots/document contents are attached here. Native General/Contents options, Printing defaults, palette/default previews and Settings layout/cancellation pass the retained b542 hosted checkpoint. Owner visual/scaling acceptance remains open; this issue stays open. Reviewed correction/checkpoint82256f3 is pushed and fresh CI38201379210 is running, without altered assertions/deadlines.

Substantial AI assistance: Codex.

Reviewed source confirms that theme management, theme editing and external-editor configuration are hosted inside the existing Settings window through `ShowEditorAsync`; the historical Dialog class names do not imply separate modal windows. This corrects the earlier screenshot-comparison summary. Short confirmations and native file pickers remain modal. All eight Mac reference screenshots were inspected privately; no screenshots/document contents are attached here. Native General/Contents options, Printing defaults, palette/default previews and Settings layout/cancellation pass the retained b542 hosted checkpoint. Owner visual/scaling acceptance remains open; this issue stays open. Reviewed correction/checkpoint82256f3 is pushed and [fresh CI38201379210](https://github.com/KofTwentyTwo/MarkdownViewr.Windows/actions/runs/38201379210) is running, without altered assertions/deadlines. Substantial AI assistance: Codex.

KofTwentyTwo commented

2026-10-12 05:19:48 +00:00

(Migrated from github.com)

The actual e6fd281 native111/111 and focused28/28 pass, but published acceptance in CI38204216325 fails the existing theme-import cancellation case waiting10 seconds for filename1148, then the suite reaches its unchanged600-second bound without final TRX. Installed phases are not reached. Actual logs and screenshot are retained; the screenshot shows Settings/main inactive and no error status. Source application files are unchanged from the preceding all-channel accepted checkpoint82256f3.

Signed/pushed d0526ef activates the exact owned Settings window and focuses its real action before UIA invocation; timeout evidence now includes LastInvocation/LastMouseInput. This is a candidate for an inactive-owner timing problem, not confirmed causation. Existing selector, assertions, every case and deadlines remain intact. Local isolated Release/format/hooks pass. Fresh CI38207252597 is live; preserve it before further pushes. No local GUI input, signing or release claim.

Substantial AI assistance: Codex.

The actual e6fd281 native111/111 and focused28/28 pass, but published acceptance in CI38204216325 fails the existing theme-import cancellation case waiting10 seconds for filename1148, then the suite reaches its unchanged600-second bound without final TRX. Installed phases are not reached. Actual logs and screenshot are retained; the screenshot shows Settings/main inactive and no error status. Source application files are unchanged from the preceding all-channel accepted checkpoint82256f3. Signed/pushed d0526ef activates the exact owned Settings window and focuses its real action before UIA invocation; timeout evidence now includes LastInvocation/LastMouseInput. This is a candidate for an inactive-owner timing problem, not confirmed causation. Existing selector, assertions, every case and deadlines remain intact. Local isolated Release/format/hooks pass. [Fresh CI38207252597](https://github.com/KofTwentyTwo/MarkdownViewr.Windows/actions/runs/38207252597) is live; preserve it before further pushes. No local GUI input, signing or release claim. Substantial AI assistance: Codex.

KofTwentyTwo commented

2026-10-12 05:48:17 +00:00

(Migrated from github.com)

Current exact-head evidence: d0526ef passes core677/100% measured lines, focused28, full native111 and fuzz9+2, zero failures/skips. The published picker cancellation case now passes. Published acceptance still hits its unchanged600-second process bound after110/111 passed cases, without a recorded assertion failure or final TRX; installed lifecycle is not reached.

The last passing published run took590.96 seconds (109 cases); its unchanged final Appearance/PDF case took13.07 seconds. Three current close fallbacks versus four previously do not establish a new shutdown regression. Aggregate duration versus a blocked final operation remains unresolved.

Signed de70220 adds temporary retained fixture launch/slow-wait/closure timings and exact Appearance/PDF phase markers to distinguish those causes. Every existing case, assertion, wait and deadline remains intact. Isolated full Release:0 warnings/errors; format, direct lint of both changed documents and signed commit hooks pass. Fresh hosted proof remains required; no production signing or release is claimed.

Current exact-head evidence: `d0526ef` passes core677/100% measured lines, focused28, full native111 and fuzz9+2, zero failures/skips. The published picker cancellation case now passes. Published acceptance still hits its unchanged600-second process bound after110/111 passed cases, without a recorded assertion failure or final TRX; installed lifecycle is not reached. The last passing published run took590.96 seconds (109 cases); its unchanged final Appearance/PDF case took13.07 seconds. Three current close fallbacks versus four previously do not establish a new shutdown regression. Aggregate duration versus a blocked final operation remains unresolved. Signed `de70220` adds temporary retained fixture launch/slow-wait/closure timings and exact Appearance/PDF phase markers to distinguish those causes. Every existing case, assertion, wait and deadline remains intact. Isolated full Release:0 warnings/errors; format, direct lint of both changed documents and signed commit hooks pass. Fresh hosted proof remains required; no production signing or release is claimed.

KofTwentyTwo commented

2026-10-12 06:08:29 +00:00

(Migrated from github.com)

Exact de70220 completed evidence: core677/677 with100% measured lines, native111/111, focused28/28 and parser campaigns9+2 all pass with zero skips. Full native duration is10m6s. Actual TRXs/logs and current Settings/toolbar screenshots are retained.

Focused timing evidence covers24 fixture sessions: launch total14.050s, closure9.184s; the unchanged Appearance case completes in10.739s, with2.918s between PDF-start and PDF/find verification. One Printing-defaults fixture uses the existing5-second close fallback. A non-GUI benchmark of five retained PDF images measures original pixel scanning at0.11-0.13s each, so no pixel-helper change is justified.

Distributed job114680286861 / CI38209503930 remains live. Its current phase is not inferable from the unavailable live log; no published or installed success is claimed. Preserve the run, inspect its actual retained timing trace when available, and remove temporary diagnostics after verified resolution. No test/assertion/deadline or production source is changed.

Exact `de70220` completed evidence: core677/677 with100% measured lines, native111/111, focused28/28 and parser campaigns9+2 all pass with zero skips. Full native duration is10m6s. Actual TRXs/logs and current Settings/toolbar screenshots are retained. Focused timing evidence covers24 fixture sessions: launch total14.050s, closure9.184s; the unchanged Appearance case completes in10.739s, with2.918s between PDF-start and PDF/find verification. One Printing-defaults fixture uses the existing5-second close fallback. A non-GUI benchmark of five retained PDF images measures original pixel scanning at0.11-0.13s each, so no pixel-helper change is justified. Distributed job114680286861 / CI38209503930 remains live. Its current phase is not inferable from the unavailable live log; no published or installed success is claimed. Preserve the run, inspect its actual retained timing trace when available, and remove temporary diagnostics after verified resolution. No test/assertion/deadline or production source is changed.

KofTwentyTwo commented

2026-10-12 10:17:47 +00:00

(Migrated from github.com)

Exact signed de70220 / CI38209503930 now passes every acceptance suite: core677/100% measured lines, native/published111/111, focused28/28 and installed stable/dev110/110 each; fuzz9+2, zero failures/skips. Actual AppKit apply/restart and target-byte assertions, simultaneous channel/profile isolation, five-extension registration and stable-uninstall/default preservation all pass. Fixtures are unsigned.

Published duration572.71 seconds; stable496.38 and dev574.75. Each final Appearance/PDF case completed; the previous aggregate timeout does not establish a blocked product operation. Temporary timing instrumentation will be removed with a fresh final-head check, retaining all original assertions/cases/deadlines.

Release still requires the shared signing/configuration decisions, verified Azure CI authority and final shipped signatures, actual standards enrollment/inventory, owner Windows11/default-chooser acceptance and genuine final-head maintainer review. The latter is also required by configuration Apply, so the prepared strictly additive fuzz check remains unapplied. No production release or shared merge has occurred.

Exact signed `de70220` / CI38209503930 now passes every acceptance suite: core677/100% measured lines, native/published111/111, focused28/28 and installed stable/dev110/110 each; fuzz9+2, zero failures/skips. Actual AppKit apply/restart and target-byte assertions, simultaneous channel/profile isolation, five-extension registration and stable-uninstall/default preservation all pass. Fixtures are unsigned. Published duration572.71 seconds; stable496.38 and dev574.75. Each final Appearance/PDF case completed; the previous aggregate timeout does not establish a blocked product operation. Temporary timing instrumentation will be removed with a fresh final-head check, retaining all original assertions/cases/deadlines. Release still requires the shared signing/configuration decisions, verified Azure CI authority and final shipped signatures, actual standards enrollment/inventory, owner Windows11/default-chooser acceptance and genuine final-head maintainer review. The latter is also required by configuration Apply, so the prepared strictly additive fuzz check remains unapplied. No production release or shared merge has occurred.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: KofTwentyTwo/MarkdownViewr.Windows#21