diff options
author | Gerald Combs <gerald@wireshark.org> | 2022-01-02 14:41:53 -0800 |
---|---|---|
committer | Gerald Combs <gerald@wireshark.org> | 2022-01-02 14:41:53 -0800 |
commit | c6f328c6a95b34975b60bb7c2e6d4cfb156bc162 (patch) | |
tree | 6a01f13a4c2549b3eb1d37856c9273f0ad910a0a /ui | |
parent | 17e4032e81c2d601699d6fea10f0a6a8e2c486b0 (diff) |
Qt: Disable alternating table rows.
Disable QAbstractItemView's alternatingRowColors in places where we have
that set. One of Wireshark's most heavily used features is packet
colorization; we use color in packet list and detail rows to convey
information. Simple alternating color rows doesn't do that, and as my
blatant appeal to authority^W^W^W^W^WEdward Tufte points out, "Strips
are merely bureaucratic or designer chartjunk; good typography can
always organize a table, no stripes needed."
https://www.edwardtufte.com/bboard/q-and-a-fetch-msg?msg_id=0001IV
Diffstat (limited to 'ui')
-rw-r--r-- | ui/qt/bluetooth_device_dialog.ui | 5 | ||||
-rw-r--r-- | ui/qt/bluetooth_hci_summary_dialog.ui | 11 | ||||
-rw-r--r-- | ui/qt/credentials_dialog.ui | 6 | ||||
-rw-r--r-- | ui/qt/resolved_addresses_dialog.ui | 3 | ||||
-rw-r--r-- | ui/qt/sctp_all_assocs_dialog.ui | 3 | ||||
-rw-r--r-- | ui/qt/sctp_chunk_statistics_dialog.ui | 3 |
6 files changed, 6 insertions, 25 deletions
diff --git a/ui/qt/bluetooth_device_dialog.ui b/ui/qt/bluetooth_device_dialog.ui index 73fd59f4df..fcc7d09362 100644 --- a/ui/qt/bluetooth_device_dialog.ui +++ b/ui/qt/bluetooth_device_dialog.ui @@ -37,9 +37,6 @@ <property name="dragDropOverwriteMode"> <bool>false</bool> </property> - <property name="alternatingRowColors"> - <bool>true</bool> - </property> <property name="verticalScrollMode"> <enum>QAbstractItemView::ScrollPerPixel</enum> </property> @@ -256,7 +253,7 @@ <string>Mark/Unmark Row</string> </property> <property name="shortcut"> - <string>Ctrl-M</string> + <string>Ctrl+M</string> </property> </action> <action name="actionMark_Unmark_Cell"> diff --git a/ui/qt/bluetooth_hci_summary_dialog.ui b/ui/qt/bluetooth_hci_summary_dialog.ui index c2ba52a27a..c477500804 100644 --- a/ui/qt/bluetooth_hci_summary_dialog.ui +++ b/ui/qt/bluetooth_hci_summary_dialog.ui @@ -31,9 +31,6 @@ <property name="showDropIndicator" stdset="0"> <bool>false</bool> </property> - <property name="alternatingRowColors"> - <bool>true</bool> - </property> <property name="selectionMode"> <enum>QAbstractItemView::ExtendedSelection</enum> </property> @@ -67,15 +64,15 @@ <attribute name="headerCascadingSectionResizes"> <bool>false</bool> </attribute> + <attribute name="headerMinimumSectionSize"> + <number>100</number> + </attribute> <attribute name="headerDefaultSectionSize"> <number>100</number> </attribute> <attribute name="headerHighlightSections"> <bool>false</bool> </attribute> - <attribute name="headerMinimumSectionSize"> - <number>100</number> - </attribute> <attribute name="headerShowSortIndicator" stdset="0"> <bool>false</bool> </attribute> @@ -602,7 +599,7 @@ <item> <layout class="QHBoxLayout" name="horizontalLayout" stretch="0,0,0,0"> <property name="spacing"> - <number>-1</number> + <number>6</number> </property> <property name="sizeConstraint"> <enum>QLayout::SetDefaultConstraint</enum> diff --git a/ui/qt/credentials_dialog.ui b/ui/qt/credentials_dialog.ui index 16e5b3361d..fc41414b30 100644 --- a/ui/qt/credentials_dialog.ui +++ b/ui/qt/credentials_dialog.ui @@ -15,11 +15,7 @@ </property> <layout class="QVBoxLayout" name="verticalLayout"> <item> - <widget class="QTreeView" name="auths"> - <property name="alternatingRowColors"> - <bool>true</bool> - </property> - </widget> + <widget class="QTreeView" name="auths"/> </item> <item> <widget class="QDialogButtonBox" name="buttonBox"> diff --git a/ui/qt/resolved_addresses_dialog.ui b/ui/qt/resolved_addresses_dialog.ui index a1a194e9d4..c77c5cc134 100644 --- a/ui/qt/resolved_addresses_dialog.ui +++ b/ui/qt/resolved_addresses_dialog.ui @@ -43,9 +43,6 @@ <property name="editTriggers"> <set>QAbstractItemView::NoEditTriggers</set> </property> - <property name="alternatingRowColors"> - <bool>true</bool> - </property> <property name="sortingEnabled"> <bool>true</bool> </property> diff --git a/ui/qt/sctp_all_assocs_dialog.ui b/ui/qt/sctp_all_assocs_dialog.ui index d697b4b614..d25b1ca875 100644 --- a/ui/qt/sctp_all_assocs_dialog.ui +++ b/ui/qt/sctp_all_assocs_dialog.ui @@ -25,9 +25,6 @@ <property name="dragDropOverwriteMode"> <bool>false</bool> </property> - <property name="alternatingRowColors"> - <bool>true</bool> - </property> <property name="selectionMode"> <enum>QAbstractItemView::SingleSelection</enum> </property> diff --git a/ui/qt/sctp_chunk_statistics_dialog.ui b/ui/qt/sctp_chunk_statistics_dialog.ui index 26d1186b00..0a12082301 100644 --- a/ui/qt/sctp_chunk_statistics_dialog.ui +++ b/ui/qt/sctp_chunk_statistics_dialog.ui @@ -47,9 +47,6 @@ <property name="editTriggers"> <set>QAbstractItemView::NoEditTriggers</set> </property> - <property name="alternatingRowColors"> - <bool>true</bool> - </property> <property name="selectionMode"> <enum>QAbstractItemView::SingleSelection</enum> </property> |