Problem/Motivation

The current embed dialog only allows settings custom dimensions for image media. The original image styles select was hidden.

Proposed resolution

Re-enable image styles select box and add some additional functionality:
Add additional functionality to the settings form to configure allowed image styles.
Enhance embed dialog - some cleanups and re-organize image dimensions select element.

Remaining tasks

Everything should be implemented. Needs review.

User interface changes

Creates a slight change in embed dialog, where additional select appears where the user can select either custom dimensions or pre-defined image styles.

Comments

rang501 created an issue. See original summary.

rang501’s picture

Status: Active » Needs review
ytsurk’s picture

StatusFileSize
new13.24 KB
new1.56 KB

Looks good to me. I adopted one comment and removed the return types of the methods, are not yet a Drupalism ;)

It's also good to hide the link setting, and never allow linking the drupal-media, as it's not working. Or do you think this could be fixed .. just, does it make sense to link to content (what content that would be?) or the media entity (nobody want's that ..) - dunno if this can be configured elsewhere or will allow more options in the future ..

I'm a bit unsure about having the option always not only for image medias, fe. when using the formatter for a remote-video, it will display the preview image ..

ytsurk’s picture

Status: Needs review » Reviewed & tested by the community
ytsurk’s picture

StatusFileSize
new13.23 KB
new2.1 KB

Uhi - there was a bug. Other then media_directories_editor settings were not saved, now all are saved again.

ytsurk’s picture

Status: Reviewed & tested by the community » Fixed

  • ytsurk committed 499e82c on 2.x authored by rang501
    Issue #3170259 by ytsurk, rang501: Add support for image styles in...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.