Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v8]

2025-03-04 Thread Andy Goryachev
On Thu, 30 Jan 2025 23:02:36 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v8]

2025-03-04 Thread Andy Goryachev
On Thu, 30 Jan 2025 23:02:36 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v8]

2025-03-04 Thread Ambarish Rapte
On Thu, 30 Jan 2025 23:02:36 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v8]

2025-03-01 Thread Kevin Rushforth
On Thu, 30 Jan 2025 23:02:36 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v8]

2025-01-31 Thread Kevin Rushforth
On Thu, 30 Jan 2025 23:02:36 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v8]

2025-01-30 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > Removed earlier manual tester in favor of the monkey tester. > > It is important to note that even though the constraints are given by the > us

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v6]

2025-01-29 Thread Kevin Rushforth
On Mon, 27 Jan 2025 15:41:55 GMT, Andy Goryachev wrote: >>> we will violate the max constraint if we snap to a larger value. >> >> Well, it's inherently an over-constrained problem. Conceptually, though, >> this does not violate the constraint. When snap-to-pixel is enabled, all >> values, inc

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v7]

2025-01-29 Thread Kevin Rushforth
On Mon, 27 Jan 2025 20:22:30 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v7]

2025-01-29 Thread Kevin Rushforth
On Wed, 5 Jul 2023 16:53:41 GMT, Andy Goryachev wrote: >> modules/javafx.controls/src/main/java/com/sun/javafx/scene/control/ResizeHelper.java >> line 77: >> >>> 75: double cmin = snapCeil(c.getMinWidth()); // always >>> honor min width! >>> 76: double cmax = sn

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v7]

2025-01-29 Thread Andy Goryachev
On Mon, 27 Jan 2025 20:22:30 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v7]

2025-01-27 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > Removed earlier manual tester in favor of the monkey tester. > > It is important to note that even though the constraints are given by the > us

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v6]

2025-01-27 Thread Andy Goryachev
On Wed, 15 Jan 2025 15:41:23 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v6]

2025-01-27 Thread Andy Goryachev
On Wed, 15 Jan 2025 15:41:23 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v6]

2025-01-27 Thread Andy Goryachev
On Sat, 25 Jan 2025 00:37:28 GMT, Kevin Rushforth wrote: >> we will violate the max constraint if we snap to a larger value. >> >> so for the max constraint, we should `snapInnerSpace()` which calls the >> private (?) `Region.snapPortionX()`. > >> we will violate the max constraint if we snap

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v6]

2025-01-24 Thread Kevin Rushforth
On Sat, 25 Jan 2025 00:07:55 GMT, Andy Goryachev wrote: > we will violate the max constraint if we snap to a larger value. Well, it's inherently an over-constrained problem. Conceptually, though, this does not violate the constraint. When snap-to-pixel is enabled, all values, including min and

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v6]

2025-01-24 Thread Andy Goryachev
On Fri, 24 Jan 2025 19:38:24 GMT, Kevin Rushforth wrote: >> Andy Goryachev has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 39 commits: >> >> - Merge branch 'master' into 8299753.resize >> - Merge remote-tracking branch 'origin/m

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v6]

2025-01-24 Thread Kevin Rushforth
On Wed, 15 Jan 2025 15:41:23 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v6]

2025-01-15 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > Removed earlier manual tester in favor of the monkey tester. > > It is important to note that even though the constraints are given by the > us

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v5]

2025-01-15 Thread Andy Goryachev
On Mon, 4 Nov 2024 17:21:57 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v5]

2024-12-17 Thread Andy Goryachev
On Mon, 4 Nov 2024 17:21:57 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v5]

2024-11-19 Thread Andy Goryachev
On Mon, 4 Nov 2024 17:21:57 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v5]

2024-11-04 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > Removed earlier manual tester in favor of the monkey tester. > > It is important to note that even though the constraints are given by the > us

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v4]

2024-10-22 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > Removed earlier manual tester in favor of the monkey tester. > > It is important to note that even though the constraints are given by the > us

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v3]

2024-10-22 Thread Andy Goryachev
On Thu, 22 Aug 2024 22:39:25 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v3]

2024-09-23 Thread Andy Goryachev
On Thu, 22 Aug 2024 22:39:25 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale

2024-08-22 Thread Andy Goryachev
On Thu, 22 Aug 2024 15:29:45 GMT, Kevin Rushforth wrote: > snapInnerSpace method. The main idea is to handle the case when computing snapped coordinates within the unsnapped container. Granted, this is somewhat unusual case, so I am ok with using simple rounding here. FYI, I've added Page ->

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v3]

2024-08-22 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > Removed earlier manual tester in favor of the monkey tester. > > It is important to note that even though the constraints are given by the > us

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v2]

2024-08-22 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > Removed earlier manual tester in favor of the monkey tester. > > It is important to note that even though the constraints are given by the > us

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale

2024-08-22 Thread Kevin Rushforth
On Thu, 15 Jun 2023 19:38:00 GMT, Andy Goryachev wrote: > Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > Removed earlier manual tester in favor of the monkey tester. > > It is important to no

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale

2024-08-22 Thread Andy Goryachev
On Thu, 15 Jun 2023 19:38:00 GMT, Andy Goryachev wrote: > Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > Removed earlier manual tester in favor of the monkey tester. > > It is important to no

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale

2024-07-24 Thread Andy Goryachev
On Thu, 15 Jun 2023 19:38:00 GMT, Andy Goryachev wrote: > Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > Removed earlier manual tester in favor of the monkey tester. > > It is important to no

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale

2024-06-27 Thread Andy Goryachev
On Wed, 5 Jul 2023 20:31:09 GMT, Michael Strauß wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale

2024-06-27 Thread Andy Goryachev
On Thu, 6 Jul 2023 00:23:09 GMT, Michael Strauß wrote: >> I agree with John that a layout algorithm that uses incremental calculations >> will always be flawed in principle. The correct approach is to store the >> initial configuration, and then for each configuration change, go back to >> the

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale

2024-06-27 Thread Andy Goryachev
On Fri, 30 Jun 2023 05:38:35 GMT, Karthik P K wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important to

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale

2024-06-27 Thread Michael Strauß
On Wed, 5 Jul 2023 20:31:09 GMT, Michael Strauß wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale

2024-06-27 Thread Michael Strauß
On Thu, 15 Jun 2023 19:38:00 GMT, Andy Goryachev wrote: > Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > Removed earlier manual tester in favor of the monkey tester. > > It is important to no

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale

2024-06-27 Thread Andy Goryachev
On Thu, 15 Jun 2023 19:38:00 GMT, Andy Goryachev wrote: > Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > Removed earlier manual tester in favor of the monkey tester. > > It is important to no

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale

2024-06-27 Thread John Hendrikx
On Thu, 15 Jun 2023 19:38:00 GMT, Andy Goryachev wrote: > Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > Removed earlier manual tester in favor of the monkey tester. > > It is important to no

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale

2024-06-27 Thread Andy Goryachev
On Sun, 2 Jul 2023 12:53:05 GMT, John Hendrikx wrote: > My observation is that this algorithm seems unable to provide a proper user > resizing experience as it seems to discard important information it would > need to do so. please elaborate, or point to a specific problem. It is entirely pos

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale

2024-06-27 Thread John Hendrikx
On Tue, 27 Jun 2023 18:23:29 GMT, Andy Goryachev wrote: > @hjohn would you mind taking a look at this? using the ideas from your > SpaceDistributor Will take a look this weekend - PR Comment: https://git.openjdk.org/jfx/pull/1156#issuecomment-1614422355

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale

2024-06-27 Thread Karthik P K
On Thu, 15 Jun 2023 19:38:00 GMT, Andy Goryachev wrote: > Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > Removed earlier manual tester in favor of the monkey tester. > > It is important to no

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v15]

2024-06-26 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > Removed earlier manual tester in favor of the monkey tester. > > It is important to note that even though the constraints are given by the > us

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v14]

2024-06-26 Thread Andy Goryachev
On Mon, 10 Jun 2024 15:06:30 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v14]

2024-06-10 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > Removed earlier manual tester in favor of the monkey tester. > > It is important to note that even though the constraints are given by the > us

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v13]

2024-06-10 Thread Andy Goryachev
On Fri, 12 Apr 2024 22:47:21 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v13]

2024-05-13 Thread Andy Goryachev
On Fri, 12 Apr 2024 22:47:21 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> Removed earlier manual tester in favor of the monkey tester. >> >> It is important

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v13]

2024-04-12 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > Removed earlier manual tester in favor of the monkey tester. > > It is important to note that even though the constraints are given by the > us

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v12]

2024-04-02 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > Removed earlier manual tester in favor of the monkey tester. > > It is important to note that even though the constraints are given by the > us

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v11]

2024-03-07 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > It is important to note that even though the constraints are given by the > user in unsnapped coordinates, they are converted to snapped values,

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v10]

2024-02-29 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > It is important to note that even though the constraints are given by the > user in unsnapped coordinates, they are converted to snapped values,

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v9]

2024-02-06 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > It is important to note that even though the constraints are given by the > user in unsnapped coordinates, they are converted to snapped values,

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v8]

2023-12-12 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > It is important to note that even though the constraints are given by the > user in unsnapped coordinates, they are converted to snapped values,

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v7]

2023-12-09 Thread Karthik P K
On Thu, 2 Nov 2023 15:31:40 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> It is important to note that even though the constraints are given by the >> user i

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v2]

2023-11-30 Thread Andy Goryachev
On Fri, 30 Jun 2023 05:38:35 GMT, Karthik P K wrote: >> Andy Goryachev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> review comments > > Left a inline comment. > Otherwise looks good to me @karthikpandelu @hjohn @kevinrushforth could

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v7]

2023-11-02 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > It is important to note that even though the constraints are given by the > user in unsnapped coordinates, they are converted to snapped values,

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v6]

2023-10-19 Thread Andy Goryachev
On Thu, 21 Sep 2023 14:50:39 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> It is important to note that even though the constraints are given by the >> user

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v6]

2023-09-21 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > It is important to note that even though the constraints are given by the > user in unsnapped coordinates, they are converted to snapped values,

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v5]

2023-09-11 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > It is important to note that even though the constraints are given by the > user in unsnapped coordinates, they are converted to snapped values,

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v4]

2023-08-03 Thread Andy Goryachev
On Wed, 5 Jul 2023 19:30:21 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> It is important to note that even though the constraints are given by the >> user i

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v4]

2023-07-06 Thread Andy Goryachev
On Thu, 6 Jul 2023 00:23:09 GMT, Michael Strauß wrote: >> I agree with John that a layout algorithm that uses incremental calculations >> will always be flawed in principle. The correct approach is to store the >> initial configuration, and then for each configuration change, go back to >> the

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v4]

2023-07-05 Thread Michael Strauß
On Wed, 5 Jul 2023 20:31:09 GMT, Michael Strauß wrote: >> Andy Goryachev has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains ten additional >> commi

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v4]

2023-07-05 Thread Andy Goryachev
On Wed, 5 Jul 2023 20:31:09 GMT, Michael Strauß wrote: >> Andy Goryachev has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains ten additional >> commi

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v4]

2023-07-05 Thread Andy Goryachev
On Wed, 5 Jul 2023 19:45:39 GMT, John Hendrikx wrote: >> Andy Goryachev has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains ten additional >> commit

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v4]

2023-07-05 Thread John Hendrikx
On Wed, 5 Jul 2023 19:30:21 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> It is important to note that even though the constraints are given by the >> user i

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v4]

2023-07-05 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > It is important to note that even though the constraints are given by the > user in unsnapped coordinates, they are converted to snapped values,

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v3]

2023-07-05 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > It is important to note that even though the constraints are given by the > user in unsnapped coordinates, they are converted to snapped values,

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v2]

2023-07-05 Thread Andy Goryachev
On Sun, 2 Jul 2023 12:40:17 GMT, John Hendrikx wrote: >> Andy Goryachev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> review comments > > modules/javafx.controls/src/main/java/com/sun/javafx/scene/control/ResizeHelper.java > line 125:

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v2]

2023-07-05 Thread Andy Goryachev
On Sun, 2 Jul 2023 12:11:56 GMT, John Hendrikx wrote: >> Andy Goryachev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> review comments > > modules/javafx.controls/src/main/java/com/sun/javafx/scene/control/ResizeHelper.java > line 58:

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v2]

2023-07-05 Thread Andy Goryachev
On Sun, 2 Jul 2023 12:53:05 GMT, John Hendrikx wrote: > My observation is that this algorithm seems unable to provide a proper user > resizing experience as it seems to discard important information it would > need to do so. please elaborate, or point to a specific problem. It is entirely pos

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v2]

2023-07-02 Thread John Hendrikx
On Fri, 23 Jun 2023 15:45:21 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> It is important to note that even though the constraints are given by the >> user

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v2]

2023-06-30 Thread Andy Goryachev
On Fri, 30 Jun 2023 05:38:03 GMT, Karthik P K wrote: >> Andy Goryachev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> review comments > > modules/javafx.controls/src/main/java/com/sun/javafx/scene/control/ResizeHelper.java > line 112:

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v2]

2023-06-30 Thread John Hendrikx
On Tue, 27 Jun 2023 18:23:29 GMT, Andy Goryachev wrote: > @hjohn would you mind taking a look at this? using the ideas from your > SpaceDistributor Will take a look this weekend - PR Comment: https://git.openjdk.org/jfx/pull/1156#issuecomment-1614422355

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v2]

2023-06-29 Thread Karthik P K
On Fri, 23 Jun 2023 15:45:21 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> It is important to note that even though the constraints are given by the >> user

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v2]

2023-06-27 Thread Andy Goryachev
On Fri, 23 Jun 2023 15:45:21 GMT, Andy Goryachev wrote: >> Modified the resize algorithm to work well with fractional scale, thanks for >> deeper understanding of the problem thanks to @hjohn and @mstr2 . >> >> It is important to note that even though the constraints are given by the >> user

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale [v2]

2023-06-23 Thread Andy Goryachev
> Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > It is important to note that even though the constraints are given by the > user in unsnapped coordinates, they are converted to snapped values,

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale

2023-06-23 Thread Andy Goryachev
On Thu, 15 Jun 2023 19:38:00 GMT, Andy Goryachev wrote: > Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > It is important to note that even though the constraints are given by the > user in un

Re: RFR: 8299753: Tree/TableView: Column Resizing With Fractional Scale

2023-06-23 Thread Karthik P K
On Thu, 15 Jun 2023 19:38:00 GMT, Andy Goryachev wrote: > Modified the resize algorithm to work well with fractional scale, thanks for > deeper understanding of the problem thanks to @hjohn and @mstr2 . > > It is important to note that even though the constraints are given by the > user in un