Skip to content

Conversation

@iiAtlas
Copy link
Owner

@iiAtlas iiAtlas commented Oct 22, 2025

This commit enables the GUI to properly render local image references in markdown files (e.g., Image).

Changes:

  • Modified ToHTML() to accept markdown file path parameter
  • Added processLocalImages() function to convert relative image paths to base64-encoded data URIs
  • Updated app.go to pass file path to ToHTML()
  • Updated all test cases to pass empty string for new parameter
  • Added example markdown file with test images

The solution converts local images to data URIs, which are embedded directly in the HTML. This works seamlessly in the Wails webview without requiring file system access from the frontend.

Supported image formats: PNG, JPEG, GIF, SVG, WebP, BMP, ICO

🤖 Generated with Claude Code

@github-actions
Copy link

🧪 Test Release Summary

All binaries built and signed successfully! ✅

Version: test-b712521

Artifacts Created

-rw-r--r-- 1 runner runner  18M Oct 22 02:32 dist/mdv- gui_1.1.1-next_darwin_arm64.tar.gz
-rw-r--r-- 1 runner runner  19M Oct 22 02:32 dist/mdv- gui_1.1.1-next_darwin_x86_64.tar.gz
-rw-r--r-- 1 runner runner 5.8M Oct 22 02:32 dist/mdv- tui_1.1.1-next_linux_arm64.tar.gz
-rw-r--r-- 1 runner runner 6.1M Oct 22 02:32 dist/mdv- tui_1.1.1-next_linux_x86_64.tar.gz
-rw-r--r-- 1 runner runner 5.8M Oct 22 02:32 dist/mdv- tui_1.1.1-next_windows_arm64.zip
-rw-r--r-- 1 runner runner 6.3M Oct 22 02:32 dist/mdv- tui_1.1.1-next_windows_x86_64.zip
-rw-r--r-- 1 runner runner  19M Oct 22 02:32 dist/mdv_1.1.1-next_darwin_arm64.tar.gz
-rw-r--r-- 1 runner runner  20M Oct 22 02:32 dist/mdv_1.1.1-next_darwin_x86_64.tar.gz
-rw-r--r-- 1 runner runner 7.4M Oct 22 02:32 mdv-gui_test-b712521_linux_arm64.tar.gz
-rw-r--r-- 1 runner runner 7.8M Oct 22 02:32 mdv-gui_test-b712521_linux_x86_64.tar.gz
-rw-r--r-- 1 runner runner 8.5M Oct 22 02:32 mdv-gui_test-b712521_windows_x86_64.zip

Linux GUI archives

-rw-r--r-- 1 runner runner 7.4M Oct 22 02:32 mdv-gui_test-b712521_linux_arm64.tar.gz
-rw-r--r-- 1 runner runner 7.8M Oct 22 02:32 mdv-gui_test-b712521_linux_x86_64.tar.gz

Windows GUI archives

-rw-r--r-- 1 runner runner 8.5M Oct 22 02:32 mdv-gui_test-b712521_windows_x86_64.zip

GUI Checksums (SHA256)

05a4a816dd0bc3ddb3b740b200fb4cbc452712d0f1aba3500035f8388bd3c781  mdv-gui_test-b712521_linux_x86_64.tar.gz
d0632734d7441ce4dd7e97aa0c7bd8b7db85af9fc59eca60f35c1abcd7ad970a  mdv-gui_test-b712521_linux_arm64.tar.gz
578685f0e0feae261be8cc280e6ac6181e481fdf9f10a573029f8dbbee1a5607  /Users/runner/work/mdv/mdv/mdv-gui_test-b712521_windows_x86_64.zip

Platform Coverage

  • ✅ macOS (darwin/amd64, darwin/arm64) - Signed & Notarized
  • ✅ Linux (linux/amd64, linux/arm64)
  • ✅ Windows (windows/amd64)

What was tested

  • GUI builds on all platforms
  • Code signing (macOS)
  • Notarization (macOS)
  • Archive creation
  • Checksum generation

Note: This is a test release. No artifacts were published to GitHub Releases or Homebrew.

This commit enables the GUI to properly render local image references
in markdown files (e.g., ![Image](path/to/image.png)).

Changes:
- Modified ToHTML() to accept markdown file path parameter
- Added processLocalImages() function to convert relative image paths
  to base64-encoded data URIs
- Updated app.go to pass file path to ToHTML()
- Updated all test cases to pass empty string for new parameter
- Added example markdown file with test images

The solution converts local images to data URIs, which are embedded
directly in the HTML. This works seamlessly in the Wails webview
without requiring file system access from the frontend.

Supported image formats: PNG, JPEG, GIF, SVG, WebP, BMP, ICO

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@iiAtlas iiAtlas force-pushed the claude/render-local-markdown-images-011CUMRSoZTicntqCQcXJE85 branch from dc103ea to 1fb605f Compare October 22, 2025 20:57
@iiAtlas iiAtlas marked this pull request as ready for review October 22, 2025 21:02
@github-actions
Copy link

🧪 Test Release Summary

All binaries built and signed successfully! ✅

Version: test-25c2b60

Artifacts Created

-rw-r--r-- 1 runner runner  18M Oct 22 21:07 dist/mdv-gui_1.1.1-next_darwin_arm64.tar.gz
-rw-r--r-- 1 runner runner  19M Oct 22 21:07 dist/mdv-gui_1.1.1-next_darwin_x86_64.tar.gz
-rw-r--r-- 1 runner runner 5.8M Oct 22 21:07 dist/mdv-tui_1.1.1-next_linux_arm64.tar.gz
-rw-r--r-- 1 runner runner 6.1M Oct 22 21:07 dist/mdv-tui_1.1.1-next_linux_x86_64.tar.gz
-rw-r--r-- 1 runner runner 5.8M Oct 22 21:07 dist/mdv-tui_1.1.1-next_windows_arm64.zip
-rw-r--r-- 1 runner runner 6.3M Oct 22 21:07 dist/mdv-tui_1.1.1-next_windows_x86_64.zip
-rw-r--r-- 1 runner runner  19M Oct 22 21:07 dist/mdv_1.1.1-next_darwin_arm64.tar.gz
-rw-r--r-- 1 runner runner  20M Oct 22 21:07 dist/mdv_1.1.1-next_darwin_x86_64.tar.gz
-rw-r--r-- 1 runner runner 7.4M Oct 22 21:07 mdv-gui_test-25c2b60_linux_arm64.tar.gz
-rw-r--r-- 1 runner runner 7.8M Oct 22 21:07 mdv-gui_test-25c2b60_linux_x86_64.tar.gz
-rw-r--r-- 1 runner runner 8.5M Oct 22 21:07 mdv-gui_test-25c2b60_windows_x86_64.zip

Linux GUI archives

-rw-r--r-- 1 runner runner 7.4M Oct 22 21:07 mdv-gui_test-25c2b60_linux_arm64.tar.gz
-rw-r--r-- 1 runner runner 7.8M Oct 22 21:07 mdv-gui_test-25c2b60_linux_x86_64.tar.gz

Windows GUI archives

-rw-r--r-- 1 runner runner 8.5M Oct 22 21:07 mdv-gui_test-25c2b60_windows_x86_64.zip

GUI Checksums (SHA256)

73ae64a57c3d3a24ba03325d561a143568b6e5d158367db8c05a8c0a0f38529a  mdv-gui_test-25c2b60_linux_x86_64.tar.gz
ae258f57053cb2d071ad55e7b5b36eccb5c2d12ad4fdc79ae0df10a139e8ab48  mdv-gui_test-25c2b60_linux_arm64.tar.gz
00eae568e773270166cd5d8a42bac4608315330b59ecf3b9336c4bb40259d1ba  /Users/runner/work/mdv/mdv/mdv-gui_test-25c2b60_windows_x86_64.zip

Platform Coverage

  • ✅ macOS (darwin/amd64, darwin/arm64) - Signed & Notarized
  • ✅ Linux (linux/amd64, linux/arm64)
  • ✅ Windows (windows/amd64)

What was tested

  • GUI builds on all platforms
  • Code signing (macOS)
  • Notarization (macOS)
  • Archive creation
  • Checksum generation

Note: This is a test release. No artifacts were published to GitHub Releases or Homebrew.

@github-actions
Copy link

🧪 Test Release Summary

All binaries built and signed successfully! ✅

Version: test-422886c

Artifacts Created

-rw-r--r-- 1 runner runner  18M Oct 22 21:12 dist/mdv-gui_1.1.1-next_darwin_arm64.tar.gz
-rw-r--r-- 1 runner runner  19M Oct 22 21:12 dist/mdv-gui_1.1.1-next_darwin_x86_64.tar.gz
-rw-r--r-- 1 runner runner 5.8M Oct 22 21:12 dist/mdv-tui_1.1.1-next_linux_arm64.tar.gz
-rw-r--r-- 1 runner runner 6.1M Oct 22 21:12 dist/mdv-tui_1.1.1-next_linux_x86_64.tar.gz
-rw-r--r-- 1 runner runner 5.8M Oct 22 21:12 dist/mdv-tui_1.1.1-next_windows_arm64.zip
-rw-r--r-- 1 runner runner 6.3M Oct 22 21:12 dist/mdv-tui_1.1.1-next_windows_x86_64.zip
-rw-r--r-- 1 runner runner  19M Oct 22 21:12 dist/mdv_1.1.1-next_darwin_arm64.tar.gz
-rw-r--r-- 1 runner runner  20M Oct 22 21:12 dist/mdv_1.1.1-next_darwin_x86_64.tar.gz
-rw-r--r-- 1 runner runner 7.4M Oct 22 21:12 mdv-gui_test-422886c_linux_arm64.tar.gz
-rw-r--r-- 1 runner runner 7.8M Oct 22 21:12 mdv-gui_test-422886c_linux_x86_64.tar.gz
-rw-r--r-- 1 runner runner 8.5M Oct 22 21:12 mdv-gui_test-422886c_windows_x86_64.zip

Linux GUI archives

-rw-r--r-- 1 runner runner 7.4M Oct 22 21:12 mdv-gui_test-422886c_linux_arm64.tar.gz
-rw-r--r-- 1 runner runner 7.8M Oct 22 21:12 mdv-gui_test-422886c_linux_x86_64.tar.gz

Windows GUI archives

-rw-r--r-- 1 runner runner 8.5M Oct 22 21:12 mdv-gui_test-422886c_windows_x86_64.zip

GUI Checksums (SHA256)

d0dc476e17ffc7fd16f162d259bd31b62f0d412dbbd20a0064f624660f2fc916  mdv-gui_test-422886c_linux_x86_64.tar.gz
88217bb5e6577e417829c4dfa5a1602ffe184170aa4a0c329cd2fec8f1738b7a  mdv-gui_test-422886c_linux_arm64.tar.gz
85d3cc443c325755704fc8d7d4858023157614c5b0e410c07cefc7b8d750c653  /Users/runner/work/mdv/mdv/mdv-gui_test-422886c_windows_x86_64.zip

Platform Coverage

  • ✅ macOS (darwin/amd64, darwin/arm64) - Signed & Notarized
  • ✅ Linux (linux/amd64, linux/arm64)
  • ✅ Windows (windows/amd64)

What was tested

  • GUI builds on all platforms
  • Code signing (macOS)
  • Notarization (macOS)
  • Archive creation
  • Checksum generation

Note: This is a test release. No artifacts were published to GitHub Releases or Homebrew.

@iiAtlas iiAtlas merged commit b591b4f into main Oct 22, 2025
7 checks passed
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.

2 participants