extern/egroupware
1
0
Fork 1
mirror of https://github.com/EGroupware/egroupware.git synced 2024-11-07 08:34:42 +01:00
Code Issues Packages Projects Releases Wiki Activity

Et2Description: Fix links were not clickable

Browse Source 
activateLinks property got typoed to activeLinks in the template conversion
This commit is contained in:
nathan 2023-02-01 14:31:17 -07:00
parent 346a720d27
commit 1fa9d91d59
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
api/js/etemplate/Et2Description/Et2Description.ts
View File

@ -49,7 +49,7 @@ export class Et2Description extends Et2Widget(LitElement) implements et2_IDetach
* Scan the value, and if there are any links (URL, mailto:) then wrap them in a clickable
* <a/> tag
*/
activeLinks: {
activateLinks: {
type: Boolean,
reflect: true
},
@ -88,7 +88,7 @@ export class Et2Description extends Et2Widget(LitElement) implements et2_IDetach
super();
// Initialize properties
this.activeLinks = false;
this.activateLinks = false;
this.extraLinkPopup = "";
this.extraLinkTarget = "_browser";
// Don't initialize this to avoid href(unknown) when rendered
@ -148,7 +148,7 @@ export class Et2Description extends Et2Widget(LitElement) implements et2_IDetach
super.requestUpdate(...arguments);
// Due to how we do the rendering into the light DOM (not sure it's right) we need this after
// value change or it won't actually show up
if(["value", "href", "activeLinks"].indexOf(attribute) != -1 && this.parentNode)
if(["value", "href", "activateLinks"].indexOf(attribute) != -1 && this.parentNode)
{
this.updateComplete.then(() => render(this._renderContent(), <HTMLElement><unknown>this));
}
@ -171,7 +171,7 @@ export class Et2Description extends Et2Widget(LitElement) implements et2_IDetach
render = this.wrapLink(this.href, this.value);
}
// If we want to activate links inside, do that
else if(this.activeLinks && this.value)
else if(this.activateLinks && this.value)
{
render = this.getActivatedValue(this.value, this.href ? this.extraLinkTarget : '_blank');
}