Review Request: Editable KComboBox instead of KLineEdit with autocompletion box for the custom title setting in the folderview's config dialog

[Farhad Hedayati Fard]

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

Review request for Plasma.


Summary
-------

IMO and As J Janz said in my previous review, using an editable KComboBox instead of a KLineEdit with completion box is a cleaner and less confusing approach for letting the user choose one of the predefined values for the title bar in the folderview's config dialog. I've edited the UI file and replaced the KLineEdit with a KComboBox, and used a KLineEdit for the LineEdit of this combobox as QLineEdit doesn't provide the clickMessage afaik.


Diffs
-----

  plasma/applets/folderview/folderview.h ef4e205 
  plasma/applets/folderview/folderview.cpp 67f9ec0 
  plasma/applets/folderview/folderviewDisplayConfig.ui 6568640 

Diff: http://git.reviewboard.kde.org/r/100834/diff


Testing
-------

Compiled and tested this patch, works fine here.


Thanks,

Farhad

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20110310/722ecbbc/attachment.htm