Wireshark-commits: [Wireshark-commits] master 1e48766: Qt: Add option to back white line separator
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Mon, 20 Jul 2015 21:21:38 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=1e487664bc2a902dbfd8d2d3ef9ddcd09a4b3d12
Submitter: Michal Labedzki (michal.labedzki@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

1e48766 by Michal Labedzki (michal.labedzki@xxxxxxxxx):

    Qt: Add option to back white line separator for packet list items
    
    This uses one line (on bottom) of items and makes it white.
    Seen in Wireshark on some configurations of GTK 2 on Linux,
    so backports it to Qt for people who like it.
    
    In my opinion it helps if you use packet list background colors
    for frames and you have a lot of following frames with
    the same background color.
    
    Bug: 10954
    Change-Id: Id8f58520d7224db4eb8181bcc04febd7416a8578
    Reviewed-on: https://code.wireshark.org/review/7293
    Petri-Dish: Michal Labedzki <michal.labedzki@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Michal Labedzki <michal.labedzki@xxxxxxxxx>
    

Actions performed:

    from  7e13cac   Try to minimize sorting overhead.
    adds  1e48766   Qt: Add option to back white line separator for packet list items


Summary of changes:
 epan/prefs.c                       |    8 ++++++++
 epan/prefs.h                       |    2 +-
 ui/qt/layout_preferences_frame.cpp |   10 ++++++++++
 ui/qt/layout_preferences_frame.h   |    2 ++
 ui/qt/layout_preferences_frame.ui  |   25 ++++++++++++++++++++++++-
 ui/qt/packet_list.cpp              |   12 ++++++++++++
 ui/qt/packet_list.h                |    2 ++
 7 files changed, 59 insertions(+), 2 deletions(-)