diff -r bf7eb7911fc5 -r 997a02608b3a emailuis/nmailui/src/nmuieffects.cpp --- a/emailuis/nmailui/src/nmuieffects.cpp Wed Jun 23 18:00:21 2010 +0300 +++ b/emailuis/nmailui/src/nmuieffects.cpp Tue Jul 06 14:04:34 2010 +0300 @@ -17,7 +17,7 @@ #include "nmuiheaders.h" -static const char *SendAnimation = ":/effects/mail_send.fxml"; +static const char *NmSendAnimation = ":/effects/mail_send.fxml"; /*! \class NmEffects @@ -55,32 +55,35 @@ NM_FUNCTION; switch (effect) { - case NmEditorSendMessageAnimation: - // delete any existing stuff - resetSendAnimation(); - - // This effect is for editor send message. Get the screen capture of - // editor view for animation which will be lauched soon. - mDoSendAnimation = true; - - // take screen shot - mSendAnimationScreenShot = screenShot(); + case NmEditorSendMessageAnimation: { + // delete any existing stuff + resetSendAnimation(); + + // This effect is for editor send message. Get the screen capture of + // editor view for animation which will be lauched soon. + mDoSendAnimation = true; + + // take screen shot + mSendAnimationScreenShot = screenShot(); + + if (mSendAnimationScreenShot){ + // Create graphics item based pixmap image but don't show it yet. + mSendAnimationScreenShot->hide(); + mSendAnimationScreenShot->setPos(0,0); + mSendAnimationScreenShot->setZValue(0); - // Create graphics item based pixmap image but don't show it yet. - mSendAnimationScreenShot->hide(); - mSendAnimationScreenShot->setPos(0,0); - mSendAnimationScreenShot->setZValue(0); - - // Adds or moves the item and all its childen to this scene. - // This scene takes ownership of the item. - mMainWindow.scene()->addItem(mSendAnimationScreenShot); - - // Set editor screen capture visible before old view is popped. - // New view is drawn under this image. - mSendAnimationScreenShot->show(); - - HbEffect::add(mSendAnimationScreenShot, SendAnimation, "mail_send"); - break; + // Adds or moves the item and all its childen to this scene. + // This scene takes ownership of the item. + mMainWindow.scene()->addItem(mSendAnimationScreenShot); + + // Set editor screen capture visible before old view is popped. + // New view is drawn under this image. + mSendAnimationScreenShot->show(); + + HbEffect::add(mSendAnimationScreenShot, NmSendAnimation, "mail_send"); + } + break; + } } } @@ -92,14 +95,15 @@ NM_FUNCTION; switch (effect) { - case NmEditorSendMessageAnimation: - // Send message animation for editor view. - if (mDoSendAnimation && mSendAnimationScreenShot) { - mDoSendAnimation = false; - // Start animation and connect completion signal to sendAnimationComplete slot. - HbEffect::start(mSendAnimationScreenShot, "mail_send", this, "sendAnimationComplete"); + case NmEditorSendMessageAnimation: { + // Send message animation for editor view. + if (mDoSendAnimation && mSendAnimationScreenShot) { + mDoSendAnimation = false; + // Start animation and connect completion signal to sendAnimationComplete slot. + HbEffect::start(mSendAnimationScreenShot, "mail_send", this, "sendAnimationComplete"); + } + break; } - break; } } @@ -113,8 +117,7 @@ // Grab whole view into pixmap image (also chrome is included) QPixmap screenCapture = QPixmap::grabWindow(mMainWindow.internalWinId()); - - QGraphicsPixmapItem *ret = NULL; + QGraphicsPixmapItem *ret(NULL); // for landscape, the screenshot must be rotated if(mMainWindow.orientation() == Qt::Horizontal) { @@ -138,7 +141,7 @@ if (mSendAnimationScreenShot) { // Clean send animation - HbEffect::remove(mSendAnimationScreenShot, SendAnimation, "mail_send"); + HbEffect::remove(mSendAnimationScreenShot, NmSendAnimation, "mail_send"); // Ownership of QGraphicsPixmapItem is tranferred to GraphicsScene when it has been added // to it GraphicsScene. // GraphicsPixmapItem needs to be removed from the GraphicsScene before deleting