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.

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

Reordering checklists works strangely when "Move checked items to the bottom" is checked #59

Open
7 tasks done
nemliza opened this issue Jul 28, 2024 · 4 comments
Open
7 tasks done
Assignees
Labels
bug Something is not working

Comments

@nemliza
Copy link

nemliza commented Jul 28, 2024

Checklist

  • I can reproduce the bug with the latest version given here.
  • I made sure that there are no existing issues - open or closed - to which I could contribute my information.
  • I made sure that there are no existing discussions - open or closed - to which I could contribute my information.
  • I have read the FAQs inside the app (Menu -> About -> FAQs) and my problem isn't listed.
  • I have taken the time to fill in all the required details. I understand that the bug report will be dismissed otherwise.
  • This issue contains only one bug.
  • I have read and understood the contribution guidelines.

Affected app version

1.1.0

Affected Android/Custom ROM version

Android 13

Affected device model

Zenfone 8

How did you install the app?

F-Droid / IzzyOnDroid

Steps to reproduce the bug

  1. Add a new checklist
  2. Add items
  3. Check some of the items
  4. Make sure that the option "Move checked items to the bottom" is checked under Sort by
  5. Long tap an unchecked item and try to reorder the list by dragging it

Expected behavior

The dragged item stays in the place where it was dragged

Actual behavior

The dragged item jumps to a different position (mostly when dragged upwards from the bottom of the list)

Screenshots/Screen recordings

No response

Additional information

No response

@nemliza nemliza added bug Something is not working needs triage Issue is not yet ready for PR authors to take up labels Jul 28, 2024
@Aga-C Aga-C removed the needs triage Issue is not yet ready for PR authors to take up label Jul 28, 2024
@naveensingh
Copy link
Member

I could not reproduce this, how many items do you have?

@Aga-C
Copy link
Member

Aga-C commented Jul 28, 2024

@naveensingh

  1. Open any checklist and set sorting by date, with moving items to the bottom.
  2. Create a new checklist and add some items to it.
  3. Check some items.
  4. Try to move any non-checked item.

I've tested on two devices and on one I'm always able to reproduce it (Android 14), and on the other it sometimes works properly (Android 13).

EDIT: It appears to be very random. Now, on Android 14, I have troubles with reproducing it. Once it works, once it doesn't work. However, when I turned on the toast informing about saving a list, it appears that the checklist is constantly saving during dragging.

@nemliza
Copy link
Author

nemliza commented Jul 28, 2024

I could not reproduce this, how many items do you have?

I just tried with eight items, check this video.

@naveensingh
Copy link
Member

I can reproduce it now, thanks guys!

@naveensingh naveensingh self-assigned this Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working
Projects
None yet
Development

No branches or pull requests

3 participants