<Swing Dev> RFR: JDK-8235818: Inline information from broken external links in java.desktop

Prasanta Sadhukhan prasanta.sadhukhan at oracle.com
Thu Dec 12 10:40:06 UTC 2019


It seems jdk/jdk has introduced stricter check for img tag to have "alt" 
subtag.

Modified webrev to add alt subtag to img tag like this

201  * <p style="text-align:center"><img src="doc-files/plain1.gif"
  202  * alt="The preceding text describes this graphic.">

No visible change to the spec for this modification. mach5 is green

Bug: https://bugs.openjdk.java.net/browse/JDK-8235818

webrev: http://cr.openjdk.java.net/~psadhukhan/8235818/webrev.0/

Regards

Prasanta

On 12-Dec-19 12:06 AM, Phil Race wrote:
> +1
>
> -phil.
>
> On 12/10/19 2:12 AM, Prasanta Sadhukhan wrote:
>>
>> Modified webrev: http://cr.openjdk.java.net/~psadhukhan/8227607/webrev.3/
>>
>> with updated spec 
>> http://cr.openjdk.java.net/~psadhukhan/8227607/docs1/api/java.desktop/javax/swing/text/Document.html
>>
>> Regards
>>
>> Prasanta
>>
>> On 09-Dec-19 11:00 PM, Philip Race wrote:
>>> >
>>>
>>>   * ||
>>>     <http://cr.openjdk.java.net/%7Epsadhukhan/8227607/docs/api/java.desktop/javax/swing/text/Document.html#putProperty%28java.lang.Object,java.lang.Object%29>
>>>
>>>
>>> For more information on the |Document| class, see The Swing 
>>> Connection 
>>> <http://www.oracle.com/technetwork/java/javase/tech/articles-jsp-139072.html> 
>>>
>>>
>>>
>>> This link above  seems to be obsolete as well. I think it should 
>>> just be removed
>>>
>>> > If an element is a a leaf,isLeaf() returns true.
>>>
>>> add a space after the comma.
>>>
>>> Also can we wrap all  - or at least the ones that make sense -  API 
>>> references in {@code ... }  ?
>>>
>>> Method calls for sure.
>>>
>>> interrface -> interface.
>>>
>>> -phil.
>>>
>>> On 12/9/19, 3:16 AM, Prasanta Sadhukhan wrote:
>>>>
>>>> Modified version with new subtitle ""Overview and Programming Tips" 
>>>> and graphical_schema
>>>>
>>>> http://cr.openjdk.java.net/~psadhukhan/8227607/webrev.2/
>>>>
>>>> spec change will look like 
>>>> http://cr.openjdk.java.net/~psadhukhan/8227607/docs/api/java.desktop/javax/swing/text/Document.html
>>>>
>>>> Regards
>>>> Prasanta
>>>> On 08-Dec-19 1:47 AM, Philip Race wrote:
>>>>> I got very confused as to whether there was even an updated 
>>>>> version here.
>>>>> This conversation became too complex for the inline diff.
>>>>> I think that it would be best to first generate a webrev put it on cr.
>>>>>
>>>>>
>>>>> I do think we need to have a subtitle that prefaces this new 
>>>>> content rather
>>>>> than just appending it to "Properties" ..
>>>>>
>>>>> I suggest "Overview and Programming Tips" which makes it clearer 
>>>>> it is not (so much) specification as developer guide.
>>>>>
>>>>> > Swing has a interesting interface, called the _|Element| 
>>>>> <http://cr.openjdk.java.net/%7Epsadhukhan/docs/api/java.desktop/javax/swing/text/Element.html>_,
>>>>>
>>>>> I do think this needs rephrasing.
>>>>>
>>>>> "Element is an important interface used in constructing a Document.
>>>>> It has the power ...."
>>>>>
>>>>> And each time it says "In Swing" I think, why does it bother 
>>>>> saying that ...
>>>>> confusion with an SGML Element ?
>>>>> Maybe it would read better written as "In the Swing text API's 
>>>>> document model"
>>>>>
>>>>> -phil.
>>>>>
>>>>> On 12/6/19, 7:22 PM, Prasanta Sadhukhan wrote:
>>>>>> Anything more needs to be added?!!! Can it be committed?
>>>>>>
>>>>>> On 06-Dec-19 9:49 AM, Prasanta Sadhukhan wrote:
>>>>>>> I guess I already shared... 
>>>>>>> http://cr.openjdk.java.net/~psadhukhan/docs/api/java.desktop/javax/swing/text/Document.html
>>>>>>>
>>>>>>> Regards
>>>>>>>
>>>>>>> Prasanta
>>>>>>>
>>>>>>> On 06-Dec-19 4:44 AM, Sergey Bylokhov wrote:
>>>>>>>> Hi, Prasanta.
>>>>>>>>
>>>>>>>> Can you please share the final generated JavaDoc, it is hard to 
>>>>>>>> read the diff below.
>>>>>>>>
>>>>>>>> On 12/4/19 10:09 am, Prasanta Sadhukhan wrote:
>>>>>>>>> Inlined information from element_interface page into Document 
>>>>>>>>> class <spec will look like 
>>>>>>>>> http://cr.openjdk.java.net/~psadhukhan/docs/api/java.desktop/javax/swing/text/Document.html]
>>>>>>>>>
>>>>>>>>> /diff -r 53eff6c5da11 
>>>>>>>>> src/java.desktop/share/classes/javax/swing/Timer.java/
>>>>>>>>>
>>>>>>>>> --- a/src/java.desktop/share/classes/javax/swing/Timer.java 
>>>>>>>>> Sun Oct 06 21:42:50 2019 -0700
>>>>>>>>> +++ b/src/java.desktop/share/classes/javax/swing/Timer.java 
>>>>>>>>> Wed Dec 04 22:07:48 2019 +0530
>>>>>>>>> @@ -124,13 +124,6 @@
>>>>>>>>>    * <a 
>>>>>>>>> href="https://docs.oracle.com/javase/tutorial/uiswing/misc/timer.html" 
>>>>>>>>>
>>>>>>>>>    * target = "_top">How to Use Timers</a>,
>>>>>>>>>    * a section in <em>The Java Tutorial.</em>
>>>>>>>>> *- * For more examples and help in choosing between**
>>>>>>>>> **- * this <code>Timer</code> class and**
>>>>>>>>> **- * <code>java.util.Timer</code>,**
>>>>>>>>> **- * see**
>>>>>>>>> **- * <a 
>>>>>>>>> href="http://java.sun.com/products/jfc/tsc/articles/timer/"**
>>>>>>>>> **- * target="_top">Using Timers in Swing Applications</a>,**
>>>>>>>>> **- * an article in <em>The Swing Connection.</em>**
>>>>>>>>> *  * <p>
>>>>>>>>>    * <strong>Warning:</strong>
>>>>>>>>>    * Serialized objects of this class will not be compatible with
>>>>>>>>>
>>>>>>>>> /diff -r 53eff6c5da11 
>>>>>>>>> src/java.desktop/share/classes/javax/swing/text/Document.java/
>>>>>>>>>
>>>>>>>>> --- 
>>>>>>>>> a/src/java.desktop/share/classes/javax/swing/text/Document.java 
>>>>>>>>> Sun Oct 06 21:42:50 2019 -0700
>>>>>>>>> +++ 
>>>>>>>>> b/src/java.desktop/share/classes/javax/swing/text/Document.java 
>>>>>>>>> Wed Dec 04 22:07:48 2019 +0530
>>>>>>>>> @@ -167,9 +167,38 @@
>>>>>>>>>    *
>>>>>>>>>    * <p>For more information on the <code>Document</code> 
>>>>>>>>> class, see
>>>>>>>>>    * <a 
>>>>>>>>> href="http://www.oracle.com/technetwork/java/javase/tech/articles-jsp-139072.html">The 
>>>>>>>>> Swing Connection</a>
>>>>>>>>> *- * and most particularly the article,**
>>>>>>>>> **- * <a 
>>>>>>>>> href="http://java.sun.com/products/jfc/tsc/articles/text/element_interface">**
>>>>>>>>> **- * The Element Interface</a>.**
>>>>>>>>> **+ * <p>Swing has a interesting interface, called the 
>>>>>>>>> <u>{@link javax.swing.text.Element}</u>, **
>>>>>>>>> **+ * which has the power to describe various structural parts 
>>>>>>>>> of a document, **
>>>>>>>>> **+ * such as paragraphs, lines of text, or even (in HTML 
>>>>>>>>> documents) items in lists. **
>>>>>>>>> **+ * Conceptually, the Element interface captures some of the 
>>>>>>>>> spirit of an SGML document. **
>>>>>>>>> **+ * So if you know SGML, you may already have some 
>>>>>>>>> understanding of Swing's Element interface.**
>>>>>>>>> **+ * <p>In Swing, the interface Element defines a structural 
>>>>>>>>> piece of a Document, **
>>>>>>>>> **+ * like a paragraph, a line of text, or a list item in an 
>>>>>>>>> HTML document.**
>>>>>>>>> **+ * <p>Every Element is either a branch or a leaf. If an 
>>>>>>>>> element is a branch, **
>>>>>>>>> **+ * the isLeaf() method returns false. If an element is a a 
>>>>>>>>> leaf,isLeaf() returns true.**
>>>>>>>>> **+ * <p>Branches can have any number of children. Leaves do 
>>>>>>>>> not have children. **
>>>>>>>>> **+ * To determine how many children a branch has, you can 
>>>>>>>>> call getElementCount(). **
>>>>>>>>> **+ * To determine the parent of an Element, you can call 
>>>>>>>>> getParentElement(). **
>>>>>>>>> **+ * Root elements don't have parents, so calling 
>>>>>>>>> getParentElement() on a root returns null.**
>>>>>>>>> **+ * <p>An Element represents a specific region in a Document 
>>>>>>>>> that begins with startOffset **
>>>>>>>>> **+ * and ends just before endOffset. **
>>>>>>>>> **+ * The start offset of a branch Element is usually the 
>>>>>>>>> start offset of its first child. **
>>>>>>>>> **+ * Similarly, the end offset of a branch Element is usually 
>>>>>>>>> the end offset of its last child.**
>>>>>>>>> **+ * <p>Every Element is associated with an AttributeSet that 
>>>>>>>>> you can access by calling getAttributes(). **
>>>>>>>>> **+ * In an Element, and AttributeSet is essentially a set of 
>>>>>>>>> key/value pairs. **
>>>>>>>>> **+ * These pairs are generally used for markup -- such as 
>>>>>>>>> determining the Element's **
>>>>>>>>> *
>>>>>>>>>
>>>>>>>>> *+ * foreground color, font size, and so on. But it is up to 
>>>>>>>>> the model, and the developer, **
>>>>>>>>> **+ * to determine what is stored in the AttributeSet.**
>>>>>>>>> **+ * <p>You can obtain the root Element (or Elements) of a 
>>>>>>>>> Document by calling the **
>>>>>>>>> **+ * methods getDefaultRootElement() and getRootElements(), 
>>>>>>>>> which are defined in the Document interface.**
>>>>>>>>> **+ * <p>The Document interrface is responsible for 
>>>>>>>>> translating a linear view of the **
>>>>>>>>> **+ * characters into Element operations. It is up to each 
>>>>>>>>> Document implementation **
>>>>>>>>> **+ * to define what the Element structure is.**
>>>>>>>>> **+ * <p>It is recommended to look into {@link 
>>>>>>>>> javax.swing.text.EditorKit} and **
>>>>>>>>> **+ * {@link javax.swing.text.View}.**
>>>>>>>>> **+ * View is responsible for rendering a particular Element, 
>>>>>>>>> and **
>>>>>>>>> **+ * EditorKit is responsible for a ViewFactory that is able 
>>>>>>>>> to decide what **
>>>>>>>>> **+ * View should be created based on an Element.*
>>>>>>>>>
>>>>>>>>> Regards
>>>>>>>>>
>>>>>>>>> Prasanta
>>>>>>>>>
>>>>>>>>> On 04-Dec-19 10:45 AM, Sergey Bylokhov wrote:
>>>>>>>>>> On 12/3/19 9:08 pm, Prasanta Sadhukhan wrote:
>>>>>>>>>>> Hi Sergey,
>>>>>>>>>>>
>>>>>>>>>>> I could not find the page for element_interface itself so 
>>>>>>>>>>> how to get the information from there?
>>>>>>>>>>
>>>>>>>>>> I guess it is the same as you posted before:
>>>>>>>>>> https://web.archive.org/web/20120418045506/http://java.sun.com/products/jfc/tsc/articles/text/element_interface/ 
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> Regards
>>>>>>>>>>>
>>>>>>>>>>> Prasanta
>>>>>>>>>>>
>>>>>>>>>>> On 04-Dec-19 7:31 AM, Sergey Bylokhov wrote:
>>>>>>>>>>>> Hi, Prasanta.
>>>>>>>>>>>> Probably we can inline some information from the 
>>>>>>>>>>>> element_interface.html to the JavaDoc itself? The JavaDoc 
>>>>>>>>>>>> already has some similar schemes.
>>>>>>>>>>>>
>>>>>>>>>>>> On 12/2/19 2:37 am, Prasanta Sadhukhan wrote:
>>>>>>>>>>>>> Hi All,
>>>>>>>>>>>>>
>>>>>>>>>>>>> Please review a doc-fix for removing some dead links from 
>>>>>>>>>>>>> the specifications. These links were part of java.sun.com 
>>>>>>>>>>>>> but has not been migrated to docs.oracle.com so it's 
>>>>>>>>>>>>> better to remove them
>>>>>>>>>>>>>
>>>>>>>>>>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8227607
>>>>>>>>>>>>>
>>>>>>>>>>>>> diff -r 7637e77c4c8a 
>>>>>>>>>>>>> src/java.desktop/share/classes/javax/swing/Timer.java
>>>>>>>>>>>>> --- 
>>>>>>>>>>>>> a/src/java.desktop/share/classes/javax/swing/Timer.java 
>>>>>>>>>>>>> Mon Dec 02 11:58:14 2019 +0530
>>>>>>>>>>>>> +++ 
>>>>>>>>>>>>> b/src/java.desktop/share/classes/javax/swing/Timer.java 
>>>>>>>>>>>>> Mon Dec 02 16:02:08 2019 +0530
>>>>>>>>>>>>> @@ -124,13 +124,6 @@
>>>>>>>>>>>>>    * <a 
>>>>>>>>>>>>> href="https://docs.oracle.com/javase/tutorial/uiswing/misc/timer.html" 
>>>>>>>>>>>>>
>>>>>>>>>>>>>    * target = "_top">How to Use Timers</a>,
>>>>>>>>>>>>>    * a section in <em>The Java Tutorial.</em>
>>>>>>>>>>>>> *- * For more examples and help in choosing between**
>>>>>>>>>>>>> **- * this <code>Timer</code> class and**
>>>>>>>>>>>>> **- * <code>java.util.Timer</code>,**
>>>>>>>>>>>>> **- * see**
>>>>>>>>>>>>> **- * <a 
>>>>>>>>>>>>> href="http://java.sun.com/products/jfc/tsc/articles/timer/"**
>>>>>>>>>>>>> **- * target="_top">Using Timers in Swing Applications</a>,**
>>>>>>>>>>>>> **- * an article in <em>The Swing Connection.</em>*
>>>>>>>>>>>>>    * <p>
>>>>>>>>>>>>>    * <strong>Warning:</strong>
>>>>>>>>>>>>>    * Serialized objects of this class will not be 
>>>>>>>>>>>>> compatible with
>>>>>>>>>>>>> diff -r 7637e77c4c8a 
>>>>>>>>>>>>> src/java.desktop/share/classes/javax/swing/text/Document.java
>>>>>>>>>>>>> --- 
>>>>>>>>>>>>> a/src/java.desktop/share/classes/javax/swing/text/Document.java 
>>>>>>>>>>>>> Mon Dec 02 11:58:14 2019 +0530
>>>>>>>>>>>>> +++ 
>>>>>>>>>>>>> b/src/java.desktop/share/classes/javax/swing/text/Document.java 
>>>>>>>>>>>>> Mon Dec 02 16:02:08 2019 +0530
>>>>>>>>>>>>> @@ -167,9 +167,6 @@
>>>>>>>>>>>>>    *
>>>>>>>>>>>>>    * <p>For more information on the <code>Document</code> 
>>>>>>>>>>>>> class, see
>>>>>>>>>>>>>    * <a 
>>>>>>>>>>>>> href="http://www.oracle.com/technetwork/java/javase/tech/articles-jsp-139072.html">The 
>>>>>>>>>>>>> Swing Connection</a>
>>>>>>>>>>>>> *- * and most particularly the article,**
>>>>>>>>>>>>> **- * <a 
>>>>>>>>>>>>> href="http://java.sun.com/products/jfc/tsc/articles/text/element_interface">**
>>>>>>>>>>>>> **- * The Element Interface</a>.*
>>>>>>>>>>>>>    *
>>>>>>>>>>>>>    * @author  Timothy Prinzing
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>> Regards
>>>>>>>>>>>>>
>>>>>>>>>>>>> Prasanta
>>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>
>>>>>>>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/swing-dev/attachments/20191212/41f3a58d/attachment-0001.htm>


More information about the swing-dev mailing list