Wireshark-commits: [Wireshark-commits] master 56425fc: Qt: allow keyboard navigation in Unsaved pac
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 18 Mar 2018 14:10:54 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=56425fce7c16abb4138daf2e82ca5158d187fb4c
Submitter: Alexis La Goutte (alexis.lagoutte@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

56425fc by Peter Wu (peter@xxxxxxxxxxxxx):

    Qt: allow keyboard navigation in Unsaved packets dialog
    
    Currently the "Continue without Saving" button is visually highlighted
    in the "Unsaved packets..." dialog, but pressing Enter triggers "Save"
    instead. Even after changing button focus with arrow or tab keys,
    pressing Enter will not confirm the action (but Spacebar does).
    Restore the expected behavior for Linux and Windows, but preserve the
    fix for macOS since (for which this was originally added).
    
    Bug: 14531
    Change-Id: Ic20fc5809b55949f6fd960bcb32618a4fa7fd1e9
    Fixes: v2.3.0rc0-2672-gb0335359e5 ("Qt: Give discard button focus (but not as default)")
    Reviewed-on: https://code.wireshark.org/review/26511
    Petri-Dish: Peter Wu <peter@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Stig Bjørlykke <stig@xxxxxxxxxxxxx>
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    

Actions performed:

    from  dd87890   QUIC: use GLib constant
    adds  56425fc   Qt: allow keyboard navigation in Unsaved packets dialog


Summary of changes:
 ui/qt/capture_file_dialog.cpp | 2 ++
 ui/qt/main_window.cpp         | 2 ++
 2 files changed, 4 insertions(+)