mirror of
https://github.com/go-vikunja/vikunja.git
synced 2026-03-09 07:13:35 -05:00
Task list items with same name are linked together #462
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @vikunja-bot on GitHub (Apr 1, 2025).
Original issue by polgarc on 2024-01-30T19:03:18.000Z
Description
If I copy-paste a task item, they become kind of linked together: if I click on one of them, all the copies are ticked too.
This doesn't seem to be 100% consistent though: sometime not all of them get ticked, overall it's very confusing experience.
After some more testing it seems that the issue is not related to the copy-paste, rather the items having the same name. If you simply type the same item names instead of copy-paste, the issue occurs.
Steps to reproduce:
Vikunja Frontend Version
0.22.1+10-0484923b8a
Vikunja API Version
v0.22.1+2-78353d1ffe
Browser and version
Firefox 122.0 (Linux)
Can you reproduce the bug on the Vikunja demo site?
Yes
Screenshots
item2was copied a few times:After clicking on the first
item2:After clicking on the first
item2again, they look like on the first screen shot.Original issue on Gitea
@kolaente commented on 2024-02-05T11:04:16.000Z:
Well that should not happen. I'll take a look.
@kolaente commented on 2024-02-14T13:14:32.000Z:
Fixed in
3969f6ae66- please check with the next unstable build (ready in ~45 min) or on try.polgarc commented on 2024-02-14T16:00:16.000Z:
Thank you!
However I checked the demo instance (
v0.23.0+46-827c43fe12) , and it seems all items have the samedata-id, and the issue still persists. Tried with Firefox 122.0 and Chromium 121.0.6167.160 on Linux.@kolaente commented on 2024-02-17T11:26:44.000Z:
Now fixed in
32edef2d38.polgarc commented on 2024-02-17T21:42:44.000Z:
I'm sorry but this still doesn't work, tried with
v0.23.0+49-32edef2d38. Also checked out and ranmainlocally, same: all items have the same id.I think createRandomID() is only evaluated once, and not for each
TaskItem. According to this issue, this approach won't even work.BTW thanks for your time and effort - really appreciate it.
@kolaente commented on 2024-02-17T21:50:26.000Z:
Looks like this is a far deeper problem rooted in tiptap. I've tried the last few hours to make this work without actually getting to it. The root cause is from the comparison of nodes. If we take the text only, that will mark items as duplicate when they have the same text (as described here originally).
And what confuses me the most is how it sometimes is able to assign a random id to each item, just not consistently (I've tested this locally before closing this issue and it seemed to work fine).
Seems like their unique ID extension is the way out here, but that's probably not an option for an open-source project. I've emailed them about this, let's see what we get back.
@kolaente commented on 2024-02-17T22:16:19.000Z:
I've reverted the changes for now so that it works as long as the task items have different text. Moved the id changes so far to a new branch: https://kolaente.dev/vikunja/vikunja/compare/main...fix/tiptap-task-list
@kolaente commented on 2024-03-05T22:10:02.000Z:
Also mentioned here: https://community.vikunja.io/t/task-list-in-description-glitchy/2087/3
polgarc commented on 2024-06-10T06:36:38.000Z:
@dpschen May I ask why was this closed? It's still an issue, tried with the latest commit (
v0.23.0+698-165d046307).polgarc commented on 2024-10-15T09:14:18.000Z:
This is still an issue, just tried with
v0.24.1-313-3e9c41cfc6on https://try.vikunja.io/.stepanov1975 commented on 2024-10-15T13:30:01.000Z:
Yep, the task list still completely broken