diff options
author | Gerald Combs <gerald@wireshark.org> | 2015-07-09 16:47:11 -0700 |
---|---|---|
committer | Gerald Combs <gerald@wireshark.org> | 2015-07-13 02:36:41 +0000 |
commit | c305f08921feb30f5411bc8f8e0b01ebf1cfbe2e (patch) | |
tree | 5cd54f8b6c1fc7efecf347f9a389b680eaf18cdb /ui/qt/main_window.cpp | |
parent | 494457d3b55630595909d4db883e93f2d3f57986 (diff) | |
download | wireshark-c305f08921feb30f5411bc8f8e0b01ebf1cfbe2e.tar.gz |
PacketList column fixes.
Add a columnsChanged slot to PacketList and move the column update code
from redrawVisiblePackets there. Make sure we call
packet_list_model_->recreateVisibleRows, which should fix the behavior
described in bug 11324. Call columnsChanged when we, uh, change columns.
Add a sectionMoved slot to handle column reordering.
Don't rebuild the column list when we update the widgets in the column
preferences frame. Do enable and disable the "remove" button as needed.
Try to keep the user from removing all of the columns in both the packet
list and column preferences.
Left as an exercise for the reader: The GTK+ UI also fails when you
remove all of the columns via the preferences:
packet_list.c:377:packet_list_sort_column: assertion failed: (col)
Bug: 11324
Change-Id: Id58cf98e42cbda9aa2fc370ea06b8bcc6098c8ca
Reviewed-on: https://code.wireshark.org/review/9591
Petri-Dish: Gerald Combs <gerald@wireshark.org>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Gerald Combs <gerald@wireshark.org>
Diffstat (limited to 'ui/qt/main_window.cpp')
-rw-r--r-- | ui/qt/main_window.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ui/qt/main_window.cpp b/ui/qt/main_window.cpp index 085139cc35..bd60aef7d7 100644 --- a/ui/qt/main_window.cpp +++ b/ui/qt/main_window.cpp @@ -408,7 +408,7 @@ MainWindow::MainWindow(QWidget *parent) : connect(wsApp, SIGNAL(recentFilesRead()), packet_list_, SLOT(applyRecentColumnWidths())); connect(wsApp, SIGNAL(columnsChanged()), - packet_list_, SLOT(redrawVisiblePackets())); + packet_list_, SLOT(columnsChanged())); connect(wsApp, SIGNAL(recentFilesRead()), this, SLOT(applyRecentPaneGeometry())); connect(wsApp, SIGNAL(packetDissectionChanged()), |