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

Update gfx for BDM #422

Merged
merged 3 commits into from
May 25, 2021
Merged

Update gfx for BDM #422

merged 3 commits into from
May 25, 2021

Conversation

KrahJohlito
Copy link
Member

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

Just updating gfx for BDM.

In order to maintain theme compatibility I've named the BDM device icon as usb.png and all block devices handled by BDM with a _bd suffix. The idea is to change the const int IDs of both the device text and icon to simple functions to determine which string/icon to display depending on which device is connected and the standard BDM icon/text for manual init or when it is set to OFF.

Currently though I'm having difficulty obtaining the device name without hacks, perhaps when/if there are EE side functions for BDM this will be easier so for now not all icons are used.

Thanks to Berion for the updated gfx.

@rickgaiser
Copy link
Member

Nice work on the icons, thanks!

@rickgaiser rickgaiser merged commit de4fcd4 into ps2homebrew:master May 25, 2021
@J013k
Copy link
Contributor

J013k commented May 29, 2021

What about line 35 (USB Games) in English template:
https://github.com/ps2homebrew/Open-PS2-Loader/blob/master/lng/lang_English.lng#L35
Does this line also should be changed into BDM Games?

I am asking because I want to update all the translations...

EDIT: I have noticed that there is a string BDM Games in line 303:
https://github.com/ps2homebrew/Open-PS2-Loader/blob/master/lng/lang_English.lng#L303.

So do we need string\line 35 or 303? Or one of this line can be "replaced", the other one deleted?

@rickgaiser
Copy link
Member

Originally I re-used the string in line 35, but this has now been changed to a new string on line 303. I would prefer re-using the string on line 35 so we don't get too many dead items in the language files, but this requires a partial revert of 991644f. @KrahJohlito do you agree?

This should be renamed back to _STR_BDM_GAMES

_STR_USB_GAMES,

And this should be removed:

_STR_BDM_GAMES,

@KrahJohlito
Copy link
Member Author

KrahJohlito commented Jun 23, 2021

Sorry @rickgaiser I must have mentions off I only just saw this when @J013k messaged me about it today.

In the next couple days I plan to use both language strings as well as adding another for FireWire Games (although I’m not sure it’s even necessary to have a seperate string for each device since only the word “Games” would need to be translated I assume). I’ll make sure template and Lang.c are consistent, sorry for the confusion.

@KrahJohlito KrahJohlito deleted the bdm-gui branch June 24, 2021 12:20
AKuHAK pushed a commit that referenced this pull request Sep 30, 2021
citronalco pushed a commit to citronalco/OPL-Daily-Builds that referenced this pull request Sep 10, 2023
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants