Bug #1235: Bug #981 is triggered in 3.2.1 - Wt - Redmine

Actions

Copy link

Bug #1235

closed

Bug #981 is triggered in 3.2.1

Added by Rob Van Dyck over 12 years ago. Updated over 12 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Koen Deforche

Target version:

3.2.2

Start date:

04/12/2012

Due date:

% Done:

Estimated time:

Description

Hi,

It appears Bug #981 is triggered in the latest release [3.2.1].

Kind regards,

Rob.

Files

paint_test_widget.cpp (2.24 KB) paint_test_widget.cpp

Koen Deforche, 04/17/2012 12:01 PM

Actions

Copy link

Updated by Koen Deforche over 12 years ago

File paint_test_widget.cpp paint_test_widget.cpp added
Status changed from New to InProgress
Assignee set to Koen Deforche
Target version set to 3.2.2

Hey Rob,

I'm not sure anymore what to think of this test case though...

I've modified it to remove some of the things in it that really aren't supposed to be:

if you add a painted widget to a layout, you may not call resize() on it, since this overrides the size set by the layout manager (making it not react to the layout manager
the root() contains a container widget with in it a layout --- this layout will size the contents according to the height of this root() widget, but this root() widget has no height set, which will take a minimal height (0), and then the painting code is never triggered.

The attached modified test case works kind of, although I still have doubts about it.

In any case, we are currently looking at a big overhaul of the layout managers, removing many of the annoyances and making them more robust (and specify better how they should be behave). I've pushed some small fixes that make this (revised) test case work again with latest git but hopefully this version will never make it to the next release and is being replaced by something better altogether.

Regards,

koen

Actions

Copy link

Updated by Rob Van Dyck over 12 years ago

Hi Koen,

Sorry for the late update, I've just returned from a holiday. Thanx for looking at this. It might be a good idea to add the first pointer to the docs of WPaintedWidget. I could not find it there, and actually started from copy pasting the info there.

After your pointers I worked around the original issue I had. I'm looking forward to the next version with the new layout managers, because I found the layout widgets complicated to use.

Kind regards,

Rob

Actions

Copy link