theCrag / website

theCrag.com: Add your voice and help guide the development of the world's largest collaborative rock climbing & bouldering platform
https://www.thecrag.com/
110 stars 8 forks source link

Error adding multiple Areas/Routes #4060

Open Arzorth opened 2 years ago

Arzorth commented 2 years ago

When adding new nodes, if one of the new nodes is inserted before an existing node and some of the new nodes are created without specifying where to be inserted (this means, at the end) they are inserted before the last specified node instead of being placed at the end of the list.

This could be fixed by adding the option “At the end” in the menu “Insert before”.

Regards

scd commented 2 years ago

If insert before is empty it goes at the end

On Sun., 8 May 2022, 8:45 am Arzorth, @.***> wrote:

When adding new nodes, if one of the new nodes is inserted before an existing node and some of the new nodes are created without specifying where to be inserted (this means, at the end) they are inserted before the last specified node instead of being placed at the end of the list.

This could be fixed by adding the option “At the end” in the menu “Insert before”.

Regards

— Reply to this email directly, view it on GitHub https://github.com/theCrag/website/issues/4060, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAC3CQQ66IJ5RO7DPV7N4LLVI3XB3ANCNFSM5VLAWH6A . You are receiving this because you are subscribed to this thread.Message ID: @.***>

Arzorth commented 2 years ago

Not always, if you insert multiple elements and the first goes before an existent element, the following ones, if the field is left empty, will be inserted before the previously specified one instead of going at the end of the list.

Arzorth commented 2 years ago

Lets say we have

A C D

and we want to insert at the same time B, E, F

If we do: B (insert before: C) E (insert before: -empty-) F (insert before: -empty-)

we will end up with the following list:

A B E F C D

killakalle commented 2 years ago

I can confirm that behavior. image

rouletout commented 1 year ago

See also https://github.com/theCrag/website/issues/4162