[Kde-pim] Review Request: KNode, make saving of the external editor setting possible again

Michael Leupold lemma at confuego.org
Fri Mar 27 22:28:43 GMT 2009


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

(Updated 2009-03-27 15:28:42.692129)


Review request for KDE PIM.


Changes
-------

Reworked patch. Everything that's needed is already available in kdelibs r945651 :)


Summary
-------

KNode uses a KUrlRequester as the widget to enter the external editor into. Unfortunately KUrlRequester doesn't quite work with KConfigXT (at least not if you want the config type String). This patch subclasses KUrlRequester and adds a "text" property for use with KConfigXT. Both methods are simply delegated to the superclass.

I left the .ui strings as is so this could be backported to 4.2. Adapting the strings so they aren't broken is just a matter of opening in designer and saving again.


This addresses bug 170672.
    https://bugs.kde.org/show_bug.cgi?id=170672


Diffs (updated)
-----

  /trunk/KDE/kdepim/knode/postnewscomposerwidget_base.ui 945063 

Diff: http://reviewboard.kde.org/r/429/diff


Testing
-------


Thanks,

Michael

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/



More information about the kde-pim mailing list