-
Notifications
You must be signed in to change notification settings - Fork 6
fix: update OpenAttestationEmbeddedRenderer to EmbeddedRenderer #25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
index.html
Outdated
|
|
||
| <section> | ||
| <h4>OpenAttestationEmbeddedRenderer</h4> | ||
| <h4>DecentralizedRenderer</h4> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
dmitrizagidulin
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
|
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