On October 5th, 2011, 11:30 a.m., David Faure wrote:

Why the setMaxLength?? What if one wants to type in a long URL?

Also, I can't reproduce the bug here (kde-4.7), but maybe only because the big icon button makes the dialog quite large?

On October 5th, 2011, 6:23 p.m., Greg T wrote:

indeed, the setmaxLength was stupid

it's quite small on 4.6.5 http://wstaw.org/m/2011/10/05/plasma-desktopF27745.jpg

On October 6th, 2011, 7:52 a.m., David Faure wrote:

Ah I see, it's the german translation which makes the label longer ("Choose an icon" -> "Wählen Sie ein Symbol aus"), leaving less room for the lineedits.

Patch 3 looks good, although I can't help but think there's a bug if a set*Width() call takes a height() as value :-) But I'll trust Christoph on this one.

On October 6th, 2011, 8:15 a.m., Peter Penz wrote:

> Patch 3 looks good, although I can't help but think there's a bug if a
> set*Width() call takes a height() as value :-) But I'll trust Christoph on this one.

Using QFontMetrics::averageCharWidth() might be an alternative in comparison to use the height as reference for calculating the width (but probably this has some drawbacks I'm not aware of).

First, averageCharWidth() is slow. It goes through every character to compute the average. Second, that average might be skewed when there are broken characters in the font.

I'll add a comment why the lineedits width depends on the fonts height.

- Christoph

On October 5th, 2011, 6:22 p.m., Greg T wrote:

Hi, this patch sets a minimum size for the widget. 

it's working