Qt: "Conversation" → "Coloring"

Change actionViewColorizeNewConversationRule to
actionViewColorizeNewColoringRule and change its text accordingly.
We're building a rule based on a general filter, not a conversation.

Change-Id: Ic408050eed6f24690e3e759e6963f9930fe4b5e9
Reviewed-on: https://code.wireshark.org/review/13266
Reviewed-by: Gerald Combs <gerald@wireshark.org>
Petri-Dish: Gerald Combs <gerald@wireshark.org>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Anders Broman <a.broman58@gmail.com>

diff --git a/ui/qt/main_window.h b/ui/qt/main_window.h
index 6d0a610aea082119636796026978ed6bfbdcafd7..e0cbdcc668c188d7745eabde4d69dc8190da7a89 100644 (file)
--- a/ui/qt/main_window.h
+++ b/ui/qt/main_window.h
@@ -417,7 +417,7 @@ private slots:
     void colorizeConversation(bool create_rule = false);
     void colorizeWithFilter();
     void on_actionViewColorizeResetColorization_triggered();
-    void on_actionViewColorizeNewConversationRule_triggered();
+    void on_actionViewColorizeNewColoringRule_triggered();
     void on_actionViewResizeColumns_triggered();
 
     void on_actionViewInternalsConversationHashTables_triggered();

diff --git a/ui/qt/main_window.ui b/ui/qt/main_window.ui
index af190c0ac49513c213f14d299e2f1c95e84cd885..667a3e2bbdfbb853f353c99b5f63845e11ee5620 100644 (file)
--- a/ui/qt/main_window.ui
+++ b/ui/qt/main_window.ui
@@ -306,7 +306,7 @@
      <addaction name="actionViewColorizeConversation10"/>
      <addaction name="separator"/>
      <addaction name="actionViewColorizeResetColorization"/>
-     <addaction name="actionViewColorizeNewConversationRule"/>
+     <addaction name="actionViewColorizeNewColoringRule"/>
     </widget>
     <widget class="QMenu" name="menuInternals">
      <property name="title">
@@ -2783,12 +2783,12 @@
     <string>Mark the current conversation with its own color.</string>
    </property>
   </action>
-  <action name="actionViewColorizeNewConversationRule">
+  <action name="actionViewColorizeNewColoringRule">
    <property name="text">
-    <string>New Conversation Rule…</string>
+    <string>New Coloring Rule…</string>
    </property>
    <property name="toolTip">
-    <string>Create a new coloring rule based on this conversation.</string>
+    <string>Create a new coloring rule based on this field.</string>
    </property>
   </action>
   <action name="actionViewColorizeResetColorization">

diff --git a/ui/qt/main_window_slots.cpp b/ui/qt/main_window_slots.cpp
index 17c3eba1c6b9fb69dcf3d0e79134d1424929ef4e..06abeb5ae4b84ba6440e4de8b0fd41fe42e03a5e 100644 (file)
--- a/ui/qt/main_window_slots.cpp
+++ b/ui/qt/main_window_slots.cpp
@@ -1200,12 +1200,10 @@ void MainWindow::setMenusForSelectedPacket()
     foreach (QAction *cc_action, cc_actions) {
         cc_action->setEnabled(frame_selected);
     }
-    main_ui_->actionViewColorizeNewConversationRule->setEnabled(frame_selected);
+    main_ui_->actionViewColorizeNewColoringRule->setEnabled(frame_selected);
 
     main_ui_->actionViewColorizeResetColorization->setEnabled(tmp_color_filters_used());
 
-    main_ui_->actionViewColorizeNewConversationRule->setEnabled(frame_selected);
-
     main_ui_->actionViewShowPacketInNewWindow->setEnabled(frame_selected);
     main_ui_->actionViewEditResolvedName->setEnabled(frame_selected && is_ip);
 
@@ -1242,7 +1240,7 @@ void MainWindow::setMenusForSelectedPacket()
             connect(colorize_action, SIGNAL(triggered()), this, SLOT(colorizeWithFilter()));
         }
 
-        QAction *conv_rule_action = submenu->addAction(main_ui_->actionViewColorizeNewConversationRule->text());
+        QAction *conv_rule_action = submenu->addAction(main_ui_->actionViewColorizeNewColoringRule->text());
         conv_rule_action->setData(conv_action->data());
         conv_rule_action->setEnabled(enable);
         connect(conv_rule_action, SIGNAL(triggered()), this, SLOT(colorizeWithFilter()));
@@ -1375,7 +1373,7 @@ void MainWindow::setMenusForSelectedTreeRow(field_info *fi) {
         connect(colorize_action, SIGNAL(triggered()), this, SLOT(colorizeWithFilter()));
     }
 
-    QAction *conv_rule_action = proto_tree_->colorizeMenu()->addAction(main_ui_->actionViewColorizeNewConversationRule->text());
+    QAction *conv_rule_action = proto_tree_->colorizeMenu()->addAction(main_ui_->actionViewColorizeNewColoringRule->text());
     conv_rule_action->setData(field_filter);
     conv_rule_action->setEnabled(!field_filter.isEmpty());
     connect(conv_rule_action, SIGNAL(triggered()), this, SLOT(colorizeWithFilter()));
@@ -2433,7 +2431,7 @@ void MainWindow::on_actionViewColorizeResetColorization_triggered()
     setMenusForSelectedPacket();
 }
 
-void MainWindow::on_actionViewColorizeNewConversationRule_triggered()
+void MainWindow::on_actionViewColorizeNewColoringRule_triggered()
 {
     colorizeConversation(true);
 }