Re: Review Request: Make the maximum # of connection per host in KIO::Scheduler user-configurable

2012-03-03 Thread Dawit Alemayehu
. This addresses bug 64647. http://bugs.kde.org/show_bug.cgi?id=64647 Diffs (updated) - kio/kio/scheduler.cpp 6b4ed52 Diff: http://git.reviewboard.kde.org/r/104120/diff/ Testing --- Thanks, Dawit Alemayehu

Re: Review Request: Always request small icons for Konqueror's status bar.

2012-02-29 Thread Dawit Alemayehu
://git.reviewboard.kde.org/r/104071/diff/ Testing --- Thanks, Dawit Alemayehu

Review Request: Make the maximum # of connection per host in KIO::Scheduler user-configurable

2012-02-29 Thread Dawit Alemayehu
://git.reviewboard.kde.org/r/104120/diff/ Testing --- Thanks, Dawit Alemayehu

Review Request: Always request small icons for Konqueror's status bar.

2012-02-24 Thread Dawit Alemayehu
convenience function. This addresses bug 294321. http://bugs.kde.org/show_bug.cgi?id=294321 Diffs - konqueror/src/konqframestatusbar.cpp 0142bd0 Diff: http://git.reviewboard.kde.org/r/104071/diff/ Testing --- Thanks, Dawit Alemayehu

Re: Review Request: New KPart extension for manupilating a browser engine's settings

2012-02-21 Thread Dawit Alemayehu
://git.reviewboard.kde.org/r/103973/diff/ Testing --- Thanks, Dawit Alemayehu

Re: Review Request: New KPart extension for manupilating a browser engine's settings

2012-02-20 Thread Dawit Alemayehu
--- On Feb. 18, 2012, 9:09 p.m., Dawit Alemayehu wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103973

Re: Review Request: New KPart extension for manupilating a browser engine's settings

2012-02-20 Thread Dawit Alemayehu
ced53a3 khtml/khtml_ext.cpp 6e8a846 kparts/htmlextension.h 9833d9a Diff: http://git.reviewboard.kde.org/r/103973/diff/ Testing --- Thanks, Dawit Alemayehu

Review Request: Fix signal/slot connections in kcmdolphinviewmodes

2012-02-20 Thread Dawit Alemayehu
--- Thanks, Dawit Alemayehu

Re: Review Request: New KPart extension for manupilating a browser engine's settings

2012-02-18 Thread Dawit Alemayehu
://git.reviewboard.kde.org/r/103973/diff/ Testing --- Thanks, Dawit Alemayehu

Re: Review Request: New KPart extension for manupilating a browser engine's settings

2012-02-14 Thread Dawit Alemayehu
/khtml_ext.cpp 6e8a846 khtml/khtml_part.cpp 24589e4 kparts/CMakeLists.txt 96fa31f kparts/browsersettingsextension.h PRE-CREATION kparts/browsersettingsextension.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/103973/diff/ Testing --- Thanks, Dawit Alemayehu

Review Request: New KPart extension for manupilating a browser engine's settings

2012-02-13 Thread Dawit Alemayehu
, Dawit Alemayehu

Re: Review Request: New KPart extension for manupilating a browser engine's settings

2012-02-13 Thread Dawit Alemayehu
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103973/#review10609 --- On Feb. 13, 2012, 7 p.m., Dawit Alemayehu wrote

Review Request: Prompt for the folder name when creating one in the sidebar bookmark module

2012-02-02 Thread Dawit Alemayehu
/bookmark_module.cpp 23b8e05 Diff: http://git.reviewboard.kde.org/r/103845/diff/diff Testing --- Follow the steps outlined in the bug report before and after the patch. Thanks, Dawit Alemayehu

Re: Review Request: Fix a VLC crash by delaying object deletion to avoid invalid access by QtDBus

2012-01-29 Thread Dawit Alemayehu
d8a9cb7 kio/kio/fileundomanager.cpp f580c29 kio/kio/scheduler.cpp 6b36d9d Diff: http://git.reviewboard.kde.org/r/100577/diff/diff Testing --- Thanks, Dawit Alemayehu

[kde-doc-english] [kde-baseapps/KDE/4.8] konqueror: Make sure the user's preference to show or hide the close buttons takes effect

2012-01-29 Thread Dawit Alemayehu
Git commit 2f63fe74e513580879cde3d4678d8268bd91da52 by Dawit Alemayehu. Committed on 29/01/2012 at 07:09. Pushed by adawit into branch 'KDE/4.8'. Make sure the user's preference to show or hide the close buttons takes effect as soon as the changes are applied. Not after restarting Konqueror. BUG

[kde-doc-english] [kde-baseapps] konqueror: Make sure the user's preference to show or hide the close buttons takes effect

2012-01-29 Thread Dawit Alemayehu
Git commit e828d4e693ef9d3648de808be10901cf04a840b7 by Dawit Alemayehu. Committed on 29/01/2012 at 07:09. Pushed by adawit into branch 'master'. Make sure the user's preference to show or hide the close buttons takes effect as soon as the changes are applied. Not after restarting Konqueror. BUG

Re: Review Request: Moving tabs causes wrong tab to be highlighted upon reload

2012-01-24 Thread Dawit Alemayehu
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103740/#review10065 --- On Jan. 22, 2012, 6:18 p.m., Dawit Alemayehu wrote

Re: Review Request: Moving tabs causes wrong tab to be highlighted upon reload

2012-01-22 Thread Dawit Alemayehu
, visit: http://git.reviewboard.kde.org/r/103740/#review9993 --- On Jan. 19, 2012, 9:28 p.m., Dawit Alemayehu wrote: --- This is an automatically generated e-mail. To reply, visit

Re: Review Request: Append .desktop to generated link files to prevent them from being identified as something else

2012-01-18 Thread Dawit Alemayehu
, Dawit Alemayehu

Review Request: Show the correct remote charset in encoding in Konqueror and Dolphin.

2012-01-18 Thread Dawit Alemayehu
to the remote encoding and check what is selected. Thanks, Dawit Alemayehu

Re: Review Request: Show the correct remote charset encoding in Konqueror's and Dolphin's Set Remote Encoding menu

2012-01-18 Thread Dawit Alemayehu
://git.reviewboard.kde.org/r/103730/diff/diff Testing --- 1.) Connect to a remote server. 2.) Change the remote charset encoding to Western European ( ISO-8859-1 ). 3.) Go back to the remote encoding and check what is selected. Thanks, Dawit Alemayehu

Review Request: Append .desktop to generated link files to prevent them from being identified as something else

2012-01-13 Thread Dawit Alemayehu
fbb0e48 Diff: http://git.reviewboard.kde.org/r/103691/diff/diff Testing --- Generate a link file to a URL location and see what KDE identifies the file as. Thanks, Dawit Alemayehu

Review Request: Prevent Konqueror crash when the location toolbar is disabled

2012-01-09 Thread Dawit Alemayehu
Diffs - konqueror/src/konqmainwindow.cpp 7675e3a Diff: http://git.reviewboard.kde.org/r/103663/diff/diff Testing --- Thanks, Dawit Alemayehu

Re: Review Request: Fix for stale permissions information in properties dialog

2012-01-05 Thread Dawit Alemayehu
ec3d622 Diff: http://git.reviewboard.kde.org/r/103555/diff/diff Testing --- Thanks, Dawit Alemayehu

Re: Review Request: Factor out and reuse the code that guesses mime-type from filename

2012-01-03 Thread Dawit Alemayehu
/browseropenorsavequestion.cpp 092198f Diff: http://git.reviewboard.kde.org/r/103617/diff/diff Testing --- Thanks, Dawit Alemayehu

Re: Review Request: Factor out and reuse the code that guesses mime-type from filename

2012-01-03 Thread Dawit Alemayehu
) - kparts/browseropenorsavequestion.cpp 092198f Diff: http://git.reviewboard.kde.org/r/103617/diff/diff Testing --- Thanks, Dawit Alemayehu

Re: Review Request: Do not show path info the Properties dialog edit box

2012-01-02 Thread Dawit Alemayehu
--- On Dec. 31, 2011, 5:23 a.m., Dawit Alemayehu wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103594

Re: Review Request: Do not emit KEditToolBar::newToolBarConfig twice

2012-01-02 Thread Dawit Alemayehu
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103579/#review9429 --- On Dec. 30, 2011, 6:39 a.m., Dawit Alemayehu wrote

Re: Review Request: Do not emit KEditToolBar::newToolBarConfig twice

2012-01-02 Thread Dawit Alemayehu
after pressing Apply. IOW, the changes have already been applied. Diffs (updated) - kdeui/dialogs/kedittoolbar.cpp ed572bf Diff: http://git.reviewboard.kde.org/r/103579/diff/diff Testing --- Thanks, Dawit Alemayehu

Review Request: Fix for SSL handeshake failure in TcpSlaveBase due to server not supporting SSL compression

2012-01-02 Thread Dawit Alemayehu
/tcpslavebase.h fea2c08 kio/kio/tcpslavebase.cpp d0f92b4 Diff: http://git.reviewboard.kde.org/r/103610/diff/diff Testing --- Visit the site listed in the bug report: https://tim.rz.rwth-aachen.de/mail-lifecycle/ Thanks, Dawit Alemayehu

Review Request: Use the safer Qt's version of string functions in kio_ftp

2012-01-02 Thread Dawit Alemayehu
versions of strlen, strcmp and strncmp. - Since there is no Qt version for atoi, make sure NULL pointers will not be used in atoi calls. Diffs - kioslave/ftp/ftp.cpp 41da792 Diff: http://git.reviewboard.kde.org/r/103616/diff/diff Testing --- Thanks, Dawit Alemayehu

Review Request: Factor out and reuse the code that guesses mime-type from filename

2012-01-02 Thread Dawit Alemayehu
/103617/diff/diff Testing --- Thanks, Dawit Alemayehu

Re: Review Request: Fix for SSL handeshake failure in TcpSlaveBase due to server not supporting SSL compression

2012-01-02 Thread Dawit Alemayehu
--- Visit the site listed in the bug report: https://tim.rz.rwth-aachen.de/mail-lifecycle/ Thanks, Dawit Alemayehu

Re: Review Request: Fix for stale permissions information in properties dialog

2012-01-02 Thread Dawit Alemayehu
on demand by KFileItem. Dawit Alemayehu wrote: Yes, but changing permissions is only one case for ending up here. The most common case is that KDirWatch (in Stat mode) notifies us that a directory has changed because files have been created in it, or deleted, etc. In that case we do

Re: Review Request: Factor out and reuse the code that guesses mime-type from filename

2012-01-02 Thread Dawit Alemayehu
/r/103617/#review9484 --- On Jan. 2, 2012, 8:41 p.m., Dawit Alemayehu wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103617

Re: Review Request: Factor out and reuse the code that guesses mime-type from filename

2012-01-02 Thread Dawit Alemayehu
(updated) - kparts/browseropenorsavequestion.cpp 092198f Diff: http://git.reviewboard.kde.org/r/103617/diff/diff Testing --- Thanks, Dawit Alemayehu

Re: Review Request: KOpenWithDialog: Quote paths selected in the file dialog

2012-01-01 Thread Dawit Alemayehu
simply quoting the user selected text in KOpenWithDialog::slotSelected not suffice ? - Dawit Alemayehu On Jan. 1, 2012, 9:49 p.m., Ingomar Wesp wrote: --- This is an automatically generated e-mail. To reply, visit: http

Review Request: Do not add two actions with the same name to Konqueror's actionCollection.

2011-12-30 Thread Dawit Alemayehu
in filenmanagement mode. Thanks, Dawit Alemayehu

Review Request: Do not show path info the Properties dialog edit box

2011-12-30 Thread Dawit Alemayehu
on the file listed and select Properties. * Check the name of the file displayed before and after the patch. * To verify the patch works, change the name of the file. Thanks, Dawit Alemayehu

Review Request: Do not emit KEditToolBar::newToolBarConfig twice

2011-12-29 Thread Dawit Alemayehu
/103579/diff/diff Testing --- Thanks, Dawit Alemayehu

Re: Review Request: Make kfmclient honor the user configured browser settings for local resources

2011-12-28 Thread Dawit Alemayehu
not factorize this and call that code for local files too? It would make the code shorter and easier to maintain. Dawit Alemayehu wrote: Well that would be nice and that was my first choice, but unfortunately that is not possible because KRun::init would need to be modified to use

Re: Review Request: Fix for stale permissions information in properties dialog

2011-12-28 Thread Dawit Alemayehu
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103555/#review9322 --- On Dec. 27, 2011, 6:26 p.m., Dawit Alemayehu wrote

Re: Review Request: Make kfmclient honor the user configured browser settings for local resources

2011-12-27 Thread Dawit Alemayehu
not factorize this and call that code for local files too? It would make the code shorter and easier to maintain. Dawit Alemayehu wrote: Well that would be nice and that was my first choice, but unfortunately that is not possible because KRun::init would need to be modified to use

Re: Review Request: Make kfmclient honor the user configured browser settings for local resources

2011-12-27 Thread Dawit Alemayehu
the browser in the default application list to firefox and make sure all of the following commands open the URL in firefox: * kfmclient openURL http://www.kde.org * kfmclient openURL /usr/share/doc/qt/html/index.html * kfmclient openURL ftp://ftp.kde.org Thanks, Dawit Alemayehu

Review Request: Fix for stale permissions information in properties dialog

2011-12-27 Thread Dawit Alemayehu
not been updated. This patch addresses that bug. This addresses bug 173733. http://bugs.kde.org/show_bug.cgi?id=173733 Diffs - kio/kio/kdirlister.cpp ec3d622 Diff: http://git.reviewboard.kde.org/r/103555/diff/diff Testing --- Thanks, Dawit Alemayehu

Re: Review Request: Make kfmclient honor the user configured browser settings for local resources

2011-12-26 Thread Dawit Alemayehu
not factorize this and call that code for local files too? It would make the code shorter and easier to maintain. Dawit Alemayehu wrote: Well that would be nice and that was my first choice, but unfortunately that is not possible because KRun::init would need to be modified to use

Re: Review Request: Make kfmclient honor the user configured browser settings for local resources

2011-12-25 Thread Dawit Alemayehu
not factorize this and call that code for local files too? It would make the code shorter and easier to maintain. Dawit Alemayehu wrote: Well that would be nice and that was my first choice, but unfortunately that is not possible because KRun::init would need to be modified to use

Re: Review Request: Make kfmclient honor the user configured browser settings for local resources

2011-12-24 Thread Dawit Alemayehu
. - Dawit --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103524/#review9228 --- On Dec. 24, 2011, 4:14 a.m., Dawit Alemayehu wrote

Review Request: Make kfmclient honor the user configured browser settings for local resources

2011-12-23 Thread Dawit Alemayehu
: * kfmclient openURL http://www.kde.org * kfmclient openURL /usr/share/doc/qt/html/index.html * kfmclient openURL ftp://ftp.kde.org Thanks, Dawit Alemayehu

[Okular-devel] [Bug 245134] konqueror crashes on session restore - takes down all windows

2011-12-21 Thread Dawit Alemayehu
https://bugs.kde.org/show_bug.cgi?id=245134 Dawit Alemayehu ada...@kde.org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Version

Review Request: Stop warning messages printed by KHTML when closing Konqueror's configuration dialog

2011-12-21 Thread Dawit Alemayehu
that is attached disables the bogus debug message printed out when closing Konqueror's configuration dialog. Diffs - khtml/khtml_part.cpp 9a02248 Diff: http://git.reviewboard.kde.org/r/103500/diff/diff Testing --- Thanks, Dawit Alemayehu

Re: Review Request: Add 20px margin when to info pages generated by kio_info

2011-12-16 Thread Dawit Alemayehu
://bugs.kde.org/show_bug.cgi?id=230959 Diffs (updated) - kioslave/info/kde-info2html 03ace75 Diff: http://git.reviewboard.kde.org/r/103426/diff/diff Testing --- Tested by displaying info:/make/Goals. Thanks, Dawit Alemayehu

Re: Review Request: Add 1em right padding to info pages generated by kio_info

2011-12-16 Thread Dawit Alemayehu
230959. http://bugs.kde.org/show_bug.cgi?id=230959 Diffs - kioslave/info/kde-info2html 03ace75 Diff: http://git.reviewboard.kde.org/r/103426/diff/diff Testing --- Tested by displaying info:/make/Goals. Thanks, Dawit Alemayehu

Re: Review Request: Add 1em right padding to info pages generated by kio_info

2011-12-16 Thread Dawit Alemayehu
/103426/#review8994 --- On Dec. 16, 2011, 2:15 p.m., Dawit Alemayehu wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103426

[Okular-devel] [Bug 240516] Viewing a PDF URL loses the keyboard input focus

2011-12-15 Thread Dawit Alemayehu
https://bugs.kde.org/show_bug.cgi?id=240516 Dawit Alemayehu ada...@kde.org changed: What|Removed |Added CC|ada...@kde.org | AssignedTo|konq-b

Review Request: Add 20px margin when to info pages generated by kio_info

2011-12-15 Thread Dawit Alemayehu
by displaying info:/make/Goals. Thanks, Dawit Alemayehu

Re: Review Request: Proper password caching when opening remote directories in KFileDialog

2011-11-29 Thread Dawit Alemayehu
--- On Nov. 24, 2011, 3:16 p.m., Dawit Alemayehu wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103226/ --- (Updated Nov

Re: Review Request: Proper password caching when opening remote directories in KFileDialog

2011-11-29 Thread Dawit Alemayehu
://git.reviewboard.kde.org/r/103226/diff/diff Testing --- Tested with the scenario outlined in the afforementioned bug report using a publicly available demo webdav server, webdav://demo.sabredav.org. Thanks, Dawit Alemayehu

Review Request: Do not show Remember Password checkbox if KWallet is disabled

2011-11-29 Thread Dawit Alemayehu
/show_bug.cgi?id=148171 Diffs - kpasswdserver/kpasswdserver.cpp cc8ded2 Diff: http://git.reviewboard.kde.org/r/103291/diff/diff Testing --- Thanks, Dawit Alemayehu

Review Request: Make it possible to login to 2 different sites from two separate instance of same application

2011-11-26 Thread Dawit Alemayehu
--- Attempt to log into two different sites from two different applications of two different instances of the same application. Thanks, Dawit Alemayehu

Re: Review Request: Make it possible to login to 2 different sites from two separate instance of same application

2011-11-26 Thread Dawit Alemayehu
://git.reviewboard.kde.org/r/103245/diff/diff Testing --- Attempt to log into two different sites from two different applications of two different instances of the same application. Thanks, Dawit Alemayehu

Re: Review Request: Proper password caching when opening remote directories in KFileDialog

2011-11-24 Thread Dawit Alemayehu
--- On Nov. 24, 2011, 7:37 a.m., Dawit Alemayehu wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103226

Re: Review Request: Proper password caching when opening remote directories in KFileDialog

2011-11-24 Thread Dawit Alemayehu
., Dawit Alemayehu wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103226/ --- (Updated Nov. 24, 2011, 7:37 a.m.) Review

Re: Review Request: Proper password caching when opening remote directories in KFileDialog

2011-11-24 Thread Dawit Alemayehu
outlined in the afforementioned bug report using a publicly available demo webdav server, webdav://demo.sabredav.org. Thanks, Dawit Alemayehu

Review Request: Proper password caching when opening remote directories in KFileDialog

2011-11-23 Thread Dawit Alemayehu
://demo.sabredav.org. Thanks, Dawit Alemayehu

Re: Review Request: Proper password caching when opening remote directories in KFileDialog

2011-11-23 Thread Dawit Alemayehu
a publicly available demo webdav server, webdav://demo.sabredav.org. Thanks, Dawit Alemayehu

[Okular-devel] [Bug 245134] konqueror crashes on session restore - takes down all windows

2011-11-21 Thread Dawit Alemayehu
https://bugs.kde.org/show_bug.cgi?id=245134 Dawit Alemayehu ada...@kde.org changed: What|Removed |Added CC||ada...@kde.org

[Bug 199375] Kontact/kmail crash during exit (QListData::shared_null, qDeleteAllQListKIO::Slave*, ProtocolInfo, Scheduler related)

2011-11-18 Thread Dawit Alemayehu
https://bugs.kde.org/show_bug.cgi?id=199375 Dawit Alemayehu ada...@kde.org changed: What|Removed |Added CC||regna...@web.de

[Bug 199375] Kontact/kmail crash during exit (QListData::shared_null, qDeleteAllQListKIO::Slave*, ProtocolInfo, Scheduler related)

2011-11-18 Thread Dawit Alemayehu
https://bugs.kde.org/show_bug.cgi?id=199375 Dawit Alemayehu ada...@kde.org changed: What|Removed |Added CC||ada...@kde.org --- Comment

[Bug 199375] Kontact/kmail crash during exit (QListData::shared_null, qDeleteAllQListKIO::Slave*, ProtocolInfo, Scheduler related)

2011-11-18 Thread Dawit Alemayehu
https://bugs.kde.org/show_bug.cgi?id=199375 --- Comment #102 from Dawit Alemayehu adawit kde org 2011-11-19 07:14:57 --- *** Bug 209374 has been marked as a duplicate of this bug. *** -- Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email --- You are receiving this mail

Review Request: Fix crash in KonqView

2011-11-17 Thread Dawit Alemayehu
/show_bug.cgi?id=285844 Diffs - konqueror/src/konqrun.cpp 9e9310c konqueror/src/konqview.h 45c5bde Diff: http://git.reviewboard.kde.org/r/103176/diff/diff Testing --- Thanks, Dawit Alemayehu

Re: Review Request: Fix crash in KonqView

2011-11-17 Thread Dawit Alemayehu
generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103176/#review8281 --- On Nov. 17, 2011, 8:33 p.m., Dawit Alemayehu wrote: --- This is an automatically generated e

Re: Review Request: Fix wrong whatsThis strings in proxy kcm

2011-11-14 Thread Dawit Alemayehu
of them. Please remove anything that talks about default values from all and commit that. Thanks. - Dawit Alemayehu On Nov. 14, 2011, 10:24 a.m., Burkhard Lück wrote: --- This is an automatically generated e-mail. To reply, visit: http

Review Request: [kio_sftp] Implement SlaveBase::copy to make file upload/download faster

2011-11-10 Thread Dawit Alemayehu
. Thanks, Dawit Alemayehu

Re: Review Request: Change KIO::DeleteJob to use KIO::http_delete for HTTP protocol

2011-11-04 Thread Dawit Alemayehu
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103038/#review7939 --- On Nov. 3, 2011, 5:59 p.m., Dawit Alemayehu wrote

Review Request: Change KIO::DeleteJob to use KIO::http_delete for HTTP protocol

2011-11-03 Thread Dawit Alemayehu
/deletejob.cpp 87c630a Diff: http://git.reviewboard.kde.org/r/103038/diff/diff Testing --- Thanks, Dawit Alemayehu

[Bug 145006] Changes in konqueror's proxy settings only effective after restart

2011-10-28 Thread Dawit Alemayehu
https://bugs.kde.org/show_bug.cgi?id=145006 Dawit Alemayehu ada...@kde.org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED CC

Re: Review Request: Add KIO::http_delete

2011-10-25 Thread Dawit Alemayehu
://git.reviewboard.kde.org/r/102938/diff/diff Testing (updated) --- Tested using http://www.mnot.net/javascript/xmlhttprequest/. Thanks, Dawit Alemayehu

Re: Review Request: Fix for MMB paste on Konqueror tabs

2011-10-25 Thread Dawit Alemayehu
://bugs.kde.org/show_bug.cgi?id=157753 Diffs (updated) - konqueror/src/konqmisc.cpp c46dc07 konqueror/src/konqtabs.cpp d627fad Diff: http://git.reviewboard.kde.org/r/102931/diff/diff Testing --- Thanks, Dawit Alemayehu

Re: Review Request: Cleanup the use of HTTPProtocol::httpClose

2011-10-25 Thread Dawit Alemayehu
) - kioslave/http/http.h 4c62841 kioslave/http/http.cpp 235ce7d Diff: http://git.reviewboard.kde.org/r/102801/diff/diff Testing --- Thanks, Dawit Alemayehu

Re: Review Request: [PATCH] Change HTML thumbnail generator to kdewebkit

2011-10-23 Thread Dawit Alemayehu
, Dawit Alemayehu

Review Request: Fix for MMB paste on Konqueror tabs

2011-10-21 Thread Dawit Alemayehu
d627fad Diff: http://git.reviewboard.kde.org/r/102931/diff/diff Testing --- Thanks, Dawit Alemayehu

[kde-doc-english] [kde-baseapps] konqueror/settings/kio: Rewrote the proxy configuration dialog to include missing SOCKS

2011-10-20 Thread Dawit Alemayehu
Git commit f90f318f2162df18a08b889d24ae8bbeab3b834c by Dawit Alemayehu. Committed on 01/08/2011 at 07:44. Pushed by adawit into branch 'master'. Rewrote the proxy configuration dialog to include missing SOCKS proxy entry and make it much simpler to use. BUG:82352 BUG:115704 BUG:139789 BUG:147340

Re: Review Request: Proxy overhaul Part 6: Proxy Configuration dialog rewrite

2011-10-18 Thread Dawit Alemayehu
? Dawit Alemayehu wrote: I guess Port: can be added. I simply copied what was in the old configuration dialog with very minor modifications. I do not have a preference one way or the other. I will add the port labels to clarify the interface and make sure users understand the purpose

Re: Review Request: Proxy overhaul Part 6: Proxy Configuration dialog rewrite

2011-10-18 Thread Dawit Alemayehu
visible http://git.reviewboard.kde.org/r/102802/s/303/ Thanks, Dawit Alemayehu

Re: Review Request: Proxy overhaul Part 6: Proxy Configuration dialog rewrite

2011-10-14 Thread Dawit Alemayehu
/s/301/ Alternate dialog with all options visible http://git.reviewboard.kde.org/r/102802/s/303/ Thanks, Dawit Alemayehu

Re: Review Request: Cleanup the use of HTTPProtocol::httpClose

2011-10-13 Thread Dawit Alemayehu
., Dawit Alemayehu wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102801/ --- (Updated Oct. 8, 2011, 5:16 a.m.) Review

Review Request: Improved proxy data caching in KProtocolManager

2011-10-13 Thread Dawit Alemayehu
are for different URLs. Diffs - kio/kio/kprotocolmanager.cpp 92519e4 Diff: http://git.reviewboard.kde.org/r/102840/diff/diff Testing --- Thanks, Dawit Alemayehu

Re: Review Request: kio_http: fix keepalive timeout parsing

2011-10-13 Thread Dawit Alemayehu
the iteratator function with the next in HTTPTokenizer ; so as Andreas already indicated feel free to commit this. - Dawit Alemayehu On Oct. 13, 2011, 5:59 p.m., Andrea Iacovitti wrote: --- This is an automatically generated e-mail. To reply

Review Request: Proxy overhaul Part 6: Proxy Configuration dialog rewrite

2011-10-13 Thread Dawit Alemayehu
. Screenshots --- No Proxy http://git.reviewboard.kde.org/r/102802/s/296/ PAC Proxy http://git.reviewboard.kde.org/r/102802/s/297/ System Proxy http://git.reviewboard.kde.org/r/102802/s/298/ Manual Proxy http://git.reviewboard.kde.org/r/102802/s/299/ Thanks, Dawit Alemayehu

Re: Review Request: Proxy overhaul Part 6: Proxy Configuration dialog rewrite

2011-10-13 Thread Dawit Alemayehu
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102802/#review7304 --- On Oct. 13, 2011, 9:13 p.m., Dawit Alemayehu wrote

Re: Review Request: Proxy overhaul Part 6: Proxy Configuration dialog rewrite

2011-10-13 Thread Dawit Alemayehu
/ System Proxy http://git.reviewboard.kde.org/r/102802/s/298/ Manual Proxy http://git.reviewboard.kde.org/r/102802/s/301/ Thanks, Dawit Alemayehu

Re: Review Request: kio_http: fix keepalive timeout parsing

2011-10-11 Thread Dawit Alemayehu
/#comment6307 This seems unnecessary to me since the parser produces a lower case token to begin with. - Dawit Alemayehu On Oct. 10, 2011, 10:35 p.m., Andrea Iacovitti wrote: --- This is an automatically generated e-mail. To reply

Review Request: Cleanup the use of HTTPProtocol::httpClose

2011-10-07 Thread Dawit Alemayehu
: http://git.reviewboard.kde.org/r/102801/diff/diff Testing --- Thanks, Dawit Alemayehu

Review Request: Fix HTTPProtocol::put

2011-09-30 Thread Dawit Alemayehu
/diff/diff Testing --- - Log into git.reviewboard.kde.org - Create a sample review request. - Attempt to change the People: assigned to it. - If you do not get a server error message, then try to Discard your changes. Thanks, Dawit Alemayehu

Re: Review Request: Fix KPasswdServer usage of KWallet

2011-09-30 Thread Dawit Alemayehu
On Sept. 29, 2011, 8:14 p.m., Dawit Alemayehu wrote: Sorry, but this is simply wrong. There is a specific reason why passwords are not blindly saved into the wallet before they are validated. The idea of prompting the user for password and actually storing that password need

Re: Review Request: Fix KPasswdServer usage of KWallet

2011-09-29 Thread Dawit Alemayehu
passwords are not blindly saved into the wallet before they are validated. The idea of prompting the user for password and actually storing that password need to be two separate actions. So the question is from which application or ioslave are you having this problem ? - Dawit Alemayehu

Re: Review Request: Proxy overhaul Part 5: Add support for trying multiple proxies to KIO HTTP

2011-09-27 Thread Dawit Alemayehu
; return SOCKS 127.0.0.1:; DIRECT; } Thanks, Dawit Alemayehu

Re: Review Request: Proxy overhaul Part 4: More proxy changes and fixes for KProtocolManager

2011-09-25 Thread Dawit Alemayehu
Testing --- Thanks, Dawit Alemayehu

Re: Review Request: Proxy overhaul Part 4: More proxy changes and fixes for KProtocolManager

2011-09-25 Thread Dawit Alemayehu
://git.reviewboard.kde.org/r/102691/#review6796 --- On Sept. 25, 2011, 4:15 p.m., Dawit Alemayehu wrote: --- This is an automatically generated e-mail. To reply, visit: http

<    1   2   3   4   5   6   7   >