- Issue created by @ammaletu
- π©πͺGermany ammaletu Bonn, Germany
Once this is fixed, it would be nice to tag a new release. We just migrated from CKEditor 4 to version 5, and this makes this button harder to use for German users.
- π¬π§United Kingdom aaron.ferris
aaron.ferris β made their first commit to this issueβs fork.
- Merge request !8Issue #3439422: make the abbreviation button translatable β (Closed) created by aaron.ferris
- π¬π§United Kingdom aaron.ferris
Ive had a go at this, the diff for the build is a bit odd though, what am I doing wrong?
1. npm install
2. npm run build?
- Status changed to Needs review
8 months ago 5:40pm 8 April 2024 - π¨π¦Canada mparker17 UTC-4
I think @boromino is more knowledgeable than myself about whether this is the correct solution (and whether the JS got built correctly).
The code looks like Drupal Core's MissingAlternativeTextView custom CKEditor plugin, and I know from experience that
Drupal.t()
is the correct way to use Drupal's translation system in Drupal-JavaScript β , so it seems good to me. -
boromino β
committed bce4a856 on 4.0.x
Issue #3439422 by aaron.ferris, Ammaletu, mparker17: CKEditor 5: Toolbar...
-
boromino β
committed bce4a856 on 4.0.x
- Status changed to Fixed
8 months ago 9:27am 10 April 2024 - π¨πSwitzerland boromino
Thanks for the patch. Fixed and created new release.
- π¬π§United Kingdom aaron.ferris
Hello @boromino sorry to be a pain, but would appreciate a contribution credit for this please!
- π¨π¦Canada mparker17 UTC-4
You were credited in the commit message; but I've credited you in the issue node as well! Thanks very much!
Automatically closed - issue fixed for 2 weeks with no activity.