fix: fix '(' mapping and add more key mappings

[Romulad]

What:

Extend the default key map to support additional characters such as / and :.

Why:

Currently, some common characters are not handled correctly by the default key map.

For example:

userEvent.keyboard('https://google.com');

Produces:

httpsUnknownUnknownUnknowngoogle.com

This happens because characters like / and : are not mapped, leading to Unknown outputs.

How:

• Added missing key mappings for:
  • / (\x2f)
  • : (\x3a)
  • additional characters
• Fixed an incorrect mapping for (:
  • previously mapped to \x29
  • corrected to \x28

Checklist:

• Tests (N\A)
• TypeScript definitions updated (N\A)
• Ready to be merged
  

[crutchcorn]

Whoops! I marked this as a "ready for review" PR despite you making it a draft. This LGTM - is it ready to go from your perspective?

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/cli-testing-library@45

commit: e1c3f88

[Romulad]

Yes, it’s ready from my perspective. I’m also thinking about a potential improvement to make the key mapping more scalable. I’ll open an issue about that.

[crutchcorn]

Love it! Look forward to seeing it :)

[Romulad]

Great!

[Romulad]

Here my suggestion: #46 (comment)
Please, let me know what you think about it :)