Follow up for #2421427-154: Improve the UX of Quick Editing single-valued image fields.

Webchick:

If you upload an image via CKEditor, as well as an image via a regular image field, the Quick Edit target will fail to allow you to target the Image field (only Body).

I ran into that issue also. It is however a bit more generic.

If an image field is floated left in a node and if you have field content after the image it often prevents you from targeting the image for quick editing. And it thus becomes impossible to edit.

This is for example the case in Bartik, but can be so in custom themes as well.

.node .field--type-image {
    float: left;
    margin: 0 1em 0 0;
}

I assume this will be the first annoyance of content admins when working with this.

Example layout, left image can't be targeted due to body text to the right:

CommentFileSizeAuthor
quick_edit_image.png27.08 KBdmsmidt
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dmsmidt created an issue. See original summary.

Wim Leers’s picture

Component: image.module » quickedit.module
Status: Active » Closed (duplicate)
Related issues: +#2637500: Fields floated via CSS can break Quick Edit — also happens for Article nodes in Bartik

This problem has existed for years. We have #2637500: Fields floated via CSS can break Quick Edit — also happens for Article nodes in Bartik for that.

This is definitely not a bug in image.module, it's a bug in themes, and arguably a bug in quickedit.module.