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.

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

GUI: small changes/additions #155

Merged
merged 3 commits into from Nov 12, 2018
Merged

GUI: small changes/additions #155

merged 3 commits into from Nov 12, 2018

Conversation

@KrahJohlito
Copy link
Contributor

KrahJohlito commented Nov 7, 2018

Pull Request checklist

Note: these are not necessarily requirements

  • I reformatted the code with clang-format
  • I checked to make sure my submission worked
  • I am the author of submission or have permission from the original author
  • Requires update of the PS2SDK
  • Requires update of the gsKit
  • Others (please specify below)

Pull Request description

Default Info Page
The info page pulls text from the users per-game CFG files so whatever language the 'attribute text' is written in is what will be displayed. The only items that were not translatable was the 'static text' that I had previously included (now removed).

GFX
Graphics have been converted to 8-bit PNG (via pngquant) & power of 2 to optimize: File Size, Dimensions, Bit Depth & Image Quality.
D-Pad Icons have been scaled to 20*20.
X & Y Positions of all GFX has not been changed at all so everything will be in exactly the same layout/position as before.
L3 & R3 button icons are included in the PR for completeness - in case anyone wants to map something to them in the future (they are not compiled into the ELF so will have no impact on the final size).

@rickgaiser has already scaled the other button icons to 20*20 in commit db1e1ac as he was planning to update the default GFX himself but I said I would take care of it.

Test-Build
http://frd.li/dc75fcf91e0a47fd12c73ef4776ba666

@TnA-Plastic

This comment has been minimized.

Copy link

TnA-Plastic commented Nov 11, 2018

Definitely worth a merge!

@ElPatas1 ElPatas1 merged commit f6d905d into ifcaro:master Nov 12, 2018
@TnA-Plastic

This comment has been minimized.

Copy link

TnA-Plastic commented Nov 12, 2018

THX @ElPatas1 for the merge!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.