-
Notifications
You must be signed in to change notification settings - Fork 384
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
order firmware groups in the dialog and in the code #4087
Conversation
there's some discrepancy between the 2 orders, because the GUI list relies on full names (of the groups) while in the code short names are used to add them to DB
I like this. Being prompted for a firmware and then having to search through the (seemingly) unsorted list is very annoying. |
The sort call is fine, but why are you then also sorting in the source? |
Because it's a nightmare to scroll through unsorted mess every time you need to add/edit something. |
Ctrl+F? |
Dude seriously? You actually need it to be unsorted, and searching it by trying to remember what the short name is is absolutely the only allowed way? |
IDRC, but it would have been nice to keep the diff clean. |
Why is clean diff better than clean code? It's reordered once and then it's always predictable where things are, but if the diff was the only thing that mattered, the code would always be a mess. That's a weird priority for something that's not just cosmetic but functional. |
It clobbers the blame, and
|
Yeah sorry. Cleanest diff is no diff. Maybe commit description will help in such a dire situation. |
there's some discrepancy between the 2 orders, because the GUI list relies on full names (of the groups) while in the code short names are used to add them to DB