Comparer: pilot pink-dot grid overlay for pixel diffs (issue #13)#38
Merged
Conversation
chammoru
force-pushed
the
feature/pixel-diff-overlay
branch
from
42f5d25 to
40fb091
Compare
chammoru
force-pushed
the
master
branch
3 times, most recently
from
6a52798 to
ff90ffa
Compare
The image area in each pane is divided into a fixed-size grid in display pixels. Every cell that contains at least one pixel that differs from the reference pane is highlighted with a translucent pink rectangle outline plus a center dot, drawn with GDI+ alpha blending so the underlying image stays visible. Because the cell size is fixed in display pixels, zooming in implicitly subdivides the source region each cell covers — at maximum zoom each dot resolves to a single differing source pixel. The overlay is hidden automatically once zoom is high enough to render per-pixel value labels, since the labels already convey the diff. 'D' toggles the overlay on and off (default on); '?' shows or hides the new Comparer shortcut panel. docs/USER_GUIDE.md gains a Comparer controls table and a mention of the diff overlay; the Viewer table now also lists the 'M' mute shortcut. Per-row red diff lines in PosInfoView are removed (the grid overlay covers the same information). The timeline is hidden when neither pane is a video; frame picking is still available for video sources.
chammoru
force-pushed
the
feature/pixel-diff-overlay
branch
from
40fb091 to
f3d2a2d
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pilot for #13. Not for merge yet — sharing the build for visual review.
What changed
How to try
Open questions