QGraphicsOpacityEffect on a QLabel

maximus

edited

maximus

I think I forgot about QObject parent property, since Fader is a subclass of QObject, I will pass a reference to the parent and the pointer should be taken care when the parentWidget is destroyed? will try.

@Fader::Fader(QWidget* parentAndTarget) : QObject( parentAndTarget ) {

 //give target and parent to have auto memory management
this->widgetPtr = parentAndTarget;

}@

maximus

Okay Version 3 is better:

Fader.h
@class Fader : public QObject
{
Q_OBJECT
public:
Fader(QWidget* parentTarget);
~Fader();

void fadeIn(int fadeInTime);
void fadeOut(int fadeInTime);

void fadeOutAfter(int fadeOutTime, int fadeAfter);
void fadeInAndOut(int fadeInTime, int fadeOutTime, int pause);

private slots:
void animateFadeIn();
void animateFadeOut();

private :
QWidget *widget;

QTimer *timerSetVisible;
QTimer *timerFadeOut;

int fadeOutTime;
int fadeInTime;

};@

Fader.cpp
@Fader::Fader(QWidget* parentAndTarget) : QObject( parentAndTarget ) {

widget = parentAndTarget;
widget->setVisible(true);

}

///////////////////////////////////////////////////////////////////////////////
void Fader::fadeIn(int fadeInTime) {
this->fadeInTime = fadeInTime;
animateFadeIn();
}
void Fader::fadeOut(int fadeOutTime) {
this->fadeOutTime = fadeOutTime;
animateFadeOut();
}

//----------------------------------------------------------------------------
void Fader::fadeOutAfter(int fadeOutTime, int fadeAfter) {

timerFadeOut = new QTimer(this);
timerSetVisible = new QTimer(this);
timerFadeOut->setSingleShot(true);
timerSetVisible->setSingleShot(true);

this->fadeOutTime = fadeOutTime;

connect(timerFadeOut, SIGNAL(timeout()), this, SLOT(animateFadeOut()) );
connect(timerSetVisible, SIGNAL(timeout()), widget, SLOT(hide()) );

timerFadeOut->start(fadeAfter);
timerSetVisible->start(fadeAfter + fadeOutTime);

}

//----------------------------------------------------------------------------
void Fader::fadeInAndOut(int fadeInTime, int fadeOutTime, int pause) {

this->fadeInTime = fadeInTime;
animateFadeIn();

timerFadeOut = new QTimer(this);
timerSetVisible = new QTimer(this);
timerFadeOut->setSingleShot(true);
timerSetVisible->setSingleShot(true);

this->fadeOutTime = fadeOutTime;

connect(timerFadeOut, SIGNAL(timeout()), this, SLOT(animateFadeOut()) );
connect(timerSetVisible, SIGNAL(timeout()), widget, SLOT(hide()) );

timerFadeOut->start(fadeInTime + pause);
timerSetVisible->start(fadeInTime + fadeOutTime + pause);

}

//---------------------------------------------------------------------------------
void Fader::animateFadeIn() {

qDebug() << "animateFadeIn";


QGraphicsOpacityEffect *effect = new QGraphicsOpacityEffect(widget);
QPropertyAnimation *anim = new QPropertyAnimation(widget);
widget->setGraphicsEffect(effect);

anim->setPropertyName("opacity");
anim->setTargetObject(effect);
anim->setDuration(fadeInTime);
anim->setStartValue(0.0);
anim->setEndValue(1.0);
anim->setEasingCurve(QEasingCurve::OutQuad);
anim->start(QAbstractAnimation::DeleteWhenStopped);

}
//---------------------------------------------------------------------------------
void Fader::animateFadeOut() {

qDebug() << "animateFadeOut";


QGraphicsOpacityEffect *effect = new QGraphicsOpacityEffect(widget);
QPropertyAnimation *anim = new QPropertyAnimation(widget);
widget->setGraphicsEffect(effect);

anim->setPropertyName("opacity");
anim->setTargetObject(effect);
anim->setDuration(fadeOutTime);
anim->setStartValue(1.0);
anim->setEndValue(0.0);
anim->setEasingCurve(QEasingCurve::OutQuad);
anim->start(QAbstractAnimation::DeleteWhenStopped);

}@

maximus

Just want to close the thread with what I chose.

After a lot of testing using the Fader class, it was working but not 100% like I wanted. I will go back to the old approach with many slots and QTimer inside the QDialog

Pro to old method:

I can see if the widget is already being animated before setting a new animation on it (The Fader just blindly set a new animation, if there is still an active animation on the Widget it can cause weird animations and bad timing)
Easier memory management

Cons:

Lot of copied code, will try to put the common animation in a function to reuse code.

Thanks for your help

maximus

Solution found inside FancyTab! (credit to QtCreator source code)

Keep a reference to QPropertyAnimation, stop the current animation every time a new one is received before activating new one, brillant!

@#ifndef FANCYTAB_H
#define FANCYTAB_H

#include <QIcon>
#include <QWidget>

#include <QPropertyAnimation>

class FancyTab : public QObject
{
Q_OBJECT

Q_PROPERTY(float fader READ fader WRITE setFader)

public:
FancyTab(QWidget *tabbar) : enabled(false), mTabBar(tabbar), mFader(0)
{
mAnimator.setPropertyName("fader");
mAnimator.setTargetObject(this);
}

float fader() { return mFader; }
void setFader(float value);

void fadeIn();
void fadeOut();

QIcon icon;
QString text;
QString toolTip;
bool enabled;

private:
QPropertyAnimation mAnimator;
QWidget *mTabBar;
float mFader;
};

#endif // FANCYTAB_H@

maximus

Okay here is a complete working solution (add it to Qt? :P)
Only wish I subclassed QWidget instead of QLabel, as I need this for other QWidget also

FaderLabel
@#include <QLabel>
#include <QTimer>
#include <QPropertyAnimation>
#include <QGraphicsOpacityEffect>

class FaderLabel : public QLabel
{
Q_OBJECT

public:
explicit FaderLabel(QWidget *parent = 0);

void fadeIn(int durationMs);
void fadeOut(int durationMs);

void fadeOutAfterPause(int fadeDuration, int pause);
void fadeInAndFadeOutAfterPause(int fadeInDuration, int fadeOutDuration, int pause);

private slots: //Used to connect with timer

void fadeInAfterTimer();
void fadeOutAfterTimer();

private :
QTimer *timerAnimatorFadeOut;
QTimer *timerAnimatorFadeIn;

QGraphicsOpacityEffect *effect;
QPropertyAnimation *anim;

int tmpDurationFadeOut;
int tmpDurationFadeIn;

};

#endif // FADERLABEL_H@

FaderLabel.cpp
@#include "faderlabel.h"

#include <QDebug>

FaderLabel::FaderLabel(QWidget *parent) :
QLabel(parent)
{

timerAnimatorFadeOut = new QTimer(this);
timerAnimatorFadeOut->setSingleShot(true);
timerAnimatorFadeIn = new QTimer(this);
timerAnimatorFadeIn->setSingleShot(true);

connect(timerAnimatorFadeOut, SIGNAL(timeout()), this, SLOT(fadeOutAfterTimer()) );
connect(timerAnimatorFadeIn, SIGNAL(timeout()), this, SLOT(fadeInAfterTimer()) );


effect = new QGraphicsOpacityEffect(this);
this->setGraphicsEffect(effect);
anim = new QPropertyAnimation(effect, "opacity", this);


tmpDurationFadeOut = 0;
tmpDurationFadeIn = 0;

}

//---------------------------------------------------
void FaderLabel::fadeInAfterTimer() {

fadeIn(this->tmpDurationFadeIn);

}
void FaderLabel::fadeOutAfterTimer() {

fadeOut(this->tmpDurationFadeOut);

}

//---------------------------------------------------------------------------------
void FaderLabel::fadeIn(int durationMs) {

qDebug() << "animateFadeIn FaderLabel" << durationMs;
anim->stop();
anim->setDuration(durationMs);
anim->setStartValue(0);
anim->setEndValue(1);
anim->setEasingCurve(QEasingCurve::OutQuad);
anim->start();

}

//---------------------------------------------------------------------------------
void FaderLabel::fadeOut(int durationMs) {

qDebug() << "animateFadeout FaderLabel" << durationMs;
anim->stop();
anim->setDuration(durationMs);
anim->setStartValue(1);
anim->setEndValue(0);
anim->setEasingCurve(QEasingCurve::OutQuad);
anim->start();

}

//--------------------------------------------------------------------------------
void FaderLabel::fadeOutAfterPause(int fadeDuration, int pause) {

this->tmpDurationFadeOut = fadeDuration;
timerAnimatorFadeOut->start(pause);

}

//-------------------------------------------------------------------------------------------------
void FaderLabel::fadeInAndFadeOutAfterPause(int fadeInDuration, int fadeOutDuration, int pause) {

fadeIn(fadeInDuration);

this->tmpDurationFadeOut = fadeOutDuration;
timerAnimatorFadeOut->start(pause);

}
@

maximus

I ended up creating a "FaderWidget", "FaderFrame" and "FaderLabel"

since QFrame and QLabel are childrens of QWidet, would it be possible to just code "FaderWidget" and make them inherit "FaderWidget"? Or do I have to edit Qt code?

Thanks!

Chris Kawa

For whatever reason i stopped receiving notifications for this thread... :/

I think creating this sort of class for each possible widget is cumbersome and it's back to copy/pasting which is no good. You want something that just takes (any) widget and animates it without intruding into that class code. And you want to write it only once.

A little tweak to my original proposal using QPointer to take care of that releasing problem:
@
class Delay: public QObject {
public:
Delay(int duration) {
QTimer::singleShot(duration, this, SLOT(deleteLater()));
}
};

class Fader : public QObject {
public:
Fader(QWidget* target, double start, double end, int fade) {
auto effect = new QGraphicsOpacityEffect(this);
target->setGraphicsEffect(effect);

auto anim = new QPropertyAnimation(effect, "opacity", this);
connect(anim, &QPropertyAnimation::finished, this, &Fader::deleteLater);
anim->setDuration(fade);
anim->setStartValue(start);
anim->setEndValue(end);
anim->setEasingCurve(QEasingCurve::OutQuad);
anim->start();

}

static void fadeInOut(QWidget* target, int fade, int pause) {
QPointer<QWidget> p(target);
if(p)connect(new Fader(target, 0.0, 1.0, fade),&Fader::destroyed,={
if(p)connect(new Delay(pause), &Delay::destroyed, ={
if(p)connect(new Fader(target,1.0,0.0,fade),&Fader::destroyed,={
if(p)target->setVisible(false);
});
});
});
}
};
@

maximus

Hey Chris thanks for your message!

I know the oriented style design is not the best in this case, but I need to have control over the fade effect (stop effect, resume,.. ) because it is tied with user interactions. Also with my lack of C++ programming experience, I'm no good to adapt your solution and it''s more "crash-prone" in my case..

Thanks for your help!
Max

? Offline

Lets discuss prof1ts! Are you afraid of big numbers? I protest, you don’t know what that is. You are no different than N4ZIs book burners if you DEF4CE this, you deserve to have it splattered around.

annualreport2013.digia .com/ financial_statement/consolidated_income_statement

@
Digia
Total assets in 2013 83,277,864.58

Operating prof1t

2006 2007 2008 2009 2010 2011 2012 2013 2014 Total
8,3 11 13,4 -7,8 17,1 -22,1 6,9 -2,8 XXX 24,0
@

You might have a hard time to find info older then 2008, but don’t take my word on it, even if you discount years before 2008 you are still on 4,7 Million green mark TOTAL.
Now listen, do you see that forbidden triple X? You are there now, that’s you critical branch-point. I could call you a bunch of cry babies, but I won’t. Here’s my message to you.

MILLIONS IN PROF1T! Most people yearly “salaries” are not even close to “1%” of that smaller million of that smaller mostly unknown company. All while paying many millions on an undervalued Qt from a fading company, with salesman altering/faking peoples CVs just to sell/dump people to customers signing whatever prof1table contracts and ultimately firing LOTS of people with fake excuses.
There’s a very pertinent question that demands an answer.
Even though, many world economies, Nokia, Digia and all other companies are still under the wise guidance of the best supreme leaders available, why does at least the first three are still in crisis or under a steep decline in operating prof1ts, among other things.
This question is not, I think, irreverent, although it may trouble whatever supreme leaders may be.

“Those worlds in space are as countless as all the grains of sand on all the beaches of the Earth. Each of those worlds is as real as ours. In every one of them, there's a succession of incidents, events, occurrences which influence its future. Countless worlds, numberless moments, an immensity of space and time. And our small planet, at this moment, here we face a critical branch-point in the history. What we do with our world, right now, will propagate down through the centuries and powerfully affect the destiny of our descendants. It is well within our power to destroy our civilization, and perhaps our species as well. If we capitulate to superstition, or greed, or stupidity we can plunge our world into a darkness deeper than time between the collapse of classical civilization and the Italian Renaissance. But, we are also capable of using our compassion and our intelligence, our technology and our wealth, to make an abundant and meaningful life for every inhabitant of this planet. To enhance enormously our understanding of the Universe, and to carry us to the stars.”
https://www.youtube dot com/watch?v=hLkC7ralR30

The Earth is the only world known, so far, to harbor life. There is nowhere else, at least in the near future, to which our species could migrate. Visit, yes. Settle, not yet. Like it or not, for the moment, the Earth is where we make our stand. It has been said that astronomy is a humbling and character-building experience. There is perhaps no better demonstration of the folly of human conceits than this distant image of our tiny world. To me, it underscores our responsibility to deal more kindly with one another and to preserve and cherish the pale blue dot, the only home we've ever known.”””

!https://upload.wikimedia.org/wikipedia/commons/7/73/Pale_Blue_Dot.png (https://upload.wikimedia.org/wikipedia/commons/7/73/Pale_Blue_Dot.png)!
123213213