A few things still need fixing, the end result should look like this :
+ * value. Subclasses may override this method to compute
+ * a value, e.g. the change required to scroll one
* (variable height) line of text or one row in a table.
+ * then most look and feel implementations will not provide scrolling
+ * to the right/down.
+ * outside the scrollbar "knob" to scroll up or down by a large amount.
+ * Subclasses may override this method to compute a
+ * value, e.g. the change required to scroll one paragraph
+ * in a text document.
+ * then most look and feel implementations will not provide scrolling
+ * to the right/down.
-phil.
On 7/18/18, 3:45 AM, Krishna Addepalli wrote:
Hi Phil,
I have reworded the sentences slightly so that it reads grammatically
correct to the best of my knowledge.
Here is the updated webrev:
http://cr.openjdk.java.net/~kaddepalli/8206343/webrev01/
<http://cr.openjdk.java.net/%7Ekaddepalli/8206343/webrev01/>
Thanks
Krishna
*From:*Phil Race
*Sent:* Monday, July 16, 2018 11:27 PM
*To:* Krishna Addepalli <[email protected]>;
[email protected]
*Subject:* Re: <Swing Dev> [12] RFR: JDK-8206343: There is a typo in
the java documentation of javax.swing.JScrollBar.
Since you are updating these lines, maybe you could fix the grammar as well as
the spelling ?
- * the most look and feels will not provide the scrolling to the right/down.
+ * then most look and feels will not provide the scrolling to the
right/down.
- * up or down by large amount. Subclasses my override this
+ * up or down by large amount. Subclasses may override this
- * the most look and feels will not provide the scrolling to the right/down.
+ * then most look and feels will not provide the scrolling to the
right/down.
-phil.
On 07/12/2018 04:46 AM, Krishna Addepalli wrote:
Hi All,
Please review a trivial typo corrections in JScrollbar. Apart from
those specified in the bug, I have also corrected few others that
I could find.
Bug: https://bugs.openjdk.java.net/browse/JDK-8206343
Webrev: http://cr.openjdk.java.net/~kaddepalli/8206343/webrev00/
<http://cr.openjdk.java.net/%7Ekaddepalli/8206343/webrev00/>
Thanks,
Krishna
