Opened 10 years ago
Last modified 9 years ago
#14550 review Bug
Direct path to handle.png in widget plugin
Reported by: | Karen Ananiev | Owned by: | Tade0 |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | General | Version: | |
Keywords: | Cc: |
Description
There is the direct path https://github.com/ckeditor/ckeditor-dev/blob/master/plugins/widget/plugin.js#L3132 to the image file instead of CKEDITOR.getUrl and that causes the issue when custom CKEDITOR_GETURL function is used.
Change History (5)
comment:1 Changed 10 years ago by
Status: | new → pending |
---|---|
Version: | 4.5.7 |
comment:2 Changed 10 years ago by
Of course. We use CDN and add site id and some fingerprint to the static files. So the path looks like https://cdn.hostname/path/file.ext?_=fingerprint:site_id and CDN returns 404 for the requests without that query param.
comment:3 Changed 10 years ago by
Status: | pending → confirmed |
---|
Having getURL
in that place could definitely solve such problems.
comment:4 Changed 10 years ago by
Correct, this could be improved. Luckily there's an easy workaround for this issue, so if anyone face this problem, you could override it in your config.contentsCss stylesheet:
.cke_widget_drag_handler_container { background-image: url(https://dev.null/my-custom-image.png) !important; }
comment:5 Changed 9 years ago by
Owner: | set to Tade0 |
---|---|
Status: | confirmed → review |
Fixed.
Changes pushed to branch:t/14550.
Ticket makes a lot of sense but before confirming it, could you perhaps tell me what issue where you getting with direct URL to handle and custom
getUrl
function?