BTextView does not properly handle wrapped lines

[humdinger]

This is hrev41650 pre-alpha3.

When you invoke the TeamMonitor (CTRL+ALT+DEL) for the first time, and click on an entry in the list of teams, the scroll view height increases with a nice little jump.

[diver]

Layout is wrong until you select any process. I think it got broken during TeamMonitor localization.

[diver]

hrev41243/hrev41250 are possible candidates.

[leavengood]

Taking ownership since I've been working on Team Monitor lately.

[pulkomandy]

hrev41250 seems to be a likely candidate. It introduces the AllShowingTextView which sets its min size when FrameResized is called.

I see two possible problems with this:

• The size could be set earlier, as soon as SetText is called. This may avoid a resize > set min size > try to resize again path, but more important is that the text can be changed at any time, and there is no guarantee that FrameResized will be called after it does.
• Only the minimal size is set, and that doesn't prevent the view from growing bigger than it. As a result, the bottom part of the window, which is a BCardLayout, is resized only when the other view added to it (the one with the app icon and name) is shown, this one fully constraints the view size and fixes the layout.

[pulkomandy]

Fixed in hrev48529.

[diver]

This is how it looks now:

[vidrep]

Team monitor window still looks the same as shown above in latest build hrev49000.

[Janus]

(In #12143) This is a duplicate of #7560.

[waddlesplash]

It's actually a BTextView layout bug: try resizing the window horizontally, you'll see that the BTextView will grow horizontally and collapse vertically, and then the buttons will be fully visible.

[humdinger]

(In #13479) Thanks for the ticket, but as discussed ​at the forums, the issue is already tracked with #7560.

[axeld]

It's not a BTextView bug, it's an API inconvenience, you'll have to call BWindow::ResizeToPreferred().

However, when I add this (to BootPromptWindow.cpp, line 259), the window size is only sometimes correct, but mostly way too large. I haven't had the time to investigate, but this is definitely a bug somewhere (and that might even be located in BTextView, after all).

[pulkomandy]

Fixed TeamMonitor in hrev51582 by setting various explicit constraints to not get a too small window. Also made similar changes to FirstBootPrompt

In both cases the issue is similar: nothing forces the window to be very wide, so ResizeToPreferred ends up asking the BTextView a HeightForWidth for a very small width (52px in my case). This causes the text view to lay out one word per line and return a very large height.

By constraining the window width to a sane minimal size, we can avoid the problem. So I now agree with Axel, there isn't really a bug here and we just need to be more careful with the way we lay things out in windows.

[axeld]

Great, makes sense! Thanks for finally getting to the bottom of this!