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

Michael Leupold lemma at confuego.org
Thu Mar 26 18:59:59 GMT 2009


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

Review request for KDE PIM.


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
-----

  /trunk/KDE/kdepim/knode/knode/knaccountmanager.cpp 944639 
  /trunk/KDE/kdepim/knode/knode/knarticlefilter.cpp 944639 
  /trunk/KDE/kdepim/knode/knode/knconfig.cpp 944639 
  /trunk/KDE/kdepim/knode/knode/knconfigwidgets.h 944639 
  /trunk/KDE/kdepim/knode/knode/knconfigwidgets.cpp 944639 
  /trunk/KDE/kdepim/knode/knode/knconvert.cpp 944639 
  /trunk/KDE/kdepim/knode/knode/knfiltermanager.cpp 944639 
  /trunk/KDE/kdepim/knode/knode/knfolder.cpp 944639 
  /trunk/KDE/kdepim/knode/knode/knfoldermanager.cpp 944639 
  /trunk/KDE/kdepim/knode/knode/knnntpaccount.cpp 944639 
  /trunk/KDE/kdepim/knode/knode/postnewscomposerwidget_base.ui 944639 
  /trunk/KDE/kdepim/knode/knode/settings.cpp 944639 

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