aboutsummaryrefslogtreecommitdiffstats
path: root/ui
diff options
context:
space:
mode:
Diffstat (limited to 'ui')
-rw-r--r--ui/gtk/prefs_dlg.c38
-rw-r--r--ui/preference_utils.c4
-rw-r--r--ui/qt/main_window_preferences_frame.ui2
-rw-r--r--ui/qt/module_preferences_scroll_area.cpp98
-rw-r--r--ui/qt/module_preferences_scroll_area.h2
-rw-r--r--ui/qt/preferences_dialog.cpp17
6 files changed, 135 insertions, 26 deletions
diff --git a/ui/gtk/prefs_dlg.c b/ui/gtk/prefs_dlg.c
index f3a2189593..fb6236c425 100644
--- a/ui/gtk/prefs_dlg.c
+++ b/ui/gtk/prefs_dlg.c
@@ -76,8 +76,8 @@ static gboolean prefs_main_delete_event_cb(GtkWidget *, GdkEvent *, gpointer);
static void prefs_main_destroy_cb(GtkWidget *, gpointer);
static void prefs_tree_select_cb(GtkTreeSelection *, gpointer);
-static GtkWidget *create_preference_filename_entry(GtkWidget *, int,
- const gchar *, const gchar *, char *);
+static GtkWidget *create_preference_path_entry(GtkWidget *, int,
+ const gchar *, const gchar *, char *, gboolean);
#define E_PREFSW_SCROLLW_KEY "prefsw_scrollw"
#define E_PREFSW_TREE_KEY "prefsw_tree"
@@ -213,10 +213,17 @@ pref_show(pref_t *pref, gpointer user_data)
break;
case PREF_FILENAME:
- pref->control = create_preference_filename_entry(main_grid, pref->ordinal,
+ pref->control = create_preference_path_entry(main_grid, pref->ordinal,
label_string,
tooltip_txt,
- pref->stashed_val.string);
+ pref->stashed_val.string, FALSE);
+ break;
+
+ case PREF_DIRNAME:
+ pref->control = create_preference_path_entry(main_grid, pref->ordinal,
+ label_string,
+ tooltip_txt,
+ pref->stashed_val.string, TRUE);
break;
case PREF_RANGE:
@@ -839,13 +846,21 @@ static void
preference_filename_entry_cb(GtkWidget *button, GtkWidget *filename_te)
{
/* XXX - use a better browser dialog title */
- file_selection_browse(button, filename_te, "Wireshark: file preference",
+ file_selection_browse(button, filename_te, "Wireshark: File preference",
FILE_SELECTION_READ_BROWSE);
}
+static void
+preference_dirname_entry_cb(GtkWidget *button, GtkWidget *filename_te)
+{
+ /* XXX - use a better browser dialog title */
+ file_selection_browse(button, filename_te, "Wireshark: Directory preference",
+ FILE_SELECTION_CREATE_FOLDER);
+}
+
static GtkWidget *
-create_preference_filename_entry(GtkWidget *main_grid, int grid_position,
- const gchar *label_text, const gchar *tooltip_text, char *value)
+create_preference_path_entry(GtkWidget *main_grid, int grid_position,
+ const gchar *label_text, const gchar *tooltip_text, char *value, gboolean dir_only)
{
GtkWidget *entry;
GtkWidget *button, *file_bt_hb;
@@ -867,8 +882,11 @@ create_preference_filename_entry(GtkWidget *main_grid, int grid_position,
gtk_widget_set_tooltip_text(entry, tooltip_text);
gtk_widget_show(entry);
- g_signal_connect(button, "clicked", G_CALLBACK(preference_filename_entry_cb), entry);
-
+ if (dir_only) {
+ g_signal_connect(button, "clicked", G_CALLBACK(preference_dirname_entry_cb), entry);
+ } else {
+ g_signal_connect(button, "clicked", G_CALLBACK(preference_filename_entry_cb), entry);
+ }
return entry;
}
@@ -956,6 +974,7 @@ pref_check(pref_t *pref, gpointer user_data)
case PREF_STRING:
case PREF_FILENAME:
+ case PREF_DIRNAME:
/* Value can't be bad. */
break;
@@ -1054,6 +1073,7 @@ pref_fetch(pref_t *pref, gpointer user_data)
case PREF_STRING:
case PREF_FILENAME:
+ case PREF_DIRNAME:
str_val = gtk_entry_get_text(GTK_ENTRY(pref->control));
if (strcmp(*pref->varp.string, str_val) != 0) {
*pref_changed_p = TRUE;
diff --git a/ui/preference_utils.c b/ui/preference_utils.c
index 29e403f5f8..005817a10c 100644
--- a/ui/preference_utils.c
+++ b/ui/preference_utils.c
@@ -61,6 +61,7 @@ pref_stash(pref_t *pref, gpointer unused _U_)
case PREF_STRING:
case PREF_FILENAME:
+ case PREF_DIRNAME:
g_free(pref->stashed_val.string);
pref->stashed_val.string = g_strdup(*pref->varp.string);
break;
@@ -117,6 +118,7 @@ pref_unstash(pref_t *pref, gpointer changed_p)
case PREF_STRING:
case PREF_FILENAME:
+ case PREF_DIRNAME:
if (strcmp(*pref->varp.string, pref->stashed_val.string) != 0) {
*pref_changed_p = TRUE;
g_free((void *)*pref->varp.string);
@@ -166,6 +168,7 @@ reset_stashed_pref(pref_t *pref) {
case PREF_STRING:
case PREF_FILENAME:
+ case PREF_DIRNAME:
g_free(pref->stashed_val.string);
pref->stashed_val.string = g_strdup(pref->default_val.string);
break;
@@ -206,6 +209,7 @@ pref_clean_stash(pref_t *pref, gpointer unused _U_)
case PREF_STRING:
case PREF_FILENAME:
+ case PREF_DIRNAME:
if (pref->stashed_val.string != NULL) {
g_free(pref->stashed_val.string);
pref->stashed_val.string = NULL;
diff --git a/ui/qt/main_window_preferences_frame.ui b/ui/qt/main_window_preferences_frame.ui
index 7a4e8af51c..42646127c3 100644
--- a/ui/qt/main_window_preferences_frame.ui
+++ b/ui/qt/main_window_preferences_frame.ui
@@ -70,7 +70,7 @@
<item row="1" column="2">
<widget class="QPushButton" name="foStyleSpecifiedPushButton">
<property name="text">
- <string>Choose...</string>
+ <string>Browse...</string>
</property>
</widget>
</item>
diff --git a/ui/qt/module_preferences_scroll_area.cpp b/ui/qt/module_preferences_scroll_area.cpp
index ada6e557d2..e30dbdbada 100644
--- a/ui/qt/module_preferences_scroll_area.cpp
+++ b/ui/qt/module_preferences_scroll_area.cpp
@@ -32,6 +32,7 @@
#include <QButtonGroup>
#include <QCheckBox>
#include <QComboBox>
+#include <QFileDialog>
#include <QHBoxLayout>
#include <QLabel>
#include <QLineEdit>
@@ -152,6 +153,29 @@ pref_show(pref_t *pref, gpointer layout_ptr)
vb->addLayout(hb);
break;
}
+ case PREF_FILENAME:
+ case PREF_DIRNAME:
+ {
+ vb->addWidget(new QLabel(pref->title));
+ QHBoxLayout *hb = new QHBoxLayout();
+ QLineEdit *path_le = new QLineEdit();
+ QStyleOption style_opt;
+ path_le->setProperty(pref_prop_, qVariantFromValue(pref));
+ path_le->setMinimumWidth(path_le->fontMetrics().height() * 20);
+ path_le->setStyleSheet(QString(
+ "QLineEdit {"
+ " margin-left: %1px;"
+ "}"
+ )
+ .arg(path_le->style()->subElementRect(QStyle::SE_CheckBoxContents, &style_opt).left()));
+ hb->addWidget(path_le);
+ QPushButton *path_pb = new QPushButton("Browse...");
+ path_pb->setProperty(pref_prop_, qVariantFromValue(pref));
+ hb->addWidget(path_pb);
+ hb->addSpacerItem(new QSpacerItem(1, 1, QSizePolicy::Expanding, QSizePolicy::Minimum));
+ vb->addLayout(hb);
+ break;
+ }
case PREF_COLOR:
{
// XXX - Not needed yet. When it is needed we can add a label + QFrame which pops up a
@@ -182,12 +206,20 @@ ModulePreferencesScrollArea::ModulePreferencesScrollArea(module_t *module, QWidg
pref_t *pref = le->property(pref_prop_).value<pref_t *>();
if (!pref) continue;
- if (pref->type == PREF_UINT) {
+ switch (pref->type) {
+ case PREF_UINT:
connect(le, SIGNAL(textEdited(QString)), this, SLOT(uintLineEditTextEdited(QString)));
- } else if (pref->type == PREF_STRING) {
+ break;
+ case PREF_STRING:
+ case PREF_FILENAME:
+ case PREF_DIRNAME:
connect(le, SIGNAL(textEdited(QString)), this, SLOT(stringLineEditTextEdited(QString)));
- } else if (pref->type == PREF_RANGE) {
+ break;
+ case PREF_RANGE:
connect(le, SIGNAL(textEdited(QString)), this, SLOT(rangeSyntaxLineEditTextEdited(QString)));
+ break;
+ default:
+ break;
}
}
@@ -200,30 +232,34 @@ ModulePreferencesScrollArea::ModulePreferencesScrollArea(module_t *module, QWidg
}
}
- foreach (QRadioButton *enum_rb, findChildren<QRadioButton *>()) {
- pref_t *pref = enum_rb->property(pref_prop_).value<pref_t *>();
+ foreach (QRadioButton *rb, findChildren<QRadioButton *>()) {
+ pref_t *pref = rb->property(pref_prop_).value<pref_t *>();
if (!pref) continue;
if (pref->type == PREF_ENUM && pref->info.enum_info.radio_buttons) {
- connect(enum_rb, SIGNAL(toggled(bool)), this, SLOT(enumRadioButtonToggled(bool)));
+ connect(rb, SIGNAL(toggled(bool)), this, SLOT(enumRadioButtonToggled(bool)));
}
}
- foreach (QComboBox *enum_cb, findChildren<QComboBox *>()) {
- pref_t *pref = enum_cb->property(pref_prop_).value<pref_t *>();
+ foreach (QComboBox *combo, findChildren<QComboBox *>()) {
+ pref_t *pref = combo->property(pref_prop_).value<pref_t *>();
if (!pref) continue;
if (pref->type == PREF_ENUM && !pref->info.enum_info.radio_buttons) {
- connect(enum_cb, SIGNAL(currentIndexChanged(int)), this, SLOT(enumComboBoxCurrentIndexChanged(int)));
+ connect(combo, SIGNAL(currentIndexChanged(int)), this, SLOT(enumComboBoxCurrentIndexChanged(int)));
}
}
- foreach (QPushButton *uat_pb, findChildren<QPushButton *>()) {
- pref_t *pref = uat_pb->property(pref_prop_).value<pref_t *>();
+ foreach (QPushButton *pb, findChildren<QPushButton *>()) {
+ pref_t *pref = pb->property(pref_prop_).value<pref_t *>();
if (!pref) continue;
if (pref->type == PREF_UAT) {
- connect(uat_pb, SIGNAL(pressed()), this, SLOT(uatPushButtonPressed()));
+ connect(pb, SIGNAL(pressed()), this, SLOT(uatPushButtonPressed()));
+ } else if (pref->type == PREF_FILENAME) {
+ connect(pb, SIGNAL(pressed()), this, SLOT(filenamePushButtonPressed()));
+ } else if (pref->type == PREF_DIRNAME) {
+ connect(pb, SIGNAL(pressed()), this, SLOT(dirnamePushButtonPressed()));
}
}
@@ -400,6 +436,44 @@ void ModulePreferencesScrollArea::uatPushButtonPressed()
qDebug() << "FIX Implement UATs:" << pref->title;
}
+void ModulePreferencesScrollArea::filenamePushButtonPressed()
+{
+ QPushButton *filename_pb = qobject_cast<QPushButton*>(sender());
+ if (!filename_pb) return;
+
+ pref_t *pref = filename_pb->property(pref_prop_).value<pref_t *>();
+ if (!pref) return;
+
+ QString filename = QFileDialog::getSaveFileName(this,
+ QString("Wireshark: ") + pref->description,
+ pref->stashed_val.string);
+
+ if (!filename.isEmpty()) {
+ g_free((void *)pref->stashed_val.string);
+ pref->stashed_val.string = qstring_strdup(filename);
+ updateWidgets();
+ }
+}
+
+void ModulePreferencesScrollArea::dirnamePushButtonPressed()
+{
+ QPushButton *dirname_pb = qobject_cast<QPushButton*>(sender());
+ if (!dirname_pb) return;
+
+ pref_t *pref = dirname_pb->property(pref_prop_).value<pref_t *>();
+ if (!pref) return;
+
+ QString dirname = QFileDialog::getExistingDirectory(this,
+ QString("Wireshark: ") + pref->description,
+ pref->stashed_val.string);
+
+ if (!dirname.isEmpty()) {
+ g_free((void *)pref->stashed_val.string);
+ pref->stashed_val.string = qstring_strdup(dirname);
+ updateWidgets();
+ }
+}
+
/*
* Editor modelines
*
diff --git a/ui/qt/module_preferences_scroll_area.h b/ui/qt/module_preferences_scroll_area.h
index 8a297ef86c..f5be4c7c1b 100644
--- a/ui/qt/module_preferences_scroll_area.h
+++ b/ui/qt/module_preferences_scroll_area.h
@@ -65,6 +65,8 @@ private slots:
void stringLineEditTextEdited(const QString &new_str);
void rangeSyntaxLineEditTextEdited(const QString &new_str);
void uatPushButtonPressed();
+ void filenamePushButtonPressed();
+ void dirnamePushButtonPressed();
};
#endif // MODULE_PREFERENCES_SCROLL_AREA_H
diff --git a/ui/qt/preferences_dialog.cpp b/ui/qt/preferences_dialog.cpp
index 4c2da2c703..8332b37025 100644
--- a/ui/qt/preferences_dialog.cpp
+++ b/ui/qt/preferences_dialog.cpp
@@ -48,7 +48,6 @@
#include <QKeyEvent>
#include <QLineEdit>
#include <QMessageBox>
-#include <QPushButton>
#include <QSpacerItem>
#include <QTreeWidgetItemIterator>
@@ -427,6 +426,7 @@ bool PreferencesDialog::stashedPrefIsDefault(pref_t *pref)
case PREF_STRING:
case PREF_FILENAME:
+ case PREF_DIRNAME:
if (!(g_strcmp0(pref->default_val.string, pref->stashed_val.string)))
return true;
break;
@@ -597,10 +597,19 @@ void PreferencesDialog::on_advancedTree_itemActivated(QTreeWidgetItem *item, int
break;
}
case PREF_FILENAME:
+ case PREF_DIRNAME:
{
- QString filename = QFileDialog::getSaveFileName(this,
- QString("Wireshark: ") + pref->description,
- pref->stashed_val.string);
+ QString filename;
+
+ if (pref->type == PREF_FILENAME) {
+ filename = QFileDialog::getSaveFileName(this,
+ QString("Wireshark: ") + pref->description,
+ pref->stashed_val.string);
+ } else {
+ filename = QFileDialog::getExistingDirectory(this,
+ QString("Wireshark: ") + pref->description,
+ pref->stashed_val.string);
+ }
if (!filename.isEmpty()) {
g_free((void *)pref->stashed_val.string);
pref->stashed_val.string = qstring_strdup(filename);
generated by cgit v1.2.3 (git 2.25.1) at 2024-05-16 11:07:37 +0000