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

Sort the output alphanumerically. #21

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

Conversation

srgvg
Copy link

@srgvg srgvg commented Jan 18, 2018

Make sure the output list shown in dmenu is sorted
alphanumerically, so items are grouped and/or not mixed up

Make sure the output list shown in dmenu is sorted
alphanumerically, so items are grouped and/or not mixed up
@OliverUv
Copy link
Owner

OliverUv commented Feb 7, 2018

Hello @srvg - sorry for not responding for a long time. You've been using this a lot now, right? So I assume it's stable and you haven't encountered any bugs?

If so I'll merge.

@OliverUv
Copy link
Owner

OliverUv commented Feb 7, 2018

Oh, and make sure you've tested it when having windows with UTF-8 in their title, please ~

@srgvg
Copy link
Author

srgvg commented Feb 7, 2018

Hi @OliverUv. No worries about the delay on getting back to me. I know how this works :)

Yes, I have been using this script daily since discovering it, and patching it. I had an initial small bug where the sorting was case sensible, which I updated (and force pushed) the same day.

I now also checked UTF-8 characters, as you can see in following screenshot.

20180207085807

If you have more questions or worries, don't hesitate to get back to me.

Thanks for sharing.

@srgvg
Copy link
Author

srgvg commented Jul 30, 2018

Any remaining issues with this? Has been working nicely since then, for me.

@eugenevd
Copy link

I like this, would be nice to have it merged ;)

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.

3 participants