Mailing-List: contact users-help@flex.apache.org; run by ezmlm
Precedence: bulk
Reply-To: users@flex.apache.org
MIME-Version: 1.0
In-Reply-To: <1BAD6DED-FE4E-47AE-99C3-C5E67C7BBE6E@gmail.com>
References: 
 <CANcBHEDTODBx-vXQQZv3cmGJKAq0Bps7+-AbgCnnp0sgLWRNQQ@mail.gmail.com>
	<8329C3A9-25F3-4762-8286-21A9C161ADA8@classsoftware.com>
	<CANcBHED3UD9mt1h0oxkkojZtX=3-_mBBsD=8Chc9hgbEu5w_WQ@mail.gmail.com>
	<C67C77B9-5B63-4A3D-ABD7-6E249831AF5E@gmail.com>
	<CANcBHEDOJVucD2C8xLvaZpgSYQBsCFO-67cvtoQvGBpe_BHtCg@mail.gmail.com>
	<CANcBHEB45B_=6zy3xwRN003xTN5h9Br47dv9X0CoJ9MeYFM7Gg@mail.gmail.com>
	<67AD7D35-6BD0-4751-8A35-27BA197894F3@gmail.com>
	<CANcBHEBuAgN6+0vAtesNYQbLbBAvkhRhkpGnWP4783CdKgFv0g@mail.gmail.com>
	<1BAD6DED-FE4E-47AE-99C3-C5E67C7BBE6E@gmail.com>
Date: Fri, 18 Mar 2016 13:21:40 -0500
Message-ID: 
 <CANcBHEAUv0=RBv=NOr4jTRhmwoOLDB94tzPb+7FnTHJ=vT_vSA@mail.gmail.com>
Subject: Re: Undo and redo in TLF
From: jude <flexcapacitor@gmail.com>
To: users@flex.apache.org
Content-Type: multipart/alternative; boundary=001a1142e72089040b052e56d13d

--001a1142e72089040b052e56d13d
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

I like it. If I commit this fix as is it will enable redo via CTRL+ Shift+Z
on Windows and Linux. If that's ok, or if I don't hear any response I'll go
ahead and commit the change.

CTRL+y will still perform redo. We'll have two ways to perform redo with
Spark / TLF based controls.
On Mar 18, 2016 7:32 AM, "Harbs" <harbs.lists@gmail.com> wrote:

> How do you like that? It looks like I did commit that code...
>
> I wonder if something changed in Flash player. Either way, feel free to
> add in a fix.
>
> On Mar 14, 2016, at 8:01 AM, jude <flexcapacitor@gmail.com> wrote:
>
> > It looks like there's already code in EditManager to handle CMD+SHIFT+Z
> but
> > it's not hitting it because it's checking for lowercase z. The shift ke=
y
> > makes it uppercase so we need to check for character code 90 too.
> >
> > The following code in the EditManager class at line 307 adds a test for
> > case 90. http://pastebin.com/gwP1WiAj
> >
> >
> > On Sun, Mar 13, 2016 at 8:21 AM, Harbs <harbs.lists@gmail.com> wrote:
> >
> >> No. I think that was how it=E2=80=99s always worked. I was going to fi=
x it so
> >> cmd/shift/z worked on Mac, but I don=E2=80=99t think that ever happene=
d.
> >>
> >> On Mar 13, 2016, at 7:35 AM, jude <flexcapacitor@gmail.com> wrote:
> >>
> >>> It looks like redo does work but it is using CTRL+Y or CMD+Y on Mac.
> I'm
> >>> pretty sure it was CMD+Z for Undo and CMD+SHIFT+Z for Redo. Does anyo=
ne
> >>> else recall this it working with this key combination?
> >>>
> >>> On Tue, Jan 26, 2016 at 12:54 PM, jude <flexcapacitor@gmail.com>
> wrote:
> >>>
> >>>> Yeah, I'll come up with some more test cases.
> >>>> On Jan 24, 2016 1:44 AM, "Harbs" <harbs.lists@gmail.com> wrote:
> >>>>
> >>>>> Jude,
> >>>>>
> >>>>> Can you test this against different Flex builds? I wonder if there
> was
> >>>>> some side effect of TLF changes.
> >>>>>
> >>>>> FWIW, redo works for me in the current build using raw TLF, but I
> have
> >>>>> not tried with Spark components.
> >>>>>
> >>>>> On Jan 16, 2016, at 3:22 AM, jude <flexcapacitor@gmail.com> wrote:
> >>>>>
> >>>>>> I remember it working in previous versions. I don't think it was
> >>>>> introduced
> >>>>>> in 4.15. I'm wondering if it was introduced in a recent AIR or Fla=
sh
> >>>>>> Player. Did it work for you?
> >>>>>>
> >>>>>> On Fri, Jan 15, 2016 at 12:47 AM, Justin Mclean <
> >>>>> justin@classsoftware.com>
> >>>>>> wrote:
> >>>>>>
> >>>>>>> Hi,
> >>>>>>>
> >>>>>>>> Does undo and more importantly redo work for everyone else?
> >>>>>>>
> >>>>>>> Does it work in previous versions? i.e. is this a regression or a=
n
> >>>>> issue
> >>>>>>> that has existed for a while?
> >>>>>>>
> >>>>>>> Thanks,
> >>>>>>> Justin
> >>>>>
> >>>>>
> >>
> >>
>
>

--001a1142e72089040b052e56d13d--