mattcasters opened a new pull request, #7905:
URL: https://github.com/apache/hop/pull/7905

   ## What
   
   Fixes intermittent truncation of the transform/action list in the Hop Web 
ContextDialog: scrolling could stop mid-list (e.g. after Transform, with 
Utility header empty below).
   
   ## Why
   
   Content height is only known after the first paint. `updateVerticalBar()` 
was called from `filter()` / resize *before* paint, while `onPaint()` resized 
the canvas without refreshing the scrollbar. On RAP that left a stale scroll 
range; search or reopen often fixed it by re-syncing.
   
   Regression relative to the original ScrolledComposite work (PR #531): 
post-`setSize` scrollbar update was removed in HOP-2857.
   
   ## How
   
   - After paint measures height: `setSize`, `setMinWidth`/`setMinHeight`, then 
`updateVerticalBar()`
   - Enable `setExpandVertical(true)` (standard ScrolledComposite/RAP pattern)
   - Drive scrollbar max from measured `totalContentHeight` (not pre-paint 
10x10 bounds)
   - On shell resize, force remeasure so column wrap updates height
   
   ## Testing
   
   - [x] `./mvnw -pl ui -am compile -DskipTests`
   - [x] `./mvnw -pl ui spotless:check`
   - Manual: Hop Web full transform list — scroll past Transform into Utility 
and remaining categories; search/clear and resize still OK
   
   Fixes #7868


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to