broulik added inline comments.

INLINE COMMENTS

> textcreator.cpp:140
>  
> -            QColor bgColor = QColor ( 245, 245, 245 ); // light-grey 
> background
> -            m_pixmap.fill( bgColor );
> +            m_pixmap.fill( 
> KColorScheme(QPalette::Active).background().color() );
>  

Can't you use `QPalette` instead of `KColorScheme`? The latter parses config 
files on disk and is somewhat heavy

REPOSITORY
  R320 KIO Extras

REVISION DETAIL
  https://phabricator.kde.org/D20766

To: eshalygin
Cc: broulik, kde-frameworks-devel, kfm-devel, alexde, feverfew, meven, 
michaelh, spoorun, navarromorales, firef, ngraham, andrebarros, bruns, 
emmanuelp, mikesomov

Reply via email to