Through #3447050: Need to deep merge options for compatibility with other modules, we implemented a way to make this module compatible, with other contrib modules providing menu link options. Unfortunately, we forgot to also deep merge menu link translation options.
We should do this in this follow-up issue here.
Deep merge Menu link translation options.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | patch-4.patch | 1014 bytes | grevil |
Show commandsStart within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #3
grevil commentedAll done, please review!
Comment #4
grevil commentedStatic patch for the time being.
Comment #5
anybodyWhao, nice one @Grevil. Sorry I didn't see that earlier. That's a run!
Comment #6
anybodyFYI: #3199785: Support for translatable attributes could be seen as follow-up to make the options individualizable per language. That isn't possible yet and has never been.
Comment #7
opiAfter some personnal debugging on 1.3 if found that the merge is missing. Sadly I looked at the issue queue only after that and found your patch !
This is a massive RTBC for me, allowing my client to use both menu_link_attributes and menu_firstchild on a multilingual D10 website.
Thanks for your work
Comment #10
jcnventuraMerged. Thanks everyone for the work and the RTBC tests.
I'd like to create a new release, but it should add D11 support, so if you can, please go over to #3447090: [Drupal 11] Call to deprecated method loadRevision() and get it to RTBC. And yes, I know it doesn't even have code yet. However, D11-beta1 is now out, so the API will not change anymore until Drupal 12 in mid-2026, so this is a good time to make this module ready for D11.