[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: KDE 3.0 - the last steps
From:       Martijn Klingens <mklingens () yahoo ! com>
Date:       2002-03-21 19:23:44
[Download RAW message or body]

On Thursday 21 March 2002 13:35, Thomas Zander wrote:
> On Thu, Mar 21, 2002 at 11:10:12AM +0100, Chris Howells wrote:
> > On Thu, 21 Mar 2002, Hetz Ben Hamo wrote:
> > > Oh, thats easy:
> > >
> > > http://witch.dyndns.org/stuff/auth.png
> >
> > Mine looked the same (CVS of yesterday morning GMT IIRC). I'm not running
> > any translations or anything.
>
> The correct fix would to make the column on the left (icon/site:) smaller
> and independent from the 'usename/password' column.
> This way the text on the right is allowed to take up the blank space
> between the icon and the text.

My KDE is not in a state that I can test or even compile stuff from CVS, but 
does the below patch solve the problem?

> If the text 'site:' would also be lined-out to top then 90% of the
> not-soo-nice dialog will be solved.

The 'site:' text is not in the file you mentioned. No idea where it comes 
from, but almost certainly it is enough to add Qt::AlignTop to the alignment 
flags of the label wherever it is constructed.

Martijn

Index: passdlg.cpp
===================================================================
RCS file: /home/kde/kdelibs/kio/kio/passdlg.cpp,v
retrieving revision 1.33
diff -u -p -r1.33 passdlg.cpp
--- passdlg.cpp 2002/03/04 04:17:37     1.33
+++ passdlg.cpp 2002/03/21 19:11:29
@@ -87,6 +87,8 @@ void PasswordDialog::init( const QString
     }
     d->prompt = new QLabel( main );
     d->prompt->setAlignment( Qt::AlignLeft|Qt::AlignVCenter|Qt::WordBreak );
+    d->prompt->setSizePolicy( QSizePolicy( QSizePolicy::MinimumExpanding,
+                              QSizePolicy::Preferred ) );
     d->layout->addWidget( d->prompt, 0, 2, Qt::AlignLeft );
     if ( prompt.isEmpty() )
         setPrompt( i18n( "You need to supply a username and a password" ) );
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic