@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
Link scoped symbols
Summary:
Elements with class na are now linked to symbol lookup. The
context is passed if we got one. See D3214.
Test Plan: Load Diffusion. Examine DOM and click on things.
Reviewers: epriestley
Reviewed By: epriestley
CC: nh, aran, Korvin
Maniphest Tasks: T1602
Differential Revision: https://secure.phabricator.com/D3215
+11
-4
+1
webroot/rsrc/css/core/syntax.css
+1
webroot/rsrc/css/core/syntax.css
+10
-4
webroot/rsrc/js/application/repository/repository-crossreference.js
+10
-4
webroot/rsrc/js/application/repository/repository-crossreference.js
···
21
21
return;
22
22
}
23
23
var target = e.getTarget();
24
-
var map = {nc : 'class', nf : 'function'};
24
+
var map = {nc : 'class', nf : 'function', na : null};
25
25
while (target !== document.body) {
26
26
if (JX.DOM.isNode(target, 'span') && (target.className in map)) {
27
27
var symbol = target.textContent || target.innerText;
28
28
var uri = JX.$U('/diffusion/symbol/' + symbol + '/');
29
-
uri.addQueryParams({
30
-
type : map[target.className],
29
+
var query = {
31
30
lang : config.lang,
32
31
projects : config.projects.join(','),
33
32
jump : true
34
-
});
33
+
};
34
+
if (map[target.className]) {
35
+
query.type = map[target.className];
36
+
}
37
+
if (target.hasAttribute('data-symbol-context')) {
38
+
query.context = target.getAttribute('data-symbol-context');
39
+
}
40
+
uri.addQueryParams(query);
35
41
window.open(uri);
36
42
e.kill();
37
43
break;