From bc56a01948d109e5f3d7ccc03fc9d54034057f05 Mon Sep 17 00:00:00 2001 From: Alan Huang Date: Thu, 9 Aug 2012 15:19:02 -0700 Subject: [PATCH] Support data-symbol-name in symbol linking Summary: See D3226. Test Plan: Browse Diffusion. Click on things. Reviewers: epriestley Reviewed By: epriestley CC: nh, aran, Korvin Maniphest Tasks: T1602 Differential Revision: https://secure.phabricator.com/D3227 --- .../js/application/repository/repository-crossreference.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/webroot/rsrc/js/application/repository/repository-crossreference.js b/webroot/rsrc/js/application/repository/repository-crossreference.js index 6a8c4c6078..076a535d1b 100644 --- a/webroot/rsrc/js/application/repository/repository-crossreference.js +++ b/webroot/rsrc/js/application/repository/repository-crossreference.js @@ -25,7 +25,6 @@ JX.behavior('repository-crossreference', function(config) { while (target !== document.body) { if (JX.DOM.isNode(target, 'span') && (target.className in map)) { var symbol = target.textContent || target.innerText; - var uri = JX.$U('/diffusion/symbol/' + symbol + '/'); var query = { lang : config.lang, projects : config.projects.join(','), @@ -37,6 +36,10 @@ JX.behavior('repository-crossreference', function(config) { if (target.hasAttribute('data-symbol-context')) { query.context = target.getAttribute('data-symbol-context'); } + if (target.hasAttribute('data-symbol-name')) { + symbol = target.getAttribute('data-symbol-name'); + } + var uri = JX.$U('/diffusion/symbol/' + symbol + '/'); uri.addQueryParams(query); window.open(uri); e.kill();