On Wed, 23 Sep 2020 06:55:42 GMT, Ambarish Rapte <ara...@openjdk.org> wrote:

>> Jeanette Winzenburg has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   widened scope of fix to address review comment
>
> modules/javafx.controls/src/main/java/javafx/scene/control/skin/TabPaneSkin.java
>  line 855:
> 
>> 853:                         } else {
>> 854:                         }
>> 855:                         validateScrollOffset();
> 
> This seems to be an unintended change. If so please revert.

good eye :) Forgot to remove the empty else block.

Validate must be called always if the tabs are not fitting: needed if the last 
tab (in the list) is visible and we
remove tabs from the end. Without, the now last tab is not kept glued to the 
trailing edge. There are two tests
covering the corner case (testRemoveXXLast) which fail if we don't.

Question is if that should be mentioned in a code comment? Personally, I tend 
to just delete the empty block.

-------------

PR: https://git.openjdk.java.net/jfx/pull/300

Reply via email to