search

woodruffw / screenfetch-c

A rewrite of screenFetch in C.
MIT License
73 stars 17 forks source link

Modern logo for Windows 8 and 10 #33

Closed darealshinji closed 8 years ago

darealshinji commented 8 years ago

Windows 8, 8.1 and 10 should use the modern logo, like in the shell script version of screenFetch.

woodruffw commented 8 years ago

Definitely. I don't remember where I got most of the original logos from, but using KittyKatt's with a GPLv3 notice in the LICENSE should be fine.

darealshinji commented 8 years ago

The Modern Windows logo was originally submitted by me: https://github.com/KittyKatt/screenFetch/commit/8d962abe4bd15ec86840c3f1daddbf56d8f8ae6d So, if I say you can add it under the MIT license I guess that should be fine (?).

woodruffw commented 8 years ago

Yeah, that should be fine then. I'll commit it shortly.

woodruffw commented 8 years ago

Included it in b7eb30d. I'll close this issue once you confirm things are working smoothly.

darealshinji commented 8 years ago

Works. But now it shows Font: Unknown. Is there even a detection method for "Font" in detect.c?

woodruffw commented 8 years ago

On Windows, only for Cygwin (see the poorly named detect_gtk function).

Font detection on Linux was actually broken because of a clobbered variable until 4a9a61e. It's still not very reliable, though.

darealshinji commented 8 years ago

Okay then. I'll close this issue.