HOOKS: Naming is unexpected

[tg-ephox]

Issue Overview

Existing hooks use naming like 'core\anchor-attribute' but the '\a' is a non-existent escape sequence so this is interpreted as 'coreanchor-attribute'.

I stumbled across this today investigating hooks for use in footnotes and created a hook named 'core\footontes' but '\f' is a valid escape sequence for ASCII character 12 (Line Feed) so my hook name was failing validation here

Incidently the coment here wasn't updated when the validation rules changed.

Possible Solution

Use '-' to delimit hook name.

Screenshots / Video

Related Issues and/or PRs

Todos

• Tests
• Documentation

[gziolo]

@aduth pinging you so you were aware why \a got replaced with - for the hook namespace.

[aduth]

I think a better permanent solution is to update the underlying hooks implementation to use / for namespacing.

[gziolo]

Agreed, we need to relax validation first.