RESOLVED FIXED 176619
[GTK] Improve Ctrl+W and Ctrl+Q shortcuts in MiniBrowser
https://bugs.webkit.org/show_bug.cgi?id=176619
Summary [GTK] Improve Ctrl+W and Ctrl+Q shortcuts in MiniBrowser
Michael Catanzaro
Reported 2017-09-08 11:05:14 PDT
In MiniBrowser, Ctrl+W closes the entire browser, but it should close only a single tab. It's been broken since support for multiple tabs was added.
Attachments
Patch (2.37 KB, patch)
2017-09-08 11:06 PDT, Michael Catanzaro
cgarcia: review+
Patch (4.08 KB, patch)
2017-09-09 12:24 PDT, Michael Catanzaro
no flags
Patch (4.14 KB, patch)
2017-09-09 12:28 PDT, Michael Catanzaro
no flags
Michael Catanzaro
Comment 1 2017-09-08 11:06:08 PDT
Carlos Garcia Campos
Comment 2 2017-09-08 23:56:24 PDT
Comment on attachment 320279 [details] Patch Shouldn't we try close here too?
Michael Catanzaro
Comment 3 2017-09-09 08:01:36 PDT
(In reply to Carlos Garcia Campos from comment #2) > Comment on attachment 320279 [details] > Patch > > Shouldn't we try close here too? Um... yes. I wrote this patch first and didn't remember that try_close existed. We should do it for Ctrl+Q as well.
Michael Catanzaro
Comment 4 2017-09-09 12:24:30 PDT
Michael Catanzaro
Comment 5 2017-09-09 12:28:16 PDT
WebKit Commit Bot
Comment 6 2017-09-10 02:54:35 PDT
Comment on attachment 320353 [details] Patch Clearing flags on attachment: 320353 Committed r221833: <http://trac.webkit.org/changeset/221833>
WebKit Commit Bot
Comment 7 2017-09-10 02:54:36 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.