> The [existing 
> documentation](https://docs.oracle.com/en/java/javase/25/docs/api/java.base/java/net/URL.html#constructor-deprecation)
>  when rendered was hard to read as the entire string was in a `pre` and 
> rendered in the same font.  Coupled with the fact the the first URL would 
> contain a space this made it hard to scan as you had no visual clues and had 
> to backtrack to re-read what the sentence was trying to say.
> 
> Switch to using inline definition using {@code} and swapped the order so that 
> the string without spaces came first negating any need for back tracking when 
> scanning

James Nord has updated the pull request with a new target base due to a merge 
or a rebase. The incremental webrev excludes the unrelated changes brought in 
by the merge/rebase. The pull request contains three additional commits since 
the last revision:

 - 8381184: incorporate feedback
 - Merge remote-tracking branch 'origin/master' into patch-2
 - make the reading of URL encoding issues easier to read
   
   The existing documentation when rendered was hard to read for a as the 
entire string was in a pre and rendered in the same font.  Coupled with the 
fact the the first URL would contain a space this made it hard to scan as you 
had no visual clues and had to backtrack to re-read what the sentence was 
trying to say.
   
   Switch to using inline definition using {@code} and swapped the order so 
that the string without spaces came first negating any need for back tracking 
when scanning

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/30074/files
  - new: https://git.openjdk.org/jdk/pull/30074/files/df0b76e9..9904dc64

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=30074&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=30074&range=00-01

  Stats: 92146 lines in 2572 files changed: 51314 ins; 17516 del; 23316 mod
  Patch: https://git.openjdk.org/jdk/pull/30074.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/30074/head:pull/30074

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

Reply via email to