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

Improve sort of SecurityTaxonomyPage by weight #4341

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Nirus2000
Copy link
Member

grafik

@Morpheus1w3
Copy link
Contributor

Sorry but this make updates more challenging because if you're updating a list of attributes, you have to read the whole list until you'll find the value who need to be updated. Alphabetical sort sequence is might preferable from my point of view.

@Nirus2000
Copy link
Member Author

Nirus2000 commented Nov 30, 2024

Hello @buchen, @Morpheus1w3 and @auchri
The background here is that the sense of securities editing used and displayed correctly. It makes no sense to go by category when updating the weightings, as the total weighting can only be 100%, regardless of the number of categories or subcategories.

The point is to display a weighting correctly in order to be able to update the data more quickly, but not to sort the categories. These are already sorted alphabetically in the dropdown field.

It would also make no sense to sort an index of a book by the first letter of an index, but first by the page number and then by the index. 😉

Example:

Regards
Alex

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants