fix: update OpenAttestationEmbeddedRenderer to EmbeddedRenderer#25
fix: update OpenAttestationEmbeddedRenderer to EmbeddedRenderer#25
Conversation
index.html
Outdated
|
|
||
| <section> | ||
| <h4>OpenAttestationEmbeddedRenderer</h4> | ||
| <h4>DecentralizedRenderer</h4> |
There was a problem hiding this comment.
A naming question here. What about the renderer itself makes it decentralized?
From the description, it sounds like this is a renderer that uses HTML in an embedded or nested browsing context, in particular, it suggests an iframe must be used. If the latter is true (and intended to be true carrying forward), perhaps IFrameRenderer would be an appropriate name to describe why/how this renderer is different from the other renderer types.
If there's some other layer to the abstraction here and use of an iframe is an implementation detail, perhaps it suggests the name ought to be ExternalRenderer or EmbeddedRenderer -- where there is some External/Embedded "ACTION" API to communicate with it?
If the main feature of this type of renderer is that it has this "ACTION" API communication capability, perhaps the name should be based on that, e.g., ActionableRenderer or ActionDrivenRenderer (or similar). Perhaps those names are slightly awkard or insufficiently specific -- but I wouldn't suggest InteractiveRenderer since "interactivity" could be a property of many different renderer types.
There was a problem hiding this comment.
Thanks for the thoughtful feedback!
After some discussion within the team, we agree that the name should stay close to how the renderer actually works. Based on your suggestions, we think EmbeddedRenderer would be the most accurate and intuitive choice, since it reflects that the rendering happens in an embedded context.
We’ll proceed with updating the naming accordingly.
|
Hi @dlongley, Thanks for the feedback, have updated it to be EmbeddedRenderer instead. |
There was a problem hiding this comment.
We will need to figure out the context issue in this PR.
- The PR should be updated to refer to the new issue on the topic.
- The images need descriptions for screen readers (for people that have low/no-vision sight needs). I have used LLMs to generate these descriptions and they do a pretty good job (sometimes) if you give them enough context on what the image is showing.
|
Issue #33 (to deal with the |
|
This was discussed during the vcwg meeting on 14 November 2025. View the transcriptw3c/vc-render-method#25 and w3c/vc-render-method#30dmitriz: those were discussed previously brent: do you mean the core data model? dmitriz: no, the render spec brent: to clarify: this method is introducing things for a specific render method dmitriz: correct phila: reminds me of a discussion I had a long time ago with Mark Nottingham dmitriz: my understanding of the PR is that it proposes a "super class" of OCABundle phila: that feels right manu_: the current way the specification is structured is: we have an extension mechanism, called render method manu_: I don't want us to take 2 years to get Render Methods 1.0 published phila: difference between linking to a render method and embedding JoeAndrieu: we want to get one, to be able to go to CR dmitriz: this is a good segway to look at PR 36, which emphasized that the spec is an extension mechanism (as manu_ and JoeAndrieu said) <dmitriz> we've implemented the HTML template, at MIT DCC. (we started with SVG, but ran into line wrapping probems, and switched to HTML) dmitriz: SVG and HTML are widely applicable, but maybe not the ones with the most implementations phila: as GS1, we use SVG and would like SVG to be in there dmitriz: dare we bring up the dreaded word "registry"? |
Rename "OpenAttestationEmbeddedRenderer" to "EmbeddedRenderer".
As there is an update to OA.
Preview | Diff