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.

Sign up for GitHub

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

Jump to bottom

Modern/Cartoon skin #4265

Merged
merged 2 commits into from
May 7, 2020
Merged

Modern/Cartoon skin #4265

merged 2 commits into from
May 7, 2020

Conversation

qwertychouskie
Copy link
Contributor

@qwertychouskie qwertychouskie commented Apr 9, 2020 •
edited
Loading

The culmination of years worth of work! Thanks to all who helped make this as amazing as possible!
PNGs were ran through pngquant then optimize_data.sh for maximum smallness.
213 items, totalling 1.5 MiB (1,576,115 bytes)

thatfiredev
thatfiredev approved these changes Apr 9, 2020
View reviewed changes
Copy link

@thatfiredev thatfiredev left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just gave it a try. Looks amazingly good!!

@Benau
Copy link
Contributor

Benau commented Apr 10, 2020

Not sure @Alayan-stk-2 is isolated because of chinese virus lol.....

@Benau
Copy link
Contributor

Benau commented Apr 10, 2020

data/gui/icons/android.png is not used anymore

Maybe it's time to bring back font thickness code to save ttf to make distro happy

@qwertychouskie
Modern skin
2e0a1dd 
The culmination of years worth of work!  Thanks to all who helped make this as amazing as possible!
PNGs were ran through `pngquant` then `optimize_data.sh` for maximum smallness
@qwertychouskie
Copy link
Contributor Author

@Benau Icons for Android/Windows/Mac/Linux removed, we can add these extra icons to the media repo later.

@qwertychouskie
Copy link
Contributor Author

5 items, totalling 618.4 KiB (633,272 bytes) As for the TTFs, distros will work their magic as they already do with the existing TTFs, and the size is small enough that Android shouldn't be really affected (especially with app bundle support coming).

@Benau
Copy link
Contributor

Benau commented Apr 11, 2020

every bytes matter...

@qwertychouskie
Shrink a multitude of icons
56c5bff 
Ran through `optipng` then `optimize_data.sh`
@Benau
Copy link
Contributor

Benau commented Apr 28, 2020

...Alayan....

@qwertychouskie
Copy link
Contributor Author

@Alayan-stk-2 ^

@Benau Real pings generally work better ;)

@Benau Benau merged commit a776650 into supertuxkart:master May 7, 2020
@Benau
Copy link
Contributor

Benau commented May 7, 2020

Alayan most stk fans and even deve think it's a good idea to include this so i will go ahead merge it...

Let hope the chinese virus doesn't affect you too much...

@deveee
Copy link
Member

deveee commented May 7, 2020

Congratulations Qwerty!

@qwertychouskie qwertychouskie deleted the new-skin-upstream branch May 7, 2020 18:06
@qwertychouskie qwertychouskie changed the title Modern skin Modern/Cartoon skin Nov 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thatfiredev thatfiredev thatfiredev approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@qwertychouskie @Benau @deveee @thatfiredev