Re: RFR: 8260314: Replace border="1" on tables with CSS [v3]

2021-01-26 Thread Sergey Bylokhov
On Tue, 26 Jan 2021 17:32:54 GMT, Alexey Ivanov wrote: >> Replace presentational attribute `border="1"` on `` element with CSS >> styles: >> table {border: outset 1px} >> th, td {border: inset 1px} >> >> These declarations produce the same rendering as the default one in Firefox >> and Edge.

Re: RFR: 8260314: Replace border="1" on tables with CSS

2021-01-26 Thread Alexey Ivanov
On Tue, 26 Jan 2021 19:17:24 GMT, Jonathan Gibbons wrote: > In general, I recommend where possible using the styles provided in the > standard stylesheet, for overall visual consistency. I totally agree. I overlooked the standard styles for the tables. Thanks to @mrserb for pointing me in

Re: RFR: 8260314: Replace border="1" on tables with CSS

2021-01-26 Thread Jonathan Gibbons
On Tue, 26 Jan 2021 18:33:48 GMT, Alexey Ivanov wrote: >>> Probably we can import the CSS used by the javadoc itself? >> >> We do. For example, [AWT Modality in JDK >> 15](https://docs.oracle.com/en/java/javase/15/docs/api/java.desktop/java/awt/doc-files/Modality.html) >> has borders because

Re: RFR: 8260314: Replace border="1" on tables with CSS

2021-01-26 Thread Alexey Ivanov
On Tue, 26 Jan 2021 15:39:30 GMT, Alexey Ivanov wrote: >> Probably we can import the CSS used by the javadoc itself? > >> Probably we can import the CSS used by the javadoc itself? > > We do. For example, [AWT Modality in JDK >

Re: RFR: 8260314: Replace border="1" on tables with CSS [v3]

2021-01-26 Thread Alexey Ivanov
> Replace presentational attribute `border="1"` on `` element with CSS > styles: > table {border: outset 1px} > th, td {border: inset 1px} > > These declarations produce the same rendering as the default one in Firefox > and Edge. > > Another option is to use `solid` in both cases. Alexey

Re: RFR: 8260314: Replace border="1" on tables with CSS [v2]

2021-01-26 Thread Alexey Ivanov
> Replace presentational attribute `border="1"` on `` element with CSS > styles: > table {border: outset 1px} > th, td {border: inset 1px} > > These declarations produce the same rendering as the default one in Firefox > and Edge. > > Another option is to use `solid` in both cases. Alexey

Re: RFR: 8260314: Replace border="1" on tables with CSS

2021-01-26 Thread Alexey Ivanov
On Tue, 26 Jan 2021 01:54:10 GMT, Sergey Bylokhov wrote: > Probably we can import the CSS used by the javadoc itself? We do. For example, [AWT Modality in JDK 15](https://docs.oracle.com/en/java/javase/15/docs/api/java.desktop/java/awt/doc-files/Modality.html) has borders because of

Re: RFR: 8260314: Replace border="1" on tables with CSS

2021-01-25 Thread Sergey Bylokhov
On Mon, 25 Jan 2021 11:50:20 GMT, Alexey Ivanov wrote: >> Replace presentational attribute `border="1"` on `` element with CSS >> styles: >> table {border: outset 1px} >> th, td {border: inset 1px} >> >> These declarations produce the same rendering as the default one in Firefox >> and Edge.

Re: RFR: 8260314: Replace border="1" on tables with CSS

2021-01-25 Thread Alexey Ivanov
On Sat, 23 Jan 2021 19:06:05 GMT, Alexey Ivanov wrote: > Replace presentational attribute `border="1"` on `` element with CSS > styles: > table {border: outset 1px} > th, td {border: inset 1px} > > These declarations produce the same rendering as the default one in Firefox > and Edge. > >

RFR: 8260314: Replace border="1" on tables with CSS

2021-01-23 Thread Alexey Ivanov
Replace presentational attribute `border="1"` on `` element with CSS styles: table {border: outset 1px} th, td {border: inset 1px} These declarations produce the same rendering as the default one in Firefox and Edge. Another option is to use `solid` in both cases. - Commit