Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #11302 from Snuffleupagus/addLinkAttributes-switch-target

Browse Source 
Slightly re-factor setting of the link `target` in `addLinkAttributes`
This commit is contained in:
Tim van der Meij 2019-11-02 23:12:11 +01:00 committed by GitHub
commit e7f24bd5b1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 18 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
src/display/display_utils.js
View File

@ -326,14 +326,6 @@ const LinkTarget = {
TOP: 4, TOP: 4,
}; };
const LinkTargetStringMap = [
'',
'_self',
'_blank',
'_parent',
'_top',
];
/** /**
* @typedef ExternalLinkParameters * @typedef ExternalLinkParameters
* @typedef {Object} ExternalLinkParameters * @typedef {Object} ExternalLinkParameters
@ -366,10 +358,24 @@ function addLinkAttributes(link, { url, target, rel, enabled = true, } = {}) {
}; };
} }
const LinkTargetValues = Object.values(LinkTarget); let targetStr = ''; // LinkTarget.NONE
const targetIndex = switch (target) {
LinkTargetValues.includes(target) ? target : LinkTarget.NONE; case LinkTarget.NONE:
link.target = LinkTargetStringMap[targetIndex]; break;
case LinkTarget.SELF:
targetStr = '_self';
break;
case LinkTarget.BLANK:
targetStr = '_blank';
break;
case LinkTarget.PARENT:
targetStr = '_parent';
break;
case LinkTarget.TOP:
targetStr = '_top';
break;
}
link.target = targetStr;
link.rel = (typeof rel === 'string' ? rel : DEFAULT_LINK_REL); link.rel = (typeof rel === 'string' ? rel : DEFAULT_LINK_REL);
} }