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.

Sign up for GitHub

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

Jump to bottom

quarks: sort list by name #2214

Merged
merged 1 commit into from
Jun 30, 2016
Merged

quarks: sort list by name #2214

merged 1 commit into from
Jun 30, 2016

Conversation

gusano
Copy link
Member

@gusano gusano commented Jun 29, 2016

Fixes #2212

@nhthn
Copy link
Contributor

nhthn commented Jun 29, 2016

lgtm

@crucialfelix
Copy link
Member

See my comments on #2212 I think that's a better solution.

@crucialfelix
Copy link
Member

I see what you did here - that does work to make the names in order. It was the dictionary that randomized it.

What I mentioned in the other comment about squirming when you un/install is still an issue, but we could merge this PR here and it would solve the main problem now.

@gusano
Copy link
Member Author

gusano commented Jun 30, 2016

👍

@crucialfelix crucialfelix added this to the 3.8 milestone Jun 30, 2016
@crucialfelix
Copy link
Member

interesting: it now orders them by installed, those you've checked out, those you've never checked out.

and each section is alphabetical

screen shot 2016-06-30 at 12 42 42

@crucialfelix crucialfelix merged commit 1746abc into supercollider:master Jun 30, 2016
@crucialfelix crucialfelix mentioned this pull request Jul 1, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
quarks
Projects
None yet
Milestone
3.8
Development

Successfully merging this pull request may close these issues.
3 participants
@gusano @nhthn @crucialfelix