iframemanager icon indicating copy to clipboard operation
iframemanager copied to clipboard

Support all configurations in the div

Open timiwahalahti opened this issue 2 years ago • 1 comments

Hi! We are using your Cookie Consent javascript plugin (😍) and the IframeManager looks like a very interesting companion for that!

For simplicity of the use, could you consider a possibility of supporting the service configurations fully straight in the div and using the same languages object for all iframes?

That would make using this plugin a much more versatile in situations when the client can update their content through some CMS and add new embeds as they wish. The current way of defining each embed separately on config is hard to achieve, when embeds change dynamically. With the approach of supporting all configs in the div, it would become much more easier just to do some regexp and replace the iframe embed with the div.

timiwahalahti avatar Sep 08 '21 08:09 timiwahalahti

Hi @timiwahalahti,

I totally get your point.

Moving the options into the div isn't an issue, what bugs me is that in the future releases I'm planning on adding support for “custom widgets” (Twitter, Facebook etc.) which (very likely) would require even more options for a correct configuration, and putting everything into the div would make it look quite “overwhelming”, there would be a lot of repeated code.

My proposed solution is to implement something like:

  • .addService() method which would allow you to configure additional services on demand (iframemanager must be loaded)
  • window.ifaramemanagerServices = [{my_service}] - a global configuration parameter set before the plugin's execution

Regarding the languages objects, sure I can add a global languages option!

orestbida avatar Sep 11 '21 15:09 orestbida