Xargon/ES-DE
1
0
Fork 0
mirror of https://github.com/RetroDECK/ES-DE.git synced 2024-11-22 06:05:38 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Frame advances no longer take place for GIF and Lottie animations unless update() has been called.

Browse source
This commit is contained in:
Leon Styhre 2022-09-26 19:00:59 +02:00
parent 1c10fb7466
commit 577ed366b4
2 changed files with 16 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
es-core/src/components/GIFAnimComponent.cpp
View file

@ -37,7 +37,7 @@ GIFAnimComponent::GIFAnimComponent()
, mTimeAccumulator {0}
, mLastRenderedFrame {-1}
, mSkippedFrames {0}
, mHoldFrame {false}
, mHoldFrame {true}
, mPause {false}
, mExternalPause {false}
, mAlternate {false}
@ -344,7 +344,7 @@ void GIFAnimComponent::applyTheme(const std::shared_ptr<ThemeData>& theme,
void GIFAnimComponent::update(int deltaTime)
{
if (!isVisible() || mThemeOpacity == 0.0f || mAnimation == nullptr)
if (mAnimation == nullptr || !isVisible() || mOpacity == 0.0f || mThemeOpacity == 0.0f)
return;
if (mWindow->getAllowFileAnimation()) {
@ -356,6 +356,9 @@ void GIFAnimComponent::update(int deltaTime)
return;
}
// Make sure no frames are advanced unless update() has been called.
mHoldFrame = false;
// If the time accumulator value is really high something must have happened such as the
// application having been suspended. Reset it to zero in this case as it would otherwise
// never recover.
@ -392,7 +395,7 @@ void GIFAnimComponent::update(int deltaTime)
void GIFAnimComponent::render(const glm::mat4& parentTrans)
{
if (!isVisible() || mThemeOpacity == 0.0f || mAnimation == nullptr)
if (mAnimation == nullptr || !isVisible() || mOpacity == 0.0f || mThemeOpacity == 0.0f)
return;
glm::mat4 trans {parentTrans * getTransform()};
@ -498,4 +501,6 @@ void GIFAnimComponent::render(const glm::mat4& parentTrans)
// Render it.
mRenderer->drawTriangleStrips(&vertices[0], 4);
}
mHoldFrame = true;
}

11
es-core/src/components/LottieAnimComponent.cpp
View file

@ -32,7 +32,7 @@ LottieAnimComponent::LottieAnimComponent()
, mTimeAccumulator {0}
, mLastRenderedFrame {-1}
, mSkippedFrames {0}
, mHoldFrame {false}
, mHoldFrame {true}
, mPause {false}
, mExternalPause {false}
, mAlternate {false}
@ -298,7 +298,7 @@ void LottieAnimComponent::applyTheme(const std::shared_ptr<ThemeData>& theme,
void LottieAnimComponent::update(int deltaTime)
{
if (mAnimation == nullptr)
if (mAnimation == nullptr || !isVisible() || mOpacity == 0.0f || mThemeOpacity == 0.0f)
return;
if (mWindow->getAllowFileAnimation()) {
@ -310,6 +310,9 @@ void LottieAnimComponent::update(int deltaTime)
return;
}
// Make sure no frames are advanced unless update() has been called.
mHoldFrame = false;
// If the time accumulator value is really high something must have happened such as the
// application having been suspended. Reset it to zero in this case as it would otherwise
// never recover.
@ -346,7 +349,7 @@ void LottieAnimComponent::update(int deltaTime)
void LottieAnimComponent::render(const glm::mat4& parentTrans)
{
if (!isVisible() || mThemeOpacity == 0.0f || mAnimation == nullptr)
if (mAnimation == nullptr || !isVisible() || mOpacity == 0.0f || mThemeOpacity == 0.0f)
return;
glm::mat4 trans {parentTrans * getTransform()};
@ -489,4 +492,6 @@ void LottieAnimComponent::render(const glm::mat4& parentTrans)
// Render it.
mRenderer->drawTriangleStrips(&vertices[0], 4);
}
mHoldFrame = true;
}