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

Updates and fixes #208

Merged
merged 6 commits into from
Aug 13, 2019
Merged

Updates and fixes #208

merged 6 commits into from
Aug 13, 2019

Conversation

Tupakaveli
Copy link
Contributor

@Tupakaveli Tupakaveli commented Jul 22, 2019

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

Memory Card icon update.

Discussion: https://www.psx-place.com/threads/open-ps2-loader-v0-9-3.13415/page-40#post-193454
To test: https://www.sendspace.com/file/emn0nc

Replace slide transition with Maximus32's fade transition.

This transition animation looks a lot better and it also fixes a memory issue for HiRes themes that use the info page.
Thanks to @rickgaiser for his code
Video for reference: https://www.youtube.com/watch?v=OKE6IZoopj4&t=5s

Fix transition sound effect.

Replaced default transition sound effect as it doesn't fit with the fade transition.
Free sound effect taken from: https://sellfy.com/p/gPCb/#

Latest test build: https://www.sendspace.com/file/tqetk2

@Tupakaveli Tupakaveli changed the title Update memory card icon Updates and fixes Jul 24, 2019
@TnA-Plastic
Copy link

I haven't seen the icon yet, but I certainly hope for the fade-transition finally being merged into the official repo!

@TnA-Plastic
Copy link

More commits sounds good.

Hopefully OPL will finally be released as 0.9.4 once all is done.

The fade-transition 'perfectens' the HD-Support and that alone (HD-Stuff) is worth a step from 0.9.3 to 0.9.4 and there where hundreds of other additions and changes, since the last release.

I am dl-ing the new icons and put it on the MC, for the time being (so I can see a new 3D-Icon).

@TnA-Plastic
Copy link

TnA-Plastic commented Jul 29, 2019

THX for the last fix.

SFX-Support per theme is a good reason for releasing OPL as 0.9.4 as well!

@TnA-Plastic
Copy link

TnA-Plastic commented Aug 4, 2019

One little bug which @Jolek found.

The code which checks for an OPL-Folder (and/or writes the MC-Icon) overwrites the OPL-Icon at every boot.

It is essentially just missing a switch-case or exception, when it should not write the new icons (i.e. if they or other icons are already present).

@Tupakaveli
Copy link
Contributor Author

Thanks, he already reported it.

@TnA-Plastic
Copy link

Alright! I just mentioned it, in case he didn't! ;)

@ElPatas1
Copy link
Contributor

Hello,

is needed to merge this commit? is well tested? or i still wait?

Best regards.

@TnA-Plastic
Copy link

TnA-Plastic commented Aug 12, 2019

It is working well... (except for the icon-replacement-bug)

The only thing I can think of, which could be merged as well is BGM-Support! :P

Anyway... It is up to @Tupakaveli and @KrahJohlito, if this is gonna be merged.

The Fade-transition technically could be merged without their consent, because it is Maximus32's/ @rickgaiser 's code and he said we can do with it, whatever we want...

@Tupakaveli
Copy link
Contributor Author

Tupakaveli commented Aug 12, 2019

Please wait until the memory card icon writing commit has been reverted. It solved one issue but caused another which we have been unable to find a fix for as of yet.

Everything else has been tested.

Best regards.

@Tupakaveli
Copy link
Contributor Author

Tupakaveli commented Aug 13, 2019

This can be merged now if you wish.

We're going to try and fix a couple of other things so I will open another PR later if required. If we can find a fix for the memory card icon writing bug it will be added.

Best regards.

@ElPatas1 ElPatas1 merged commit 3cb4707 into ps2homebrew:master Aug 13, 2019
@TnA-Plastic
Copy link

Cool! :)

@albixhafa
Copy link

When is 0.9.4 releasing ?

@BatRastard
Copy link
Contributor

Soon. As Tupakaveli said above, other fixes are in order. Especially the MC icon writing bug.

I'd rather see that bug fixed before releasing 0.9.4 if possible ...

@albixhafa
Copy link

Awesome !!

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.

5 participants