Bug 201668 - In the generic Parameter Edit View, add support for QStringList types
Summary: In the generic Parameter Edit View, add support for QStringList types
Status: RESOLVED UNMAINTAINED
Alias: None
Product: telepathy
Classification: Frameworks and Libraries
Component: accounts-kcm (show other bugs)
Version: git-latest
Platform: Unlisted Binaries Linux
: NOR normal
Target Milestone: Future
Assignee: Telepathy Bugs
URL:
Keywords: junior-jobs
Depends on:
Blocks:
 
Reported: 2009-07-27 16:00 UTC by George Goldberg
Modified: 2014-04-11 16:45 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description George Goldberg 2009-07-27 16:00:11 UTC 
In the generic Parameter Edit View (src/parameter-edit-delegate.cpp etc) some custom types have special support, such as uint and int. QStringList type parameters are currently incorrectly displayed in a QLineEdit. Add support for displaying QStringList type parameters in an intuitive way so all the items in the list can be seen and edited.
Comment 1 George Goldberg 2009-08-04 17:03:36 UTC 
Patch from Lorenzo Masini currently pending on reviewboard: http://reviewboard.kde.org/r/1168/
Comment 2 Martin Klapetek 2011-11-02 10:42:35 UTC 
What happened to this one? Was it shipped or not?
Comment 3 Adrián Chaves (Gallaecio) 2012-09-20 18:21:29 UTC 
3 years and the patch has not been shipped. The author, Lorenzo, did not answer to George’s comments,
Comment 4 Adrián Chaves (Gallaecio) 2012-09-20 18:24:40 UTC 
Lorenzo, could you please provide an updated version of the patch without the trailing spaces and have a look at George’s concerns?
Comment 5 Lorenzo Masini 2012-09-20 18:37:27 UTC 
Adrián, are you asking me to change a patch I wrote 3 years ago? :D
Comment 6 Adrián Chaves (Gallaecio) 2012-09-20 19:32:22 UTC 
XD Either that or remove the review request. I just want to close this issue!
Comment 7 Lorenzo Masini 2012-09-20 20:01:39 UTC 
Ok, I've updated the patch, just because it is a good memory :D
I think it doesn't apply anymore :D