anAcc22 / another_graph_editor

A graph editor
https://anacc22.github.io/another_graph_editor/
MIT License
96 stars 10 forks source link

The order of labels issue #3

Closed guosoun closed 2 months ago

guosoun commented 2 months ago

If the nodes are numbered from 1 to n, and n is a two-digit number, then Current Nodes will sort the nodes in lexicographical order, such as 1 10 2 3 4 5 6 7 8 9, while the order of labels given in the problem is often 1 2 3 4 5 6 7 8 9 10, which makes editing labels very annoying.

One solution I came up with is to add a switch to adjust the sorting method, switching between lexicographical order and numerical order from smallest to largest. However, when there are non-numerical values in Current Nodes, the numerical order from smallest to largest should be disabled

anAcc22 commented 2 months ago

Hi, as a quickfix for now, would you be alright if I made the default sorting method such that all numbers go in front of non-numerical values, and these numbers will be sorted in numerical order, while the non-numerical values will be sorted in lexicographical order.

When I eventually start working on the new UI, I'll add more options for users to select their preferred sorting method.

guosoun commented 2 months ago

Thank you for your quick response and the proposed workaround; I think it’s a great solution! I am looking forward to the development of the new UI and trying out more sorting options. Thanks again for your support and efforts!

anAcc22 commented 2 months ago

It has just been implemented, do let me know if the new behavior is weird/buggy (you might have to use ctrl+shift+r if the change doesn't show up right away). Thanks for trying out this tool and your feedback/suggestions are always welcome!