On Tue, 28 Apr 2020 09:38:56 GMT, Jeanette Winzenburg <[email protected]> wrote:
>> The issue is that ChoiceBoxSkin >> a) doesn't update the text of the label if the value is not contained in the >> items >> b) doesn't respect converter for label text >> >> Fixed by >> - listening to value changes to update the label >> - removing ad-hoc updates (not needed), added update on converter change >> - passing all label updates through converter >> >> Added test for text updates that failed before the fix and pass after (note: >> there were no tests for the display text, >> so for coveragy, contains also tests that passed before as well as after) > > Jeanette Winzenburg has updated the pull request incrementally with one > additional commit since the last revision: > > fixed copyright year Marked as reviewed by arapte (Reviewer). ------------- PR: https://git.openjdk.java.net/jfx/pull/191
