Review Request: Do not try to hide the current part's widget when switching views

Dawit Alemayehu adawit at kde.org
Tue Dec 13 07:37:50 GMT 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103401/
-----------------------------------------------------------

Review request for KDE Base Apps and David Faure.


Description
-------

The attach patch fixes the issue stated in the aforementioned bug report. More specifically if you split a directory view in Konqueror and attempt to preview a embeded inside of the view that does not have the focus, you get a very weird behavior that randomly affects both sides of the split view. For the description of the problem and how to reproduce it, see the bug report.

The fix was to simply comment out the code that attempt to hide the widget of the current view (inside KonqView::switchView) before switching out the dolphin part (filemanagement) for the part that is going to show the content to be previewed. When this piece of code is removed KonqViewManager::doSetActivePart is not somehow invoked with the current DolphinPart as the part to be set as the active one. Why that happens I do not know. However, after watching what happens when attempting to reproduce BR# 258278, I am tempted to say that this fix might actually resolve several other bug reports where clicking on a file to preview it in Konqueror did not result in the correct outcome. For example, see bug 279937, bug 280364 and bug 282283.


This addresses bugs 258278 and 267215.
    http://bugs.kde.org/show_bug.cgi?id=258278
    http://bugs.kde.org/show_bug.cgi?id=267215


Diffs
-----

  konqueror/src/konqview.cpp f1e12b9 

Diff: http://git.reviewboard.kde.org/r/103401/diff/diff


Testing
-------


Thanks,

Dawit Alemayehu

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20111213/33f29ac3/attachment.htm>


More information about the kde-core-devel mailing list