Re: Review Request: Fix Bug 152156 - KDE help center does not respect window placement policy

2012-04-18 Thread Parker Coates
> On April 18, 2012, 6 p.m., Parker Coates wrote: > > khelpcenter/mainwindow.cpp, line 170 > > > > > > I've seen a few applications use a certain fraction of the screen size > > as the default. See > > http://webs

Re: Review Request: Fix Bug 152156 - KDE help center does not respect window placement policy

2012-04-18 Thread Parker Coates
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104662/#review12635 --- Ship it! khelpcenter/mainwindow.cpp

Re: Review Request: Bug 263205. ksslcertificatemanager : fix the pattern matching used to display the user certificates.

2012-04-18 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104176/#review12639 --- This review has been submitted with commit 935a374a95190a34f35

Re: Review Request: Fix Bug 152156 - KDE help center does not respect window placement policy

2012-04-18 Thread Thomas Lübking
> On April 18, 2012, 6 p.m., Parker Coates wrote: > > khelpcenter/mainwindow.cpp, line 170 > > > > > > I've seen a few applications use a certain fraction of the screen size > > as the default. See > > http://webs

Review Request: Fix Bug 152156 - KDE help center does not respect window placement policy

2012-04-18 Thread Maarten De Meyer
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104662/ --- Review request for KDE Runtime and Cornelius Schumacher. Description -

Re: Review Request: Change Online help icon KHelpcenter

2012-04-18 Thread Maarten De Meyer
> On April 17, 2012, 1:45 p.m., Sebastian Kügler wrote: > > This change is wrong, as the menu entry has nothing to do with the semantic > > meaning of the icon, and the icon is not named according to the icon spec. > > > > So the correct icon is already set here, if its look doesn't match, then