Review Request: Aspect Ratio Modes in Plasma::Dialog

[Giulio Camuffo]

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

(Updated 2009-11-18 11:10:35.674293)


Review request for Plasma.


Changes
-------

modified Applet::setAspectRatioMode to make it change the dialog's mode if it is the case;
other more little things


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 (updated)
-----

  /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