Re: about the issue FLEX-33423?

[Alex Harui]

On 4/17/14 3:52 PM, "Justin Mclean" <jus...@classsoftware.com> wrote:

>Hi,
>
>> I decided to take a quick look.  It appears that maxChars is enforced in
>> RichEditableText and not in TLF.
>It is enforced in this case it just allows a maximum of 8 not 7 as it
>should.
It looks to me that it knows that maxChars is 7, but the operation
incorrectly says it is deleting a character when it isn't (it is just
adding one).

I believe that you were on the right track, that adjustOversetForward
wasn't doing the right thing.

I've made the change posted below.  It seemed to fix the bug and 784
mustella tests ran cleanly.

-Alex

diff --git a/textLayout/src/flashx/textLayout/utils/NavigationUtil.as
b/textLayout/src/flashx/t
index 85b2014..2ebb985 100644
--- a/textLayout/src/flashx/textLayout/utils/NavigationUtil.as
+++ b/textLayout/src/flashx/textLayout/utils/NavigationUtil.as
@@ -1109,7 +1109,7 @@ package flashx.textLayout.utils
                        var flowComposer:IFlowComposer =
range.textFlow.flowComposer;
                        var controller:ContainerController = null;
                        checkCompose(flowComposer, range.absoluteEnd);
-                       if (range.absoluteEnd >
flowComposer.damageAbsoluteStart - 1)
+                       if (range.absoluteEnd >=
flowComposer.damageAbsoluteStart - 1)
                        {
                                clampToFit(range,
flowComposer.damageAbsoluteStart - 1);
                                return true;