Press Start instead of Manage Interfaces on Enter
authorPeter Wu <peter@lekensteyn.nl>
Wed, 25 Nov 2015 11:51:35 +0000 (12:51 +0100)
committerPeter Wu <peter@lekensteyn.nl>
Wed, 25 Nov 2015 15:20:40 +0000 (15:20 +0000)
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 <peter@lekensteyn.nl>
Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Peter Wu <peter@lekensteyn.nl>
ui/qt/capture_interfaces_dialog.cpp
ui/qt/extcap_options_dialog.cpp

index 71a587e9fd016f47f7515d40e658f52f6e08566d..7f139890e6e7a228489aa25b99bdcfa9218e1535 100644 (file)
@@ -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()));
index 91b010dee4ca0564a6c5b05af1cf0f88dfc3a469..d83a4d892e13074b8f29fb9136fbf4d220b69e91 100644 (file)
@@ -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()));