Hi @ChristoperHowell. Thanks for helping make the AdaptiveCards JS renderer + tooling better. As additional verification, once the JS build succeeds, please go to the test site to test out your website/designer changes.

All CLA requirements met.

Thanks for the PR, @ChristoperHowell 👍

Taking a look now :)

Hi, @dclaux. I originally searched for keycode using VS code. This is what came back and I have fixed them. Would you be able to point me in the right direction to find more occurrences?

@ChristoperHowell the other references are in the designer code base, which lives in the source\nodejs\adaptivecards-designer folder.

Thanks for the reply @dclaux. I’ve manually checked every file in the designer project and can find no other occurrences of the deprecated key code property.

From my understanding, the original issue requested that all occurrences of the deprecated key code property be updated to use the code property, which I have done this. Was there an additional requirement, or did I make a mistake in my understanding of this request? I can find other event properties being used such as the event shift key property. Would you like these modified to use the event code property?

Thanks for the input!

@ChristoperHowell my apologies. I do not know what I typed in the search box, but I must have typed something else than "keyCode", as I'm now not finding the references I mentioned before. Or maybe I just dreamt them.

Either way, your changes look good, so I'll go ahead and approve your PR then merge.

Thank you very much for this contribution!

@dclaux, not a problem! Thanks for letting me contribute!