https://bugs.kde.org/show_bug.cgi?id=458933
Stefano Crocco <stefano.cro...@alice.it> changed: What |Removed |Added ---------------------------------------------------------------------------- Latest Commit|https://invent.kde.org/netw |https://invent.kde.org/netw |ork/konqueror/commit/c9f3d1 |ork/konqueror/commit/429936 |957f7b6c168a025b5b4014e46a5 |da48ebd8bcc6f75dacd9180eb08 |f49bb63 |6e4add3 --- Comment #7 from Stefano Crocco <stefano.cro...@alice.it> --- Git commit 429936da48ebd8bcc6f75dacd9180eb086e4add3 by Stefano Crocco. Committed on 22/09/2022 at 13:06. Pushed by stefanocrocco into branch 'cherry-pick-c9f3d195'. Fix crash caused by using original view rather than actual view UrlLoader::decideEmbedOrSave used the original view when deciding whether it could be reused to open the URL, instead of using the view provided using UrlLoader::setView. If the URL was opened by clicking on a sidebar item, this caused a sidebar view to be used, rather than the correct dolphin part. This lead to a crash because the sidebar view is a passive view, which caused KonqViewManager::chooseNextView to return nullptr (cherry picked from commit c9f3d1957f7b6c168a025b5b4014e46a5f49bb63) M +1 -1 src/konqmainwindow.cpp M +6 -8 src/urlloader.cpp https://invent.kde.org/network/konqueror/commit/429936da48ebd8bcc6f75dacd9180eb086e4add3 -- You are receiving this mail because: You are watching all bug changes.