How to add a new property to a img? about scala-dom-types HOT 6 CLOSED

Sure, please see my comments on that PR, it needs some small changes.

I am not sure what the Calico alternative to:

val crossOrigin: HtmlProp[String] = htmlProp("crossOrigin", StringAsIsCodec)

syntax is, it probably looks very similar, but it depends on their classes and method names.

from scala-dom-types.

Hi, if you just want to add it to your laminar project, you can just say:

val crossOrigin: HtmlProp[String] = htmlProp("crossOrigin", StringAsIsCodec)

(stringProp is not accessible to users, IIRC)

If you want to add it to SDT, you need to create a record for it in ReflectedHtmlAttrDefs.scala, with property name "crossOrigin" and attr name "crossorigin". If you're not up for that, I'll do it eventually.

MDN link https://developer.mozilla.org/en-US/docs/Web/HTML/Attributes/crossorigin

from scala-dom-types.

Hi, if you just want to add it to your laminar project, you can just say:

val crossOrigin: HtmlProp[String] = htmlProp("crossOrigin", StringAsIsCodec)

(stringProp is not accessible to users, IIRC)

If you want to add it to SDT, you need to create a record for it in ReflectedHtmlAttrDefs.scala, with property name "crossOrigin" and attr name "crossorigin". If you're not up for that, I'll do it eventually.

MDN link https://developer.mozilla.org/en-US/docs/Web/HTML/Attributes/crossorigin

Hi @raquo

like this? #100

If appropriate, could you help approve this pr?

Or what else do I need to add....

Actually, I hope to use the newly added attr in Calico's DSL

Perhaps I should submit a PR to update Calico's dependencies scala-dom-types after making changes in this repository
Finally, use the latest Calico version in my project?

from scala-dom-types.

Sure, please see my comments on that PR, it needs some small changes.

I am not sure what the Calico alternative to:

val crossOrigin: HtmlProp[String] = htmlProp("crossOrigin", StringAsIsCodec)

syntax is, it probably looks very similar, but it depends on their classes and method names.

updated pr and need approve again

from scala-dom-types.

Ok, merged, thanks. I can make a release later this week, I think. If you need this sooner, you really should figure out how to add custom attributes in your Calico project, it should be a simple one-liner. Just so that you're not blocked by the SDT release process.

from scala-dom-types.

This is a good suggestion. Thank you for your help!

from scala-dom-types.