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 messages:
 - Move table styles to common 

Integrated: 8260315: Typo "focul" instead of "focus" in FocusSpec.html

2021-01-23 Thread Alexey Ivanov
On Sat, 23 Jan 2021 12:15:04 GMT, Alexey Ivanov  wrote:

> A trivial fix for the typo in AWT Focus Spec, it replaces “focul” with 
> “focus”.

This pull request has now been integrated.

Changeset: b53d5cac
Author:Alexey Ivanov 
URL:   https://git.openjdk.java.net/jdk/commit/b53d5cac
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod

8260315: Typo "focul" instead of "focus" in FocusSpec.html

Reviewed-by: kizune, pbansal

-

PR: https://git.openjdk.java.net/jdk/pull/2209


Re: RFR: 8260315: Typo "focul" instead of "focus" in FocusSpec.html

2021-01-23 Thread Pankaj Bansal
On Sat, 23 Jan 2021 12:15:04 GMT, Alexey Ivanov  wrote:

> A trivial fix for the typo in AWT Focus Spec, it replaces “focul” with 
> “focus”.

Marked as reviewed by pbansal (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/2209


Re: RFR: 8260315: Typo "focul" instead of "focus" in FocusSpec.html

2021-01-23 Thread Alexander Zuev
On Sat, 23 Jan 2021 12:15:04 GMT, Alexey Ivanov  wrote:

> A trivial fix for the typo in AWT Focus Spec, it replaces “focul” with 
> “focus”.

Marked as reviewed by kizune (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/2209


RFR: 8260315: Typo "focul" instead of "focus" in FocusSpec.html

2021-01-23 Thread Alexey Ivanov
A trivial fix for the typo in AWT Focus Spec, it replaces “focul” with “focus”.

-

Commit messages:
 - 8260315: Typo "focul" instead of "focus" in FocusSpec.html

Changes: https://git.openjdk.java.net/jdk/pull/2209/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2209&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8260315
  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.java.net/jdk/pull/2209.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/2209/head:pull/2209

PR: https://git.openjdk.java.net/jdk/pull/2209


Integrated: 8240247: No longer need to wrap files with contentContainer

2021-01-23 Thread Alexey Ivanov
On Fri, 22 Jan 2021 19:50:17 GMT, Alexey Ivanov  wrote:

> [JDK-8231144](https://bugs.openjdk.java.net/browse/JDK-8231144) wrapped the 
> contents of plain HTML documents into `` to 
> apply the styles and add the margins around the content for a consistent look.
> 
> This is no longer necessary after 
> [JDK-8239817](https://bugs.openjdk.java.net/browse/JDK-8239817) was fixed.
> 
> These documents looks fine without being wrapped. So this fix basically 
> undoes changes under JDK-8231144 and removes the redundant `` wrapper 
> element.

This pull request has now been integrated.

Changeset: f624dba6
Author:Alexey Ivanov 
URL:   https://git.openjdk.java.net/jdk/commit/f624dba6
Stats: 45 lines in 15 files changed: 0 ins; 30 del; 15 mod

8240247: No longer need to wrap files with contentContainer

Reviewed-by: serb

-

PR: https://git.openjdk.java.net/jdk/pull/2203