Re: RFR: 8090123: Items are no longer visible when collection is changed [v6]

2023-03-07 Thread Karthik P K
On Tue, 7 Mar 2023 14:24:20 GMT, Kevin Rushforth wrote: >> Noticed a minor behavior issue, on Mac with multiple monitors. The >> secondary monitor (scale=1) is positioned above the primary retina screen >> (scale=2). When showing a popup in the case of 100 elements, the down arrow >> at the

Re: RFR: 8090123: Items are no longer visible when collection is changed [v6]

2023-03-07 Thread Kevin Rushforth
On Thu, 23 Feb 2023 17:48:39 GMT, Andy Goryachev wrote: >> Karthik P K has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Address review comments > > Noticed a minor behavior issue, on Mac with multiple monitors. The secondary > monitor

Re: RFR: 8090123: Items are no longer visible when collection is changed [v6]

2023-03-02 Thread Karthik P K
On Wed, 1 Mar 2023 13:59:41 GMT, Kevin Rushforth wrote: >> Karthik P K has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Address review comments > >

Re: RFR: 8090123: Items are no longer visible when collection is changed [v6]

2023-03-02 Thread Karthik P K
On Wed, 1 Mar 2023 13:57:46 GMT, Kevin Rushforth wrote: >> Karthik P K has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Address review comments > >

Re: RFR: 8090123: Items are no longer visible when collection is changed [v6]

2023-03-01 Thread Kevin Rushforth
On Thu, 23 Feb 2023 05:21:37 GMT, Karthik P K wrote: >> When a large number of items were scrolled in the `ChoiceBox`, the scrolled >> offset was carried forward when the list is replaced with small number of >> items. Hence the scroll up arrow was displayed with empty popup. >> >> Changed

Re: RFR: 8090123: Items are no longer visible when collection is changed [v6]

2023-02-23 Thread Karthik P K
On Thu, 23 Feb 2023 17:48:39 GMT, Andy Goryachev wrote: > Noticed a minor behavior issue, on Mac with multiple monitors. The secondary > monitor (scale=1) is positioned above the primary retina screen (scale=2). > When showing a popup in the case of 100 elements, the down arrow at the >

Re: RFR: 8090123: Items are no longer visible when collection is changed [v6]

2023-02-23 Thread Andy Goryachev
On Thu, 23 Feb 2023 05:21:37 GMT, Karthik P K wrote: >> When a large number of items were scrolled in the `ChoiceBox`, the scrolled >> offset was carried forward when the list is replaced with small number of >> items. Hence the scroll up arrow was displayed with empty popup. >> >> Changed

Re: RFR: 8090123: Items are no longer visible when collection is changed [v6]

2023-02-22 Thread Karthik P K
> When a large number of items were scrolled in the `ChoiceBox`, the scrolled > offset was carried forward when the list is replaced with small number of > items. Hence the scroll up arrow was displayed with empty popup. > > Changed code to scroll to top before popup display when content height