Bug 231241 - HTTP Authorization Dialog cuts hostname in site display
Summary: HTTP Authorization Dialog cuts hostname in site display
Status: RESOLVED FIXED
Alias: None
Product: kdelibs
Classification: Frameworks and Libraries
Component: kdeui (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: kdelibs bugs
URL:
Keywords:
: 132858 237135 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-03-18 18:04 UTC by Martin Koller
Modified: 2010-06-16 01:00 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
screenshot of dialog (25.57 KB, image/png)
2010-03-18 18:04 UTC, Martin Koller
Details
proposed fix (865 bytes, patch)
2010-05-19 21:54 UTC, Christoph Feck
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Koller 2010-03-18 18:04:00 UTC
Version:            (using KDE 4.4.1)
OS:                Linux
Installed from:    openSUSE RPMs

When I connect to a HTTP Server with konqueror which need authentication, the Authorization Dialog pops up.
This dialog shows "Site:" ... etc.
The Site: value contains here a 2 line output:

AT3.9@eiwrk039
at localhost

The second line only the half height is shown, the bottom half is covered with the LineEdit input field "Username" below.

See attached scrrenshot.
Comment 1 Martin Koller 2010-03-18 18:04:58 UTC
Created attachment 41742 [details]
screenshot of dialog
Comment 2 Christoph Feck 2010-05-11 00:15:23 UTC
*** Bug 237135 has been marked as a duplicate of this bug. ***
Comment 3 jens 2010-05-19 20:25:14 UTC
Same result here (KDE trunk from 18-05-2010), when accessing a site with NTLM authentication the dialog shows:
NTLM at
<server>
whereas <server> is completely hidden in my case.

The problem lives somewhere in http://websvn.kde.org/trunk/KDE/kdelibs/kdeui/dialogs/kpassworddialog.cpp?view=markup in function KPasswordDialog::addCommentLine. Unfortunately I don't know QT well enough to follow the logic :(
Comment 4 Christoph Feck 2010-05-19 21:54:47 UTC
Created attachment 43741 [details]
proposed fix

There is still some regression with too wide window, but should fix the issue. Please give feedback.
Comment 5 jens 2010-05-19 22:49:40 UTC
The attached patch works fine for me, the comment is fully visible now, really cool! 
(as you alreaady noted, the dialog is now indeed to wide.)
Comment 6 Christoph Feck 2010-05-20 18:05:56 UTC
Thanks for testing Jens. I will try to improve the patch to get it into 4.5.
Comment 7 Christoph Feck 2010-06-07 23:33:10 UTC
*** Bug 132858 has been marked as a duplicate of this bug. ***
Comment 8 Christoph Feck 2010-06-16 01:00:13 UTC
SVN commit 1138399 by cfeck:

Fix size limits when word wrapped comment lines are added

BUG: 231241


 M  +8 -3      kpassworddialog.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1138399