Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Demo playlist view item can be draggable #275

Open
1 task
StaehliJ opened this issue Oct 24, 2023 · 2 comments
Open
1 task

Demo playlist view item can be draggable #275

StaehliJ opened this issue Oct 24, 2023 · 2 comments
Assignees
@MGaetan89
Labels
enhancement New feature or request good first issue Good for newcomers
Milestone
Demo

Comments

@StaehliJ
Copy link
Contributor

As a user I want to reorder the playlist by moving item with touch event too.

Acceptance criteria

  • Playlist item reorder is still accessible
  • Playlist items are draggable to reorder the current playlist

Tasks

  • ...
@StaehliJ StaehliJ converted this from a draft issue Oct 24, 2023
@StaehliJ StaehliJ added good first issue Good for newcomers enhancement New feature or request labels Oct 24, 2023
@StaehliJ StaehliJ added this to the 1.0.0 milestone Oct 24, 2023
@MGaetan89 MGaetan89 moved this from 📋 Backlog to 🚧 In Progress in Pillarbox Feb 20, 2024
@MGaetan89 MGaetan89 self-assigned this Feb 20, 2024
@MGaetan89 MGaetan89 moved this from 🚧 In Progress to 📋 Backlog in Pillarbox Mar 8, 2024
@MGaetan89
Copy link
Member

MGaetan89 commented Mar 12, 2024 •
edited
Loading

The current solution provided by Compose doesn't seem to support Drap&Drop in LazyList (or in scrollable container) yet.
I've pushed my initial work in the branch linked in the Development section for future reference, when Compose will support this use case.

@MGaetan89 MGaetan89 removed this from the 1.0.0 milestone Mar 15, 2024
@MGaetan89 MGaetan89 added this to the Demo milestone May 28, 2024
@MGaetan89
Copy link
Member

I've updated the branch to include the recent changes in main and remove the Up/Down buttons from the playlist view.
Probably an other approach should be used to implement this feature. According to Compose's Drag & Drop documentation:
Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MGaetan89 MGaetan89

Labels
enhancement New feature or request good first issue Good for newcomers
Projects
Pillarbox
Status: 📋 Backlog
Milestone
Demo
Development

When branches are created from issues, their pull requests are automatically linked.

275-demo-playlist-view-item-can-be-draggable SRGSSR/pillarbox-android
2 participants
@MGaetan89 @StaehliJ