-
Notifications
You must be signed in to change notification settings - Fork 100
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add single dictionary handlebars #814
add single dictionary handlebars #814
Conversation
✔️ No visual differences introduced by this PR. View Playwright Report (note: open the "playwright-report" artifact) |
94d1cdd
to
64dcc9e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Going to merge this PR first #857 while you merge/rebase
Only problem I see with this is the handlebars dont work in the |
The tester should also support them now, nice catch. |
@@ -19,7 +19,7 @@ | |||
import {isObjectNotArray} from '../core/object-utilities.js'; | |||
|
|||
/** @type {RegExp} @readonly */ | |||
const markerPattern = /\{([\w-]+)\}/g; | |||
const markerPattern = /\{([\p{Letter}\p{Number}_-]+)\}/gu; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is just a QOL change with no functional change? I can't find the documentation for this 😅
Adds automatically generated handlebars for each of the enabled dictionaries, for splitting mono- and bilingual dictionaries and similar uses. Closes #322