Output doesn't fit on one line about ansiweather HOT 8 CLOSED

So there are two solutions I can think of:

a) Print each part of the output in its own line.
b) Add even more information to the output (e.g. fetch more data), so it will never fit one line on anybody's terminal and look like a "mistake".

Which is preferable?

from ansiweather.

I think the fact that the location can be of variable width makes it difficult to guarantee 80 columns.

However, I can think of one more solution that might help: add options to disable parts of the output. e.g.,

pressure = false

would make it so pressure isn't displayed.

from ansiweather.

That's a good idea @BurntSushi.

In my Python port of ansiweather, https://github.com/wktr/pyweather, I just removed the default background highlighting and split the output to use multiple lines. I personally favor how it looks. What do you think? I think it is a suitable default, as it is fairly concise in my opinion.

from ansiweather.

@wktr I think it looks nice, although I'm not particularly fussy about it. Just offering up other ideas. :-)

(The other way to go is to make the entire output configurable via a simple template or something. In Python, I'd reach for str.format.)

from ansiweather.

Text labels are now customizable in the configuration file. This solve this kind of issues, at least partially.

Adding options to disable parts of the output could be a nice idea as well. Not sure it's really needed though, as I would prefer keeping the logic as simple as possible :)

from ansiweather.

@fcambus there is a rogue leading space in the output of ansiweather (there is a space in your echo output line), which can only make overflow more likely to happen for no obvious gain...

from ansiweather.

@iandol It's a feature :-) There is a leading and a trailing space, the output looks better that way.

from ansiweather.

Could you leave a pointer as to how this new config solves this issue? How do I add "newline" in the delimiter of .ansiweatherrc?

EDIT: Aha, found it. Change "dashes" to "\n".

from ansiweather.