
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 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 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 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...
-
-
-
-
-
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.