On Thu, 19 Jun 2025 16:45:11 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:
>> getVisibleEditorRect() return the bounds of the text editor and the return
>> value is used to to set-size/paint the
>> `javax.swing.text.View` of this text editor component.
>> "the allocation" refers to what allocated area the view needs to paint, it
>> seems to be a common phrase used in context of View class which says
>>
>>> "The view gets its allocation from the parent at paint time, so it must be
>>> prepared to redo layout if the allocated area is different from what it is
>>> prepared to deal with. "
>>
>> In that regard, probably getAllocatedViewRect() would have been better
>> choice of the method name
>
> @prsadhuk @prrace Does this description work?
>
>
> /**
> * Gets the allocation for the root view.
> * The returned rectangle is unrelated to visibility,
> * it is passed as the {@code allocation} parameter to
> * the {@link View#paint(Graphics, Shape) View.paint} method.
> * The component must have a non-zero positive size for
> * this translation to be computed.
> *
> * @return the bounding box for the root view
> */
> protected Rectangle getVisibleEditorRect() {
>
>
> This renders like this:
>
>> Gets the allocation for the root view. The returned rectangle is unrelated
>> to visibility, it is passed as the allocation parameter to the
>> [`View.paint`](https://docs.oracle.com/en/java/javase/24/docs/api/java.desktop/javax/swing/text/View.html#paint(java.awt.Graphics,java.awt.Shape))
>> method. The component must have a non-zero positive size for this
>> translation to be computed.
>
> We can also add @TejeshR13's
> [suggestion](https://github.com/openjdk/jdk/pull/25850/files#r2153658561):
> “The method name is preserved for backward compatibility.”
It is passed to `View.setSize` also so just mentioning one may not be
right..and I guess {@code allocation} is a no-op as it is not a class..
Maybe we can mention "The returned rectangle is unrelated to visibility, and is
used as an allocation parameter in javax.swing.text.View class to set the size
and paint the allocated View"
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25850#discussion_r2157975053
