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

Domino style menu to show approximate ping times at a glance #6

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

nicpottier
Copy link

Hey Antirez, big fan of yours, we use Redis a ton in RapidPro and it is a lifesaver, always been impressed by your pragmatism and creation of useful tools.

Anyways, I spend a lot of time abroad on flakey connections and like having something like iconping running all the time. Not sure if you still use it, but the icon kind of drove me nuts so I made some small tweaks to change the icon to be a domino, between 1-9 representing the ping time. That tells me at a quick glance whether my ping times are ~<100ms or ~300ms or ~700ms or something greater (in which case it shows an exclamation mark)

Anyways, all pretty self explanatory from the code changes and icon files below. I made them at both resolutions so it looks good on retina displays.

Made these changes and been using it for six months or so and kinda forgot about it, but thought you might like it if you still use this in your daily life.

Either way, cheers!

@antirez
Copy link
Owner

antirez commented Nov 12, 2016

Thanks @nicpottier! Yes I still use iconping regularly, the change looks very useful, I'll install XCode in order to try it and merge. Thanks.

@nicpottier
Copy link
Author

Great, ya, feel free to refactor it, was mostly just a toy thing for me, so lots of copy/paste in there instead of figuring out the right way to do it in ObjC etc..

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