Re: RFR: 5108458: JTable does not properly layout its content [v13]

2023-12-04 Thread Alexey Ivanov
On Sat, 2 Dec 2023 20:55:30 GMT, Andrey Turbanov wrote: >> Tejesh R has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Review fix > > src/java.desktop/share/classes/javax/swing/JTable.java line 3016: > >> 3014: // adjust the x

Re: RFR: 5108458: JTable does not properly layout its content [v13]

2023-12-02 Thread Andrey Turbanov
On Fri, 1 Dec 2023 09:26:41 GMT, Tejesh R wrote: >> Table contents does not follow right-left Orientation when Max width of >> columns are set. This is due to not considering the offset in `x position` >> while painting table grid and table cell. The fix handles the offset and >> adjust the x

Re: RFR: 5108458: JTable does not properly layout its content [v13]

2023-12-01 Thread Damon Nguyen
On Fri, 1 Dec 2023 09:26:41 GMT, Tejesh R wrote: >> Table contents does not follow right-left Orientation when Max width of >> columns are set. This is due to not considering the offset in `x position` >> while painting table grid and table cell. The fix handles the offset and >> adjust the x

Re: RFR: 5108458: JTable does not properly layout its content [v13]

2023-12-01 Thread Alexey Ivanov
On Fri, 1 Dec 2023 09:26:41 GMT, Tejesh R wrote: >> Table contents does not follow right-left Orientation when Max width of >> columns are set. This is due to not considering the offset in `x position` >> while painting table grid and table cell. The fix handles the offset and >> adjust the x

Re: RFR: 5108458: JTable does not properly layout its content [v13]

2023-12-01 Thread Tejesh R
> Table contents does not follow right-left Orientation when Max width of > columns are set. This is due to not considering the offset in `x position` > while painting table grid and table cell. The fix handles the offset and > adjust the x position for each paint, similar to how header is paint