diff options
author | Pascal Quantin <pascal.quantin@gmail.com> | 2015-06-23 10:31:58 -0700 |
---|---|---|
committer | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2015-06-23 18:27:44 +0000 |
commit | 0901e26505ca57b8331aa3edc9e43f1f087a3d77 (patch) | |
tree | 24041180d51ece259a881064569e77fae8fdd6e1 /ui/qt/decode_as_dialog.h | |
parent | 8bc294c0d12e742d8c539177a683eff16653b31b (diff) | |
download | wireshark-0901e26505ca57b8331aa3edc9e43f1f087a3d77.tar.gz |
Qt: change default behavior of 'Decode As' window
Up to know Qt 'Decode As' window was saving manual configuration automatically, contrary to GTK UI.
This can be misleading when decoding protocols without a fixed identifier (port number, USB bus/device id, ...).
Opening a new trace might lead to previous and now irrelevant settings being applied.
Make this optional and add a Save button to permanently store the current configuration.
Change-Id: I077c560f9e71cab16a74247e2e9d87523c0ed85e
Reviewed-on: https://code.wireshark.org/review/9058
Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Gerald Combs <gerald@wireshark.org>
Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
Diffstat (limited to 'ui/qt/decode_as_dialog.h')
-rw-r--r-- | ui/qt/decode_as_dialog.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/ui/qt/decode_as_dialog.h b/ui/qt/decode_as_dialog.h index 9aab4c21c8..1b11a84702 100644 --- a/ui/qt/decode_as_dialog.h +++ b/ui/qt/decode_as_dialog.h @@ -30,6 +30,7 @@ #include <QDialog> #include <QMap> +#include <QAbstractButton> class QTreeWidgetItem; class QComboBox; @@ -85,8 +86,8 @@ private slots: void selectorEditTextChanged(const QString & text); void curProtoCurrentIndexChanged(const QString & text); void curProtoDestroyed(); - void on_buttonBox_accepted(); - void on_buttonBox_helpRequested(); + void applyChanges(); + void on_buttonBox_clicked(QAbstractButton *button); }; #endif // DECODE_AS_DIALOG_H |