For some reason the height/width context options added in #2063785: Add width/height settings per context are not stored on a per-bundle basis. I'm relatively new to Scald so perhaps there's a good reason for this? It seems limiting.

Comments

azinck created an issue.

DrCord’s picture

DrCord commented

This is bad, having the dimensions shared between images and video causes issues.

nagy.balint’s picture

nagy.balint commented

A workaround could be to have different contexts for video, and then limiting the contexts displayed to users in the wysiwyg with the hook_scald_wysiwyg_context_list_alter

But this should be solved nevertheless. Although we have to make sure that the legacy way of exporting contexts continue to work even after the change.

jcisio’s picture

jcisio commented

I'd say this is by design. In a context, we want to display atoms indifferently on the atom type. If you want different dimensions, then you should not use it at all. You'd rather use different player with different settings for different atom types.

azinck’s picture

azinck commented

@jcisio if it's something that needs to be specific to the player then why have the option here at all? Can you elaborate on what you think the downsides are of making the dimension settings apply only to the specific bundle and context? Or, conversely, what is the upside of the current behavior?

Even if we grant the current behavior as being "correct" the UI is very confusing as it does not make clear which options are applied globally to all bundles using the context and which are applied only to the current bundle. I had assumed the settings were per-bundle until I started noticing stuff not working and had to look and figure out that I was playing a game of whack-a-mole, changing the setting in one place only to have screwed up my setting in another...