Populate a QTableWidget with QTreeViews

mrjj

Oh
When they become windows it means you did not give them a parent.
So do that in constructor
or via -> setParent

new QTreeView( PARENTPTR );

VRonin

@mrjj Unfortunately setCellWidget takes ownership of the widget so it can't be because of the parent thing.

very strange that even with a base QWidget does not work.

Can you try with a clean project and see if it happens the same?

mrjj

@VRonin
" setCellWidget takes ownership of the widget"
Hi, thx for adding :)
yes 100% agree,
but what else would make them windows`?

VRonin

@mrjj said in Populate a QTableWidget with QTreeViews:

but what else would make them windows`?

my only guess would be the cell(0, treeViewAdresses.at(i)) is null so since the cell does not exist the widget is not added. I'm just 10% confident on this one though

mrjj

@VRonin
I also have a 10% question ;)
The at() returns a const ref
Will it then be able to take ownership ?

VRonin

@mrjj I think the actual widget is forest[i] which should return a non-const ref.

Anyway even at() will return a const reference to a non const pointer so it can take ownership as the content of the pointer is non const, just like QWidget* const

mrjj

@VRonin
Yeah, just thinking out loud.

Well test is always the best


    for (int i = 0 ; i < 10 ; ++i) {
        QTreeView* forest = new QTreeView(ui->tableWidget);
        forest->setVisible(true);
        ui->tableWidget->setCellWidget(i, 0, forest);
    }

This puts them inside

But if you change
QTreeView* forest = new QTreeView(ui->tableWidget);
-->
QTreeView* forest = new QTreeView();

I get windows too.

So it was the parent !?!?!

runable test project
https://www.dropbox.com/s/8l2an24l7vp0x0l/treesintable.zip?dl=0

VRonin

@mrjj Big up then. typical example of

It works... why?

Probably here it's the doc's fault as I suspect "ownership" doesn't mean parent-child ownership in this case

mrjj

@VRonin
Hehe. And after that came the rule. If it works dont mess with it :)

Well normally when reparenting a widget, all win flags are stripped so it
can be inside. Maybe in this case that part is missing?
Using a QLabel just works.

mrjj

@mrjj

Update:
Suddenly it works without setting parent. ( i rebooted. )

If i move show() to be last, its perfect as it wont show as window for a brief moment

 for (int i = 0 ; i < 10 ; ++i) {
        QTreeView* forest = new QTreeView();
        ui->tableWidget->setCellWidget(i, 0, forest);
        forest->setVisible(true);
    }

This suddenly works.

So my other test must be flawed and the parent is not important as
setCellWidget does as we think.

Meh :)

Patou355

Hello guys. Thanks for your answerS.
The "error" was simple, stupid. In fact the _manageOrderedTable is an instance of a class called ManageOrderedTable which inherits from QTableWidget.
In this class, there's a method called fillAll and I called setColumnCount() from there. Bad idea.
When I call this method from upon the for loop, everything is fine. I don't understand why.
I could understand if setColumnCount() were called too late, but in this case I should have a segfault, trying to call a cell that doesn't exist.
Well, I don't really understand why but now it works...

Tank you for your help, Patrick.