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

noping: boxless drawing option #8

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

noping: boxless drawing option #8

wants to merge 3 commits into from

Conversation

l29ah
Copy link

@l29ah l29ah commented Nov 11, 2015

No description provided.

@octo
Copy link
Owner

octo commented Jul 14, 2016

Hi @l29ah, thank you very much for your patch and sorry for the late reply!

Could you share a little about the motivation for this change? It seems to be quite a lot of extra conditionals for quite a small difference in appearance …

Best regards,
—octo

@octo octo added the Feature label Jul 14, 2016
@l29ah
Copy link
Author

l29ah commented Jul 14, 2016

On Wed, Jul 13, 2016 at 10:52:25PM -0700, Florian Forster wrote:

Hi @l29ah, thank you very much for your patch and sorry for the late reply!

Could you share a little about the motivation for this change? It seems to be quite a lot of extra conditionals for quite a small difference in appearance …

The boxes waste screen space, and that matters when you have several addresses to ping and have the pinger window occupying a small portion of the display. Maybe this is not the most optimal way to do this, as i'm new to ncurses, but they all look necessary to me.

() ascii ribbon campaign - against html mail
/\ http://arc.pasp.de/ - against proprietary attachments

@l29ah
Copy link
Author

l29ah commented Oct 22, 2016

Sup?

1 similar comment
@l29ah
Copy link
Author

l29ah commented Jun 17, 2017

Sup?

@l29ah
Copy link
Author

l29ah commented Aug 24, 2017

:(

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

Successfully merging this pull request may close these issues.

2 participants