Skip to content

add wrap-around navigation for media rows#178

Merged
RadicalMuffinMan merged 3 commits into
Moonfin-Client:mainfrom
999romain:feat/wrap-around
May 10, 2026
Merged

add wrap-around navigation for media rows#178
RadicalMuffinMan merged 3 commits into
Moonfin-Client:mainfrom
999romain:feat/wrap-around

Conversation

@999romain
Copy link
Copy Markdown
Contributor

Pull Request

Summary

Add wrap-around navigation for media rows so users can jump from the leftmost item to the rightmost item, and vice versa, instead of pressing the remote repeatedly.

Related Issues

Type of Change

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

Changes Made

List the key changes included in this PR.

  • Added "wrap-around focus" behavior on media rows.
  • Kept navigation consistent with the current navbar position.

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 a media row.
  2. Move focus to the first item and press left.
  3. Move focus to the last item and press right.

Checklist

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

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 13, 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 338503b
Workflow run Build #72

@RadicalMuffinMan
Copy link
Copy Markdown
Contributor

The new workflow for PR test builds is pretty cool right?

@RadicalMuffinMan RadicalMuffinMan merged commit 0395c85 into Moonfin-Client:main May 10, 2026
4 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