#1225 ✓resolved

Image Size Not Inserted in Image Properties Popup

Reported by Peter | October 8th, 2014 @ 03:34 PM | in 2.3.5 (closed)

When inserting an image, the image size is no longer picked up in image properties, even after clicking the refresh option. I have to insert the size manually. This has been so for a little while.

Comments and changes to this ticket

  • dleffler

    dleffler October 9th, 2014 @ 01:14 PM

    • Tag set to ckeditor
    • Assigned user changed from “expNinja” to “dleffler”
    • Milestone set to 2.3.2

    I hadn't noticed that before. It is a result of moving to a newer version of CKEditor and an updated image 'widget' which allows for captions, etc... I'm trying to track down why (other than the their CKEditor 'load' event isn't triggered)

    A 'work-around' is to go ahead and insert the image, then edit it (double-click on it, or right-click and select 'image properties), then click the 'reset' button (circle with the arrow) which will fill in the dimensions.

  • dleffler

    dleffler October 9th, 2014 @ 03:49 PM

    Ok, I've taken a long look at this and don't understand why CKEditor is not tripping the 'load' event to update the sizes. The demo on their site works, but I can't get it to work with us (it's never worked since we updated to using the new CKEditor image widget)...HOWEVER, I have coded a fix when using the new File Manager (elFinder) that will not only populate the sizes, but transfers the 'alt' info from the file manager to the editor dialog (sorry, this would require an extensive rewrite of quite a bit of code to get it working in the traditional file manager).

    I hadn't thought about a 'patch' for v2.3.1 yet, but the code will be in the pre-release v2.3.2 (github 'develop' repo) later today.

  • expNinja

    expNinja October 9th, 2014 @ 04:04 PM

    (from [a0252896db78fb9968d30930768188d419388d82]) Fixes some issues/quirks related to CKEditor; now ensures image size is transferred to insert image (only if using elFinder), also transfers 'alt' from file manager; updates/fixes text inline editor views w/CKEditor to use new insert image widget like rest of dialogs do [#1225] https://github.com/exponentcms/exponent-cms/commit/a0252896db78fb99...

  • Peter
  • dleffler

    dleffler December 20th, 2014 @ 01:45 PM

    • Milestone changed from 2.3.2 to 2.3.3
  • dleffler

    dleffler February 28th, 2015 @ 03:13 PM

    • Milestone changed from 2.3.3 to 2.3.4
  • dleffler

    dleffler May 2nd, 2015 @ 05:04 PM

    • Milestone changed from 2.3.4 to 2.3.5
  • dleffler

    dleffler August 20th, 2015 @ 01:46 PM

    • State changed from “new” to “resolved”
    • Tag changed from ckeditor to ckeditor, wysiwyg

    This works in ckeditor (except for traditional file manager) and now tinymce

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

Bug Tracker for Exponent CMS

Shared Ticket Bins

People watching this ticket