glenux-contrib/metamaps--metamaps
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

changed some disposeLabels to hideLabels - fix #156

Browse source
This commit is contained in:
Devin Howard 2013-03-14 22:55:28 -04:00
parent 1c3aa6c973
commit 86895836a0
1 changed files with 7 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
app/assets/javascripts/Jit/find.js
View file

@ -149,7 +149,9 @@ function onCanvasSearch(searchQuery, mapID, mapperID) {
function clearCanvas() { function clearCanvas() {
Mconsole.graph.eachNode(function(n) { Mconsole.graph.eachNode(function(n) {
Mconsole.graph.removeNode(n.id); Mconsole.graph.removeNode(n.id);
Mconsole.labels.disposeLabel(n.id); //TODO shouldn't we use disposeLabel? Yes, but it breaks things so it's
//hide for now
Mconsole.labels.hideLabel(n.id);
}); });
Mconsole.plot(); Mconsole.plot();
} }
@ -164,8 +166,7 @@ function clearCanvasExceptRoot() {
ids.forEach(function(id, index) { ids.forEach(function(id, index) {
if (id != root.id) { if (id != root.id) {
Mconsole.graph.removeNode(id); Mconsole.graph.removeNode(id);
//don't use disposeLabel or they'll never come back! //TODO is hideLabel correct? Maybe it is...
//maybe there's a better way that recreates the labels later??
Mconsole.labels.hideLabel(id); Mconsole.labels.hideLabel(id);
$('#topic_' + id + '_label').hide(); $('#topic_' + id + '_label').hide();
} }
@ -177,7 +178,9 @@ function clearFoundData() {
Mconsole.graph.eachNode( function(n) { Mconsole.graph.eachNode( function(n) {
if (n.getData('greenCircle') === true) { if (n.getData('greenCircle') === true) {
Mconsole.graph.removeNode(n.id); Mconsole.graph.removeNode(n.id);
Mconsole.labels.disposeLabel(n.id); //TODO is hideLabel correct? Maybe it is...
Mconsole.labels.hideLabel(n.id);
$('#topic_' + n.id + '_label').hide();
} }
}); });
Mconsole.plot(); Mconsole.plot();