Skip to content

Add path copy buttons for Text comparison results in HTML reports - #248

Merged
Widthdom merged 2 commits into
mainfrom
feat/text-detail-copy-path
Jul 29, 2026
Merged

Add path copy buttons for Text comparison results in HTML reports#248
Widthdom merged 2 commits into
mainfrom
feat/text-detail-copy-path

Conversation

@Widthdom

@Widthdom Widthdom commented Jul 29, 2026

Copy link
Copy Markdown
Owner

Summary

  • add IL-style two-file copy buttons to TextMatch and TextMismatch results in HTML reports
  • copy the quoted absolute paths of the old/new files associated with those Text comparison results using native Windows, macOS, or Linux path conventions
  • show the buttons consistently in Unchanged, Modified, and repeated timestamp-warning rows
  • share the path-pair icon between IL and Text buttons while retaining category-specific behavior and tooltips
  • update the sample HTML report, English/Japanese user guide, changelog, and automated coverage

Why

Text comparison results in HTML reports exposed only the relative file path, so opening the corresponding old/new files in an external text-based diff tool required manually locating both files. This brings the existing IL path-copy workflow to TextMatch and TextMismatch results.

User impact

Users can copy both quoted absolute old/new file paths for Text comparison results with one click directly from the HTML report's Diff Detail column. Existing IL path-copy behavior remains unchanged.

Validation

  • dotnet test FolderDiffIL4DotNet.Tests/FolderDiffIL4DotNet.Tests.csproj -c Release --no-restore --filter 'FullyQualifiedName~HtmlReportGenerateServiceTests' — 164 passed
  • npm run test:js -- --runInBand JsTests/diff_report.test.js — 127 passed
  • dotnet test FolderDiffIL4DotNet.Tests/FolderDiffIL4DotNet.Tests.csproj -c Release --no-restore — 2061 passed, 1 skipped
  • npm run test:js -- --runInBand — 190 passed
  • git diff --check

@Widthdom
Widthdom force-pushed the feat/text-detail-copy-path branch from 3ddb954 to ce82f14 Compare July 29, 2026 13:28
@Widthdom Widthdom changed the title Add compared-file path copy buttons to HTML reports Add path copy buttons for Text comparison results Jul 29, 2026
@Widthdom
Widthdom force-pushed the feat/text-detail-copy-path branch from ce82f14 to dfc0fe4 Compare July 29, 2026 13:29
@Widthdom Widthdom changed the title Add path copy buttons for Text comparison results Add path copy buttons for Text comparison results in HTML reports Jul 29, 2026
@Widthdom
Widthdom marked this pull request as ready for review July 29, 2026 14:44
@Widthdom
Widthdom merged commit 9c20360 into main Jul 29, 2026
14 checks passed
@Widthdom
Widthdom deleted the feat/text-detail-copy-path branch July 29, 2026 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant