Release v1.3.7: title-bar version + first-selection preview zoom + parameter-node UI on add

- Move app version + effect-library version from status-bar to title bar so the FPS / timing readout has room on the right.
- First selection of a node now defers FitPreviewToView until the next eval populates cachedOutput; previously the immediate fit measured an empty rect and produced the wrong zoom until the user re-selected.
- Clock / parameter nodes now show their on-node UI (play button, seek slider) immediately on add. The add path now sets m_forceRender = true so the post-eval RebuildLayout runs once with live analysis output instead of waiting for the user to click the node.

Author: Spruill-1

CHANGELOG.md

@@ -3,6 +3,30 @@
 All notable changes to ShaderLab will be documented in this file.
 Format follows [Keep a Changelog](https://keepachangelog.com/).
 
+## [1.3.7] - 2026-05-05
+
+### Changed
+- Moved the version + effect-library version readout from the bottom-status-bar to the title bar. Frees the right side of the status bar so the FPS / frame-timing readout has room to breathe (it had been clipping against the version text on smaller windows).
+
+### Fixed
+- First selection of a node in a freshly-loaded graph or a freshly-added node would render the preview at the wrong zoom (whatever the previous viewport happened to be). Subsequent selections were correct because by then the cached output existed. Now click defers `FitPreviewToView` until the next eval has populated `cachedOutput`, so the very first fit measures real bounds.
+- Newly-added Clock and other parameter nodes did not show their on-node UI (play/pause button, seek slider) until the user clicked them. The post-eval `RebuildLayout` only fired on user-forced renders; the add-node path now sets `m_forceRender = true`, triggering the same path so the controls appear on the first frame after add.
+
+## [1.3.6] - 2026-05-05
+
+### Fixed
+- Save / load of `.effectgraph` archives with embedded media froze the UI for tens of seconds while miniz compressed/decompressed. The ProgressBar painted once at the initial 16 ms yield then sat unchanged because the work ran synchronously on the UI thread. The save/load now runs on a threadpool thread (miniz is pure native, no WinRT marshalling concerns) and the progress callback marshals each update back through `DispatcherQueue` so the bar animates smoothly.
+- Video sources bound to a paused Clock free-ran in a ~1 s loop instead of holding a static frame. Cause: `TickAndUploadVideos` advanced the decoder via `provider->Tick(deltaSec)` whenever the bound `Time` value didn't differ from the current decode position by more than one frame, then the next iteration's "small forward gap" branch let it drift further. Fixed by remembering each provider's previous bound `Time` value — if it didn't change between ticks, hold the frame at exactly that position rather than ticking.
+
+### Internal
+- Resolved compiler warnings: `C4456` (shadowed `first` local in `WriteNodeJson`), `C4189` (unused `slashPos`, `imgDipX/Y`, `isPixelTraceTab`), `C4100` (unreferenced `sender` parameter on `OnPreviewPointerMoved`).
+
+## [1.3.5] - 2026-05-05
+
+### Fixed
+- Right-hand properties panel jittered horizontally during video playback. Cause: every 250 ms while ANY graph node was dirty (always true while a video was playing) we rebuilt the entire properties panel, re-creating Slider + NumberBox pairs whose auto widths varied frame to frame. Now the rebuild only fires when the selected node has at least one `propertyBinding` whose live value can change between ticks.
+- Visual Assets (app tile, splash, badges) were broken in shipped MSIX. The `Assets/` directory contained 21 corrupted `*.scale-200.scale-N.png` files (VS regenerated them by feeding existing scale-200 PNGs back as source — none of those names match any MRT lookup pattern), plus several 0-byte placeholder files. Cleaned them out and stripped the matching entries from `ShaderLab.vcxproj` / `.filters`. The remaining originals will resolve through MRT correctly; user-supplied bear artwork still needs to be dropped over the placeholder PNGs to actually appear.
+
 ## [1.2.7] - 2026-05-04
 
 ### Added

MainWindow.xaml

@@ -293,15 +293,12 @@
                        Style="{StaticResource CaptionTextBlockStyle}"
                        FontFamily="Cascadia Mono, Consolas, Courier New" />
 
-            <TextBlock x:Name="AppVersionText" Grid.Column="6"
-                       Text="v1.1.0"
-                       Margin="0,0,16,0"
-                       Foreground="{ThemeResource TextFillColorTertiaryBrush}"
-                       Style="{StaticResource CaptionTextBlockStyle}" />
-            <TextBlock x:Name="FpsText" Grid.Column="7"
+            <TextBlock x:Name="FpsText" Grid.Column="6"
                        Text="FPS: --"
                        Foreground="{ThemeResource TextFillColorSecondaryBrush}"
                        Style="{StaticResource CaptionTextBlockStyle}" />
+            <TextBlock x:Name="AppVersionText" Grid.Column="7"
+                       Visibility="Collapsed" />
         </Grid>
     </Grid>
 </Window>

MainWindow.xaml.cpp

@@ -24,7 +24,6 @@ namespace winrt::ShaderLab::implementation
         Title(std::wstring(L"ShaderLab v") + ::ShaderLab::VersionString + L" \u2014 HDR Shader Effect Development");
         // After Title() we'll refresh from RefreshTitleBar() once a graph
         // is loaded; the initial state is "Untitled".
-        AppVersionText().Text(std::wstring(L"v") + ::ShaderLab::VersionString);
 
         m_hwnd = GetWindowHandle();
 
@@ -465,12 +464,9 @@ namespace winrt::ShaderLab::implementation
 
         UpdateStatusBar();
 
-        // Set version text: app version + effect library version.
-        {
-            auto& lib = ::ShaderLab::Effects::ShaderLabEffects::Instance();
-            AppVersionText().Text(std::wstring(L"v") + ::ShaderLab::VersionString
-                + L"  (effects lib v" + std::to_wstring(lib.LibraryVersion()) + L")");
-        }
+        // Refresh title bar now that the effect library is loaded so the
+        // title shows both the app and effect-library version.
+        RefreshTitleBar();
 
         m_nodeGraphController.SetGraph(&m_graph);
         m_nodeGraphController.SetConnectionCallback(
@@ -1182,14 +1178,17 @@ namespace winrt::ShaderLab::implementation
 
     void MainWindow::RefreshTitleBar()
     {
-        // Title format: "<filename>[*] - ShaderLab <version>". The
-        // unsaved star is the standard editor convention; the
-        // filename is derived from m_currentFilePath if known, else
-        // "Untitled".
+        // Title format: "<filename>[*] - ShaderLab <version> (effects lib vN)".
+        // The unsaved star is the standard editor convention; the filename
+        // is derived from m_currentFilePath if known, else "Untitled". The
+        // effect-library version moved here from the status bar to free up
+        // room for the FPS readout on the right.
         std::wstring base = m_currentFilePath.empty() ? std::wstring(L"Untitled") :
             std::filesystem::path(m_currentFilePath).filename().wstring();
+        auto& lib = ::ShaderLab::Effects::ShaderLabEffects::Instance();
         std::wstring title = base + (m_unsavedChanges ? L"*" : L"")
-            + L" - ShaderLab " + ::ShaderLab::VersionString;
+            + L" - ShaderLab " + ::ShaderLab::VersionString
+            + L" (effects lib v" + std::to_wstring(lib.LibraryVersion()) + L")";
         try { Title(winrt::hstring(title)); } catch (...) {}
     }
 
@@ -2010,6 +2009,12 @@ namespace winrt::ShaderLab::implementation
                         m_graph.MarkAllDirty();
                         m_nodeGraphController.RebuildLayout();
                         PopulatePreviewNodeSelector();
+                        // Force a render tick that triggers the post-eval
+                        // RebuildLayout, which is what reveals on-node UI
+                        // (Clock play/pause + slider) for newly-added
+                        // parameter nodes. Without this the user has to
+                        // click the node before the controls appear.
+                        m_forceRender = true;
                     });
 
                     subItem.Items().Append(menuItem);
@@ -3150,7 +3155,12 @@ namespace winrt::ShaderLab::implementation
                 m_previewNodeId = hitNodeId;
 
             m_forceRender = true;
-            FitPreviewToView();
+            // Defer the fit until the next eval populates cachedOutput. On
+            // the very first selection of a node (before its first eval),
+            // GetPreviewImageBounds() returns an empty rect, so an immediate
+            // FitPreviewToView() lands on the wrong zoom. The deferred path
+            // in OnRenderTick re-fits once bounds are available.
+            m_needsFitPreview = true;
             UpdatePreviewOverlay();
         }
         else

Package.appxmanifest

@@ -9,7 +9,7 @@
   <Identity
     Name="ShaderLab"
     Publisher="CN=ShaderLab"
-    Version="1.3.6.0" />
+    Version="1.3.7.0" />
 
   <mp:PhoneIdentity PhoneProductId="a1b2c3d4-e5f6-7890-abcd-ef1234567890" PhonePublisherId="00000000-0000-0000-0000-000000000000"/>
 

Version.h

@@ -14,10 +14,10 @@ namespace ShaderLab
 {
     constexpr uint32_t VersionMajor = 1;
     constexpr uint32_t VersionMinor = 3;
-    constexpr uint32_t VersionPatch = 6;
+    constexpr uint32_t VersionPatch = 7;
 
     // Human-readable version string.
-    constexpr const wchar_t* VersionString = L"1.3.6";
+    constexpr const wchar_t* VersionString = L"1.3.7";
 
     // Graph format version. Increment when serialization format changes.
     // Graphs saved with a higher format version cannot be loaded by older apps.