Scenario:

  • Drupal 7.23, latest releases of Media and File Entity v2, latest WYSIWYG, TinyMCE v3.
  • Embed an image via the WYSIWYG and selecting an item from the library.
  • Image is added correctly (view mode "default") and can be displayed correctly.
  • Select image, click the Link button in the toolbar, add a link, click Insert.
  • Click Save. View the node, the image is no longer clickable.
  • Edit the node again.
  • Select image again, click the Link button in the toolbar, add a link, click Insert.
  • Note that something else in the WYSIWYG can be selected, then you can select the image again and it still registers that there is a link on the image.
  • Click Save. View the node, the link is gone again.
  • Edit the node again.
  • Select image again, click the Link button in the toolbar, add a link, click Insert.
  • Click the "Disable rich-text" link; note that the link is no longer in the HTML code.

Comments

Anonymous’s picture

Anonymous (not verified) commented

I'm having the exact same problem, though I'm probably like one version behind. Damn..

Mooff’s picture

Mooff commented

Same with CKE (latest dev version of media 7.2 and file entity, latest wysiwyg, drupal 7.23).

Also the other values in the image overlay do not work. Like resizing.

- click the image, select the link button, add a link
- hit save node -> link gone
OR - hit disable rich text -> link gone

- click the image, resize
- hit save node -> resize gone
OR - hit disable rich text -> resize gone

Did some further checking. On our livesite (drupal 7.23, media 7.x-2.0-alpha2 and entity 7.x-2.0-alpha2, latest wysiwyg) everything works. On our testsite, which is a mirror created about 4 days ago with the developer versions it doesn't.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

I've debugged this a little and found that the problem is because the link is added inside the special MEDIA-WRAPPER-START comment, so when it turns the inline image from the WYSIWYG field into a media code it chops out the link.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Digging into it a little more, this MEDIA-WRAPPER-START/MEDIA-WRAPPER-END stuff is handled by media.filter.js, so it needs some custom work

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Closed (duplicate)

This appears to be resolved by #2067063: Wysiwyg integration is broken. Or maybe it's fixing the image styles. Either way, on my sites this is working now.

Anonymous’s picture

Anonymous (not verified) commented
Status: Closed (duplicate) » Needs work

Opening it back up. Patch causes "false" on all images to display on mine.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs work » Postponed

Issue #2067063: Wysiwyg integration is broken should resolve this, lets wait for it to be fixed first and see if further work is necessary.

ParisLiakos’s picture

ParisLiakos commented
Issue summary: View changes
Related issues: +#2067063: Wysiwyg integration is broken

this should be fixed in latest dev

ParisLiakos’s picture

ParisLiakos commented
Status: Postponed » Closed (duplicate)
bendev’s picture

bendev commented
Component: WYSIWYG integration » Code

dev version solves this issue.

would be great to get a new alpha version for this.

Moreover the dev version doesn't support 'file selector' widget, but only 'file browser' So with the dev version I don't get alt and titles for images.

thank you