moe icon indicating copy to clipboard operation
moe copied to clipboard

Fix commas placement with lists

Open Bregor opened this issue 7 years ago • 1 comments

After parsing MAL's html, there is strange behaviour of the commas in strings with multiple values:

$ moe -name "shirobako" -info
...
Producers:  Sotsu  , Movic  , Warner Bros.  , KlockWorx  , Showgate  , Infinite
Genres   :  Comedy  , Drama
...

After this patch commas looks like following:

$ moe -name "shirobako" -info
...
Genres   :  Comedy, Drama
Producers:  Sotsu, Movic, Warner Bros., KlockWorx, Showgate, Infinite
...

Bregor avatar Dec 08 '17 22:12 Bregor

Thanks for the PR but it would be better if you made the changes under the comment extract info in fetchDetails https://github.com/dragonzurfer/moe/blob/master/moe.go#L240 to maintain readability since PrintParams is just for printing to terminal.

dragonzurfer avatar Dec 09 '17 12:12 dragonzurfer