Review Request: Aspect Ratio Modes in Plasma::Dialog

Giulio Camuffo giuliocamuffo at
Wed Nov 18 12:13:11 CET 2009

This is an automatically generated e-mail. To reply, visit:

(Updated 2009-11-18 11:13:11.294927)

Review request for Plasma.


sorry, was still the old diff


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/applet.cpp 1050673 
  /trunk/KDE/kdelibs/plasma/dialog.h 1050673 
  /trunk/KDE/kdelibs/plasma/dialog.cpp 1050673 
  /trunk/KDE/kdelibs/plasma/popupapplet.cpp 1050673 



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.



More information about the Plasma-devel mailing list