KittyKatt / screenFetch

Fetches system/theme information in terminal for Linux desktop screenshots.
GNU General Public License v3.0
3.88k stars 455 forks source link

Fixed bug displaying font type on macos #595

Closed coughll closed 6 years ago

coughll commented 6 years ago

Heres my PR from this issue i submitted earlier here: https://github.com/KittyKatt/screenFetch/issues/594

A simple one-line fix to remove the erroneous characters before the actual font name. I tested it on a couple different stock mono-spaced fonts and it works for me. The 288 was an arbitrary number i chose because it was sitting right above it in the code, and because i don't know how to make it always select the last char of the string. There were/are no excess characters behind the font name, only before, but font names can be fairly long, so I gave it plenty of space.

The real problem was that the script was picking up 21 random characters(including a newline) from the defaults read com.apple.Terminal "Window Settings" command and the parsing lines. So I just cut them out of the string.