Review Request: Aspect Ratio Modes in Plasma::Dialog

[Aaron Seigo]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/2202/#review3165
-----------------------------------------------------------

Ship it!


- Aaron


On 2009-11-18 11:13:11, Giulio Camuffo wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/2202/
> -----------------------------------------------------------
> 
> (Updated 2009-11-18 11:13:11)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> This patch adds the aspect ratio modes to Plasma::Dialog too. It adds the methods setAspectRatioMode(Plasma::AspectRatioMode) and aspectRatioMode(). 
> Then I modified PopupApplet to let its dialog assume the aspectRatioMode of the applet when it is expanded.
> 
> This solves an issue with the Comic applet that uses KeepAspectRatio not to deform the comic. Without this, when shown in the dialog, there is no control about that.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdelibs/plasma/applet.cpp 1050673 
>   /trunk/KDE/kdelibs/plasma/dialog.h 1050673 
>   /trunk/KDE/kdelibs/plasma/dialog.cpp 1050673 
>   /trunk/KDE/kdelibs/plasma/popupapplet.cpp 1050673 
> 
> Diff: http://reviewboard.kde.org/r/2202/diff
> 
> 
> Testing
> -------
> 
> Tried with all the aspect ratio modes. I'm actually a bit unsure about the ConstrainedSquare. It hasn't so much sense in a dialog and currently it behaves like the Square one. And the PopupApplet can't know when the mode gets changed so the mode of the dialog is changed in popupConstraintsEvent and in internalTogglePopup. But if the applet changes its ratio mode while the dialog is being displayed it won't be get.
> 
> 
> Thanks,
> 
> Giulio
> 
>