Merge pull request #214 from zazuko/fix-er-213

Show graph name label
zazuko · Dec 4, 2023 · 42d2aee · 42d2aee
2 parents 73cf1f4 + c0576e5
commit 42d2aee
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/.changeset/pink-worms-suffer.md b/.changeset/pink-worms-suffer.md
@@ -0,0 +1,5 @@
+---
+"@zazuko/trifid-entity-renderer": patch
+---
+
+Fix issue where graph name label was not shown if it is not a known prefix (closes #213)
diff --git a/packages/entity-renderer/renderer/metadata/namedGraphs.js b/packages/entity-renderer/renderer/metadata/namedGraphs.js
@@ -17,7 +17,7 @@ const getLabel = (term) => {
  if (term.constructor.name === 'DefaultGraph') {
  return 'Default graph'
  }
- return shrink(term.value)
+ return shrink(term.value) || term.value
 }
 
 const getNamedGraphsCounts = (dataset) => {

diff --git a/packages/entity-renderer/renderer/web-component/N3Term.js b/packages/entity-renderer/renderer/web-component/N3Term.js
@@ -4,7 +4,7 @@ import { shrink } from '@zazuko/prefixes'
 
 function renderTerm (term) {
  if (term.termType === 'NamedNode') {
- return html`<a href="${term.value}">${shrink(term.value)}</a>`
+ return html`<a href="${term.value}">${shrink(term.value) || term.value}</a>`
  }
  if (term.constructor.name === 'DefaultGraph') {
  return html`Default graph`