GUI -- TopoView - implemented show/hide background map ('B' keystroke)
- reworked SvgUtilService.makeVisible() to be dual-purpose .visible(el, [b]) method.
Change-Id: I8e68994627d1022066cdc9dcbed006675431d91a
diff --git a/web/gui/src/main/webapp/tests/app/fw/svg/svgUtil-spec.js b/web/gui/src/main/webapp/tests/app/fw/svg/svgUtil-spec.js
index 69fe750..6fb1a60 100644
--- a/web/gui/src/main/webapp/tests/app/fw/svg/svgUtil-spec.js
+++ b/web/gui/src/main/webapp/tests/app/fw/svg/svgUtil-spec.js
@@ -41,7 +41,7 @@
it('should define api functions', function () {
expect(fs.areFunctions(sus, [
'createDragBehavior', 'loadGlow', 'cat7', 'translate', 'stripPx',
- 'makeVisible', 'safeId'
+ 'safeId', 'visible'
])).toBeTruthy();
});
@@ -114,15 +114,17 @@
expect(sus.stripPx('4px')).toEqual('4');
});
- // === makeVisible()
+ // === visible()
it('should hide and show an element', function () {
var r = svg.append('rect');
- sus.makeVisible(r, false);
+ sus.visible(r, false);
expect(r.style('visibility')).toEqual('hidden');
+ expect(sus.visible(r)).toBe(false);
- sus.makeVisible(r, true);
+ sus.visible(r, true);
expect(r.style('visibility')).toEqual('visible');
+ expect(sus.visible(r)).toBe(true);
});
});