From dde848b3d925d040592bd3f86487e9bc86905550 Mon Sep 17 00:00:00 2001 From: Peter Wu Date: Wed, 25 Nov 2015 12:51:35 +0100 Subject: [PATCH] Press Start instead of Manage Interfaces on Enter In the Capture Interfaces dialog, the default dialog action was non-existing. Fix Extcap while at it (searched for "YesRole"). Reported on #wireshark at Freenode. Change-Id: I7920b806a855acc20dcd2081f6b0d58e993b4ac1 Reviewed-on: https://code.wireshark.org/review/12136 Petri-Dish: Peter Wu Reviewed-by: Alexis La Goutte Petri-Dish: Alexis La Goutte Tested-by: Petri Dish Buildbot Reviewed-by: Peter Wu --- ui/qt/capture_interfaces_dialog.cpp | 2 +- ui/qt/extcap_options_dialog.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/ui/qt/capture_interfaces_dialog.cpp b/ui/qt/capture_interfaces_dialog.cpp index 71a587e9fd..7f139890e6 100644 --- a/ui/qt/capture_interfaces_dialog.cpp +++ b/ui/qt/capture_interfaces_dialog.cpp @@ -120,7 +120,7 @@ CaptureInterfacesDialog::CaptureInterfacesDialog(QWidget *parent) : stat_cache_ = NULL; // XXX - Enable / disable as needed - start_bt_ = ui->buttonBox->addButton(tr("Start"), QDialogButtonBox::YesRole); + start_bt_ = ui->buttonBox->addButton(tr("Start"), QDialogButtonBox::AcceptRole); start_bt_->setEnabled((global_capture_opts.num_selected > 0)? true: false); connect(start_bt_, SIGNAL(clicked(bool)), this, SLOT(start_button_clicked())); diff --git a/ui/qt/extcap_options_dialog.cpp b/ui/qt/extcap_options_dialog.cpp index 91b010dee4..d83a4d892e 100644 --- a/ui/qt/extcap_options_dialog.cpp +++ b/ui/qt/extcap_options_dialog.cpp @@ -67,7 +67,7 @@ ExtcapOptionsDialog::ExtcapOptionsDialog(QWidget *parent) : device_idx = 0; - start_bt_ = ui->buttonBox->addButton(tr("Start"), QDialogButtonBox::YesRole); + start_bt_ = ui->buttonBox->addButton(tr("Start"), QDialogButtonBox::AcceptRole); start_bt_->setEnabled((global_capture_opts.num_selected > 0)? true: false); connect(start_bt_, SIGNAL(clicked(bool)), this, SLOT(start_button_clicked())); -- 2.34.1