I forgot to ask, should I submit a bug report for this? With the code patch above, if we add the case for uppercase Z then non Mac operating systems we would be responding to CTRL+Shift+z as well. They would still respond to CTRL+y or we could limit CTRL+Shift+z by adding a check for isMac when shiftKey is true.
The question is, do any Windows or Linux programs use or respond to CTRL+SHIFT+Z? On Sun, Mar 13, 2016 at 11:01 PM, jude <[email protected]> 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 > key 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 <[email protected]> wrote: > >> No. I think that was how it’s always worked. I was going to fix it so >> cmd/shift/z worked on Mac, but I don’t think that ever happened. >> >> On Mar 13, 2016, at 7:35 AM, jude <[email protected]> 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 anyone >> > else recall this it working with this key combination? >> > >> > On Tue, Jan 26, 2016 at 12:54 PM, jude <[email protected]> wrote: >> > >> >> Yeah, I'll come up with some more test cases. >> >> On Jan 24, 2016 1:44 AM, "Harbs" <[email protected]> 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 <[email protected]> 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 Flash >> >>>> Player. Did it work for you? >> >>>> >> >>>> On Fri, Jan 15, 2016 at 12:47 AM, Justin Mclean < >> >>> [email protected]> >> >>>> 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 an >> >>> issue >> >>>>> that has existed for a while? >> >>>>> >> >>>>> Thanks, >> >>>>> Justin >> >>> >> >>> >> >> >
