Well, at least via close button ;-) Reproducible: Always Steps to Reproduce: 1. Open dolphin 2. Open a new tab 3. Try to close dolphin Actual Results: Nothing happens Expected Results: Window should close
I can confirm the problem, thanks for reporting it! I actually have a step 4 though: Click "OK" in the confirmation dialog. Can you confirm that, or do you not get a confirmation dialog at all? Proposed fix is at https://git.reviewboard.kde.org/r/118508/
(In reply to comment #1) > I can confirm the problem, thanks for reporting it! > > I actually have a step 4 though: Click "OK" in the confirmation dialog. Can > you confirm that, or do you not get a confirmation dialog at all? Yes, i get the dialog, just i forgot to mention it ;-) (also 'Close current tab' does nothing) > Proposed fix is at https://git.reviewboard.kde.org/r/118508/ will be testing
i can confirm that the patch fixes the bug here also =) thanks for the quick fix!
Git commit 40599aaa3f4aeee9293acc692a08e839f4140d04 by Frank Reininghaus. Committed on 04/06/2014 at 19:56. Pushed by freininghaus into branch 'frameworks'. Make Dolphin windows with multiple tabs closable The return type and return values of KMessageBox::createKMessageBox() have changed in KF5. We have to adjust the code in DolphinMainWindow::closeEvent(QCloseEvent* event) in order to make windows with multiple tabs closable. REVIEW: 118508 M +2 -2 dolphin/src/dolphinmainwindow.cpp http://commits.kde.org/kde-baseapps/40599aaa3f4aeee9293acc692a08e839f4140d04
(In reply to comment #3) > i can confirm that the patch fixes the bug here also =) Great :-) > thanks for the quick fix! Thanks for testing it!
Git commit 6d42545beedb26715220a4417aa7b5558b61acd9 by Veaceslav Munteanu. Committed on 02/09/2014 at 12:13. Pushed by munteanu into branch 'master'. M +2 -1 NEWS M +18 -15 digikam/tags/tagsmanager/tagsmanager.cpp http://commits.kde.org/digikam/6d42545beedb26715220a4417aa7b5558b61acd9
I'm sorry I messed up BUG number :D This commit is not related to this bug !
(In reply to Veaceslav Munteanu from comment #7) > I'm sorry I messed up BUG number :D > > This commit is not related to this bug ! Okay, but there is no reason to reopen this bug (which has been fixed months ago). Closing again.