----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107397/#review22288 -----------------------------------------------------------
Ship it! It tested it on my system with 'Use system colors' enabled, it's ok with Oxygen and Oxygen-cold themes. - Alessandro Russo On Nov. 21, 2012, 7:10 a.m., Cristian Oneț wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/107397/ > ----------------------------------------------------------- > > (Updated Nov. 21, 2012, 7:10 a.m.) > > > Review request for KMymoney. > > > Description > ------- > > Use the 'Higlight' background color for warnings since the 'Infobackground' > can be really close to the text color. If the same would happen to the > 'Higlight' background color we'll notice it a lot faster since that is also > used in the summary headers. > > > Diffs > ----- > > kmymoney/html/kmymoney.css e44ae0a > > Diff: http://git.reviewboard.kde.org/r/107397/diff/ > > > Testing > ------- > > With a light and a dark oxygen color scheme. > > > Screenshots > ----------- > > Before using the patch with default oxygen color theme (KDE 4.8) > http://git.reviewboard.kde.org/r/107397/s/844/ > After the patch with the default oxygen color theme (KDE 4.8) > http://git.reviewboard.kde.org/r/107397/s/845/ > > > Thanks, > > Cristian Oneț > >
_______________________________________________ KMyMoney-devel mailing list KMyMoney-devel@kde.org https://mail.kde.org/mailman/listinfo/kmymoney-devel