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

overlays: add ppu compile notification and touch existing compile notifications #13423

Merged
merged 6 commits into from
Mar 5, 2023

Conversation

Megamouse
Copy link
Contributor

  • Adds PPU compile notifications. We originally used the progress dialog for this, but due to the non-working coexistence with the cellMsgDialog this is generally disabled during gameplay.
    This missing dialog added to the confusion surrounding the Castlevania issue, which is seemingly stuck but actually compiles a ton of PPU modules.
  • Additionally, we now touch/refresh existing shader and ppu compile notifications.
    They used to go through a whole fade in and fade out cycle before a new one was shown.
    This means that even if stuff was constantly compiled, the notification kept popping in and out instead of persisting.

image

m_fade_out_animation.update(rsx::get_current_renderer()->vblank_count);
}
else if (m_fade_out_animation.active)
{
m_fade_out_animation.reset(rsx::get_current_renderer()->vblank_count);
Copy link
Contributor

Choose a reason for hiding this comment

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

Isn't the snapping effect jarring? Makes more sense to reverse the animation during reset though that is harder to do. Maybe add a TODO or something so we don't forget.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's more jarring to have an overlay constanty fading in and out xD

Copy link
Contributor

Choose a reason for hiding this comment

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

It's really not. A soft fade vs pop-in is a no contest.
Also think about translation animations. A card is sliding out to the edge of the screen then "snaps" back into place.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As far as I can tell they just fade out. Feel free to make some sort of card shuffle animation.

Copy link
Contributor

Choose a reason for hiding this comment

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

It should still reverse if we're beyond some point even with fade-out. The card may be 99% gone and almost entirely see-through, makes sense to just fade back in.
Obviously you refuse do it, I'll add it in after this is merged.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't refuse to do it, I just haven't had an issue with the way it looks.

Copy link
Contributor

Choose a reason for hiding this comment

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

You just have to restart the fade-in animation as well and borrow the remaining time from the fadeout to be its progress so far.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@Megamouse Megamouse force-pushed the voevevevev branch 6 times, most recently from 10c85f2 to 9fb91d3 Compare February 27, 2023 09:24
@Megamouse Megamouse force-pushed the voevevevev branch 5 times, most recently from 3637eac to 972c1a4 Compare March 3, 2023 11:02
@Megamouse Megamouse requested a review from kd-11 March 3, 2023 16:14
@Megamouse Megamouse merged commit 0d34b97 into RPCS3:master Mar 5, 2023
@Megamouse Megamouse deleted the voevevevev branch March 5, 2023 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants