If the display setting of an image asset includes a link (to content or file), editing shows two annoying behaviors :

- The link is still active : a single left click open a new page inside ckeditor's iframe...
of course, you can't go back, and your previous edits are lost !
As the default cursor implies, left click should just select the asset - or cursor should become a pointer.

- If settings includes an align, the css layout is broken :
line 216 of asset/ckeditor/plugin.js should change from
$image.css('float', align).siblings('div.field').css('clear', 'both');
to
$image.css('float', align).parent().siblings('div.field').css('clear', 'both');

Thanks

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

agalitsyn’s picture

Version: 7.x-1.0-beta4 » 7.x-1.x-dev
FileSize
889 bytes

Thank you, made patch for this

agalitsyn’s picture

Status: Active » Patch (to be ported)
gifad’s picture

Status: Patch (to be ported) » Needs review
FileSize
868 bytes

Attached a patch to fix “links active in CKEditor” ;-)

IRuslan’s picture

Issue summary: View changes
Status: Needs review » Closed (fixed)

Thank you guys, both patches committed to dev.