Wireshark-commits: [Wireshark-commits] master-2.0 9d415ad: Qt: Hide columns after setting widths
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sat, 17 Sep 2016 20:20:09 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=9d415ad59ff3f1333ceac52de20ce836212a9560
Submitter: Stig Bjørlykke (stig@xxxxxxxxxxxxx)
Changed: branch: master-2.0
Repository: wireshark

Commits:

9d415ad by Stig Bjørlykke (stig@xxxxxxxxxxxxx):

    Qt: Hide columns after setting widths
    
    In columnsChanged() the visibility must be set after setting
    the width to avoid that setting width overwrites visibility.
    
    This should fix hidden columns displayed during first capture.
    
    Bug: 12377
    Change-Id: Idbbf36b014724970775c34b0c08803de9b006742
    Reviewed-on: https://code.wireshark.org/review/17755
    Petri-Dish: Stig Bjørlykke <stig@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Stig Bjørlykke <stig@xxxxxxxxxxxxx>
    (cherry picked from commit 6bedad69b672fdf0e67b54738edb64433938c8ae)
    Reviewed-on: https://code.wireshark.org/review/17765
    (cherry picked from commit 0f94f739fe959a83e70713b58b63be8f69c9574d)
    Reviewed-on: https://code.wireshark.org/review/17766
    

Actions performed:

    from  f768cda   Qt: Reset expert info icon when start capture failed
    adds  9d415ad   Qt: Hide columns after setting widths


Summary of changes:
 ui/qt/packet_list.cpp |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)