CTRL+Q should only close the current window, not all windows.
Bug 401495
Details
- Reviewers
drosca krishremya - Group Reviewers
Falkon
Diff Detail
- Repository
- R875 Falkon
- Lint
Lint Skipped - Unit
Unit Tests Skipped
src/lib/app/browserwindow.cpp | ||
---|---|---|
469 | Why do you need to change shortcut to close current tab ? |
In the bug report, it says to close the current window using CTRL+Q. So I just changed the shortcut for closing the current tab.
CTRL+Q added as an optional shortcut for closing the current tab instead of closing all windows.
-1 from my side.
Ctrl-Q is standard shortcut for exit application. If one wants to close a window, use Alt-F4 or (x) button in upper right. The hole behaviour of a multi-mainwindow app versus several single main window apps is unfortunately not properly definded. But IMHO this should be reassiged to the HIG Group before every multi-mainwindow application implements it's own behaviour.
Just my 2 cents.
The Ctrl-Q use case is: I use 'start falcon with the the windows and tabs as on close' . This implies I need a way to close all windows at once. Otherwise flacon will not (re)start with several windows.
After along thinking I believe this is exact example of what should be handled by Keyboard Shortcut Manager and not directly in the code.
So, instead of trying to go around all possible situations when this shortcut should or should not be active, we should work on implementing Keyboard Shortcut Manager.
Since this is the case, are there any volunteers ?
It seems nobody takes this up, shall we close it until a new proper request is submitted?