On Wed, 2 Aug 2023 08:55:19 GMT, Tejesh R <[email protected]> wrote: >> The header border uses `g.drawLine` whereas the JTable data grid lines uses >> `SwingUtilities2.drawVLine` and `SwingUtilities2.drawHLine` to draw >> horizontal and vertical lines. The SwingUtilities2 uses `Graphics.fillRect` >> which contributes to the difference between the position of these two lines >> which happens/visible at higher ui scaling (difference in alignment between >> vertical lines of these two). The fix propose to use the same methods for >> metal L&F of JTable header border paint. >> CI testing shows green. >> >>  > > Tejesh R has updated the pull request incrementally with one additional > commit since the last revision: > > Review fix
Changes requested by aivanov (Reviewer). I'll post [this comment](https://github.com/openjdk/jdk/pull/14464#pullrequestreview-1558834675) here too: > I still prefer the previous version where the exception was thrown directly. > > In fact, you don't create real UI, the test doesn't depend on processing > events — the entire test can be run on main thread *safely*. Get rid of > `invokeAndWait` altogether? ------------- PR Review: https://git.openjdk.org/jdk/pull/14766#pullrequestreview-1558832421 PR Comment: https://git.openjdk.org/jdk/pull/14766#issuecomment-1662525912
