Fix header content size when sorting is disabled

Authored by astan on Feb 2 2017, 5:03 PM.

Description

Fix header content size when sorting is disabled

Summary: This fixes the same problem as described in D4406 for Breeze.

Test Plan:

#include <QApplication>
#include <QTreeView>
#include <QStandardItemModel>
#include <QStandardItem>

int main(int argc, char *argv[])
{
    QApplication app(argc, argv);

    // Test model
    QStandardItemModel model(3, 2);
    model.setHorizontalHeaderLabels({ "Header 1", "Header 2" });
    for (int row = 0; row < 3; ++row) {
        for (int column = 0; column < 2; ++column) {
            model.setItem(row, column, new QStandardItem("Foo"));
        }
    }

    // View with sorting disabled
    QTreeView view;
    view.setWindowTitle("Sorting Disabled");
    view.setModel(&model);
    view.show();
    view.resizeColumnToContents(0);

    // View with sorting enabled
    QTreeView viewWithSorting;
    viewWithSorting.setWindowTitle("Sorting Enabled");
    viewWithSorting.setModel(&model);
    viewWithSorting.setSortingEnabled(true);
    viewWithSorting.show();
    viewWithSorting.resizeColumnToContents(0);

    return app.exec();
}

Before the fix:

After the fix:

Reviewers: Breeze, cfeck

Reviewed By: cfeck

Subscribers: plasma-devel

Tags: Plasma

Differential Revision: https://phabricator.kde.org/D4410

Details

Committed
astanFeb 3 2017, 12:58 PM
Reviewer
cfeck
Differential Revision
D4410: Fix header content size when sorting is disabled
Parents
R113:7db771ad5c4b: SVN_SILENT made messages (.desktop file) - always resolve ours
Branches
Unknown
Tags
Unknown