Skip to content

fix player icons not visible#197

Merged
RadicalMuffinMan merged 1 commit into
Moonfin-Client:mainfrom
nullbyze:fix_svg_icons
May 4, 2026
Merged

fix player icons not visible#197
RadicalMuffinMan merged 1 commit into
Moonfin-Client:mainfrom
nullbyze:fix_svg_icons

Conversation

@nullbyze
Copy link
Copy Markdown
Contributor

@nullbyze nullbyze commented May 3, 2026

Pull Request

Summary

corrected svg imports to resolve proper asset URLs (broken with latest commit 5f34080)

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Performance improvement
  • UI/UX update
  • Documentation update
  • Build/CI change
  • Other (describe):

Platform

  • Tizen (Samsung)
  • webOS (LG)
  • Both / Shared code

Testing

Describe how this change was tested.

  • Tested on emulator
  • Tested on physical device
  • Manual testing completed
  • Not tested (explain why):

Test Steps

  1. open player
  2. player icons should be visible
  3. done

Checklist

  • Code builds successfully
  • Code follows project style and conventions
  • No unnecessary commented-out code
  • No new warnings introduced

fix: correct svg imports to resolve proper asset URLs
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 3, 2026

✅ Build Successful

All platform builds passed. You can download the test artifacts below.

Platform Status Artifact
webOS ✅ Passed Moonfin_webOS_*.ipk
Tizen Regular ✅ Passed Moonfin_Tizen_Regular_*.wgt
Tizen Oblong ✅ Passed Moonfin_Tizen_Oblong_*.wgt
Tizen Legacy ✅ Passed Moonfin_Tizen_Legacy_*.wgt
Property Value
Commit 4f56308
Workflow run Build #74

@RadicalMuffinMan
Copy link
Copy Markdown
Contributor

interesting, showed up fine on the webos 4 emulator
image

@RadicalMuffinMan
Copy link
Copy Markdown
Contributor

Good catch though, thank you.

@RadicalMuffinMan RadicalMuffinMan merged commit da81ca9 into Moonfin-Client:main May 4, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants