[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: extragear/multimedia/amarok/src/context/applets/currenttrack
From:       Dan <hydrogen () notyetimplemented ! com>
Date:       2008-02-14 16:32:45
Message-ID: 47B46D2D.5040601 () notyetimplemented ! com
[Download RAW message or body]

Michaël Todorovic wrote:
> SVN commit 775003 by mtodorovic:
>
> Now the covers (not square) are centered in currentTrack applet
>
>  M  +5 -0      CurrentTrack.cpp  
>
>
> --- trunk/extragear/multimedia/amarok/src/context/applets/currenttrack/CurrentTrack.cpp #775002:775003
> @@ -334,6 +334,11 @@
>              cover = cover.scaledToHeight( size, Qt::SmoothTransformation );
>          else
>              cover = cover.scaledToWidth( size, Qt::SmoothTransformation );
> +
> +        //center the cover : if the cover is not squared, we get the missing pixels and center
> +        qreal moveBy = qAbs(cover.rect().width()-cover.rect().height())/2.0;
> +        m_albumCover->setPos(m_albumCover->x()+ moveBy, m_albumCover->y());
> +
>          m_albumCover->setPixmap( cover );
>          return true;
>      }
>   

Hi,

Please read amarok/HACKING and ensure your commits follow the style put 
forth within.

Thank you

[prev in list] [next in list] [prev in thread] [next in thread]