[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-2d-dev
Subject:    Integrated: 8259550: The content of the print out displayed incomplete with the NimbusLAF
From:       Tejesh R <tr () openjdk ! org>
Date:       2024-01-31 5:29:06
Message-ID: U7RDTM-YGQrsNQ-rDHitL3duk9yJaQW102A1q27jfJ8=.ce36746f-efda-4ba0-bf84-cf08abf5b5b0 () github ! com
[Download RAW message or body]

On Mon, 11 Dec 2023 11:25:34 GMT, Tejesh R <tr@openjdk.org> wrote:

> The issue is w.r.t to way that Synth prints the table. In Synth, the printing of \
> Table happens column wise by painting blocks of columns which is due to the fact \
> that Graphics used which is `ProxyGrpahics2D` (which get selected in \
> `RasterPrinterJob `class, otherwise it would be `WPathGraphics` for other Look and \
> feel).  Since the printing happens in block wise and the columns where the x \
> position is greater than the `table.bounds` doesn't gets painted because of the \
> intersection check condition `!bounds.intersects(clip)`. Hence only droplines are \
> painted instead of cells and gridlines.  I couldn't find any reason for adding the \
> intersection check here and when removed it didn't cause any regression too, rather \
> it solves the issue and actually made sense too. In other Look and Feel its not \
> required to apply the fix (in `BasicTableUI `class) because its not affecting \
> anything yet since the painting happens for whole table. Hence proposing the fix \
> for only SynthTableUI class. \
> ([PrintManualTest_FitWidthMultiple.java](https://github.com/openjdk/jdk/blob/master/test/jdk/javax/swing/JTable/PrintManualTest_FitWidthMultiple.java)) \
> test is been used for testing.

This pull request has now been integrated.

Changeset: 577de17d
Author:    Tejesh R <tr@openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/577de17d24e83c55ab10a5794f381243a298fc68
                
Stats:     39 lines in 2 files changed: 8 ins; 3 del; 28 mod

8259550: The content of the print out displayed incomplete with the NimbusLAF

Reviewed-by: dnguyen, psadhukhan, abhiscxk

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

PR: https://git.openjdk.org/jdk/pull/17053


[prev in list] [next in list] [prev in thread] [next in thread]