- 🇺🇸United States rosemarystanley
I appreciate this patch. We've tried using the anchor_link module but there's some conflicts with the drupal link ckeditor button. I did want to note while patch #54 does work well, I wanted to report a small bug. When you double click on an existing anchor the drupal link dialog popups up, not the anchor form. A workaround is right clicking to select "Edit anchor" to get the correct dialog box.
- Status changed to Needs work
over 1 year ago 2:28am 19 April 2023 - 🇦🇺Australia gordon Melbourne
I just got very excited but then disappointed since there is no CKEditor 5 support.
I assume that this works great in CKEditor 4, but version 2.x adds support for CKEditor 5 so this needs work in both 4 and 5.
Changed back to needs work.
- 🇱🇻Latvia biguzis
Updated #54 with change from
drupal_get_path('module', 'editor_advanced_link')
to$this->getModulePath( 'editor_advanced_link')
for D10 support. - 🇱🇻Latvia biguzis
Dont use patch from #57. It was wrong since binary was messed up. Edited #54 with http://hexfiend.com/ but still want to know how to do it properly with git.
git diff --binary > patch.patch
didn't worked for me. - 🇺🇸United States generalredneck
A note on the PNG problems... maybe use as SVG instead?
- 🇺🇸United States generalredneck
So a few things... the latest patch only works with the patch command. git apply complains of
$ git apply editor_advanced_link-add_anchor_button-2595337-58.patch editor_advanced_link-add_anchor_button-2595337-58.patch:22: trailing whitespace. �PNG editor_advanced_link-add_anchor_button-2595337-58.patch:32: trailing whitespace. �PNG editor_advanced_link-add_anchor_button-2595337-58.patch:41: trailing whitespace. �PNG editor_advanced_link-add_anchor_button-2595337-58.patch:52: trailing whitespace. �PNG error: cannot apply binary patch to 'editor_advanced_link-add_anchor_button-2595337-54.patch' without full index line error: editor_advanced_link-add_anchor_button-2595337-54.patch: patch does not apply
Additionally, this patch includes another patch in it
$ patch -p1 < editor_advanced_link-add_anchor_button-2595337-58.patch patching file editor_advanced_link.routing.yml patching file js/plugins/anchor/icons/anchor-rtl.png patching file js/plugins/anchor/icons/anchor.png patching file js/plugins/anchor/icons/hidpi/anchor-rtl.png patching file js/plugins/anchor/icons/hidpi/anchor.png patching file js/plugins/anchor/plugin.js patching file src/Form/EditorAnchorDialog.php patching file src/Plugin/CKEditorPlugin/AnchorLink.php patching file editor_advanced_link-add_anchor_button-2595337-54.patch
Lastly, the images don't load in the browser or in the OS.
- 🇺🇸United States generalredneck
Rerolling#58 with svg icons to make for a better patching experience. Removes the extraneous patch file also.
- 🇺🇸United States DamienMcKenna NH, USA
I think it might be easier to write an update script to convert existing data and train people to use the "id" attribute.