Thanks for the clarification and the wrapped lines.

Looks good to me.

-- Jon

On 7/30/19 3:50 AM, Hannes Wallnöfer wrote:
The XML structure is actually ok as the empty element is a <path .../> element inside the 
outer <svg>...</svg> element.

But that was indeed hard to read due to the  very long line. I uploaded a new 
webrev with wrapped lines:

http://cr.openjdk.java.net/~hannesw/8228393/webrev.01/

Hannes



Am 29.07.2019 um 18:01 schrieb Jonathan Gibbons <[email protected]>:

The XML for the SVG looks questionable since you have both a self-closing opening tag 
`<svg .../>` and a separate closing tag `</svg>`

Is it possible to wrap the lines?

-- Jon

On 7/25/19 10:07 AM, Hannes Wallnöfer wrote:
Please review:

JBS: https://bugs.openjdk.java.net/browse/JDK-8228393
Webrev: http://cr.openjdk.java.net/~hannesw/8228393/webrev.00/

The external-link.svg is just added just for future editing as the CSS-inlined 
bits can’t be easily modified.

This is the smaller superscript version as shown in these examples:

http://cr.openjdk.java.net/~hannesw/8228393/demo/v3/IDN.html
http://cr.openjdk.java.net/~hannesw/8228393/demo/v3/SignatureMethod.html
http://cr.openjdk.java.net/~hannesw/8228393/demo/v3/package-use.html

Thanks,
Hannes

Reply via email to