Add new ImportMapScriptElement #845
Conversation
ImportMapScriptElement renders a json importmap which allows to tell browsers how to resolve javascript dependencies. Signed-off-by: Sylvain Rabot <[email protected]>
|
|
||
| // NewImportMapScriptElement creates a new ImportMapScriptElement. | ||
| // The data must be a valid JSON string. | ||
| func NewImportMapScriptElement(id string, data string) ImportMapScriptElement { |
There was a problem hiding this comment.
to stick with the convention in jsonscript.go, the signature of this would be ImportMapScript(...) ImportMapScriptElement.
also, there is no indication that this function panics in the signature. if a function might panic, it is usually best practice to name the function Must[original func name].
however, dealing with the panicing is probably a moot point as this logic should likely be handled in Render.
There was a problem hiding this comment.
is this not just a special case of a JSONScript where type="importmap" and data is of type ImportMap? if so, we should simply reuse the logic here.
There was a problem hiding this comment.
The purpose of ImportMapScriptElement is to avoid Marshaling JSON data every render.
Important
This is a first draft, looking for inputs/insights.
ImportMapScriptElementrenders a json importmap which allows to tell browsers how to resolve javascript dependencies.