[KPhotoAlbum] Visual Updates for KPA

Jesper K. Pedersen blackie at kde.org
Sun Jun 19 10:58:11 BST 2011


> > Why did you remove this code though:
> > -void MainWindow::StatusBar::setupFixedFonts()
> > -{
> > -    // Avoid flicker in the statusbar when moving over dates from the
> > datebar -    QFont f( font() );
> > -    f.setStyleHint( QFont::TypeWriter );
> > -    f.setFamily( QString::fromLatin1( "courier" ) );
> > -    f.setBold( true );
> > -    setFont( f );
> > -}
> 
> I think that a UI should contain as little different fonts as
> possible, the typewriter font in the status bar always struck me as
> unnecessary.
> 
> Usually either a change in position or a font-change is used to
> differentiate information from each other, since in this case the
> information has its own positing (the status bar) its not necessary to
> change the font too.
> 
> Am I being understandable? Change judge...
Indeed you are being understandable.

Checking your patch without my fixed font seems to indicate that the "bug" is 
gone now, so that is all fine. I'll commit the patch as soon as the datebar 
issues is resolved. That patch has changes that blocks this one.

Cheers



More information about the Kphotoalbum mailing list