Review Request: Only show the screen config changed message once.

2012-08-26 Thread Frederik Gladhorn
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106212/ --- Review request for KDE Base Apps and Alex Fiestas. Description ---

Re: Review Request: konqueror: strip new line character from URL on paste automatically

2012-08-26 Thread Martin Koller
On Aug. 25, 2012, 10:15 p.m., David Faure wrote: Looks good. I'm not sure where the need for ParagraphSeparator and LineSeparator comes from though, do websites actually use that? I found these two in QChar and thought it might happen that the clipboard contains not \n but one of these

Re: Review Request: elide completion entries in the middle in konq-address completion list

2012-08-26 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106183/#review18025 --- This review has been submitted with commit

Re: Review Request: konqueror: strip new line character from URL on paste automatically

2012-08-26 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106184/#review18026 --- This review has been submitted with commit

Re: Review Request: elide completion entries in the middle in konq-address completion list

2012-08-26 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106183/#review18027 --- This review has been submitted with commit

Re: Review Request: Only show the screen config changed message once.

2012-08-26 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106212/#review18029 --- That warning is there to remind the maintainer to re-factor

Re: Review Request: Move checks for locking primitives in KSharedDataCache to CMake

2012-08-26 Thread Michael Pyne
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106174/#review18055 --- I've looked over everything and didn't see anything I didn't

Re: Review Request: Mutex cleanup in KSharedDataCache

2012-08-26 Thread Michael Pyne
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106221/#review18056 --- A few comments, but most of the changes are OK (though they

Re: Review Request: Focus goes to location bar when opening link in new tab in foreground

2012-08-26 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105984/#review18063 --- This review has been submitted with commit

Re: Review Request: Config cleanup on removeActivity

2012-08-26 Thread Lamarque Vieira Souza
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106097/#review18065 --- Ship it! Ship It! - Lamarque Vieira Souza On Aug. 20,