[PIVOT-237] Possible bug(s) in TerraSplitPaneSkin

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.3
• Fix Version/s: 1.3
• Component/s: wtk
• Assignee: Niclas Hedhman
• Reporter: Gregory K. Brown
• Created: 21/Aug/09 12:18
• Updated: 25/Aug/09 19:30
• Resolved: 25/Aug/09 19:30

Description

In TerraSplitPaneSkin, I am wondering if line 505 should read:

int minimumChildSize = splitPane.getTop().getMinimumPreferredHeight();

rather than:

int minimumChildSize = splitPane.getBottom().getMinimumPreferredHeight();

I'm only asking because, as written, it doesn't have parity with lines 498 and 500 - should it?

Also, it seems like we might want to gracefully handle null values for top/left/bottom/right here. Currently, if any of them are undefined, we'll get an NPE.