Trap focus

[betweenbrain]

• Adds support closing the popup with the escape key
• Implements focus trapping when the calendar button is used to open the popup

Please note that much of this depends on #81 as focus trapping is an element of keyboard accessibility.

[mst101]

Hi @betweenbrain - thanks for your excellent suggestion of using focus trapping and for sharing the vanilla JS code.

If you check out my latest 'WIP' commit in #81 from a few days ago, you'll see I've pretty much got it working, but there's still some tidying up to do. I'll have a go at this over the weekend, but I wanted to put in a PR (#88) for some more general housekeeping first.

BTW - I like the way you break up your commits into small bite sized chunks, so I've been copying your style!

[betweenbrain]

Hello @mst101!

Hi @betweenbrain - thanks for your excellent suggestion of using focus trapping and for sharing the vanilla JS code.

You're very welcome, I'm glad that it's helpful.

If you check out my latest 'WIP' commit in #81 from a few days ago, you'll see I've pretty much got it working...

That's great. To confirm, are you saying that you are implementing focus trapping as part of #81, and that this PR could be closed?

BTW - I like the way you break up your commits into small bite sized chunks, so I've been copying your style!

Awesome, thanks!

[MrWook]

Was done in #81