Skip to content

John McEleney / casperjs

Go to a project
Toggle navigation
Toggle navigation pinning
Commit 977f74e0 977f74e006ec78c70d8a137395d3a2bd33b5338d by Reid Lynch
Options

fix for visible/waitUntilVisible when selector matches multiple elements

1 parent b87c8630
Inline Side-by-side
Showing 5 changed files with 21 additions and 10 deletions
......@@ -777,12 +777,14 @@
*/
this.visible = function visible(selector) {
try {
var comp,
el = this.findOne(selector);
var el, comp,
elems = this.findAll(selector);
if (el) {
for(var i = 0; i < elems.length; i++)
{
el = elems[i];
comp = window.getComputedStyle(el, null);
return comp.visibility !== 'hidden' && comp.display !== 'none' && el.offsetHeight > 0 && el.offsetWidth > 0;
if (comp.visibility !== 'hidden' && comp.display !== 'none' && el.offsetHeight > 0 && el.offsetWidth > 0) return true;
}
return false;
} catch (e) {
......
......@@ -10,8 +10,8 @@
</script>
</head>
<body>
<img src="images/phantom.png" id="img1">
<img src="images/phantom.png" id="img2" style="display:none">
<img src="images/phantom.png" id="img1" style="display:none">
<img src="images/phantom.png" id="img2">
<img src="images/phantom.png" id="img3" style="visibility:hidden">
</body>
</html>
\ No newline at end of file
......
......@@ -15,6 +15,8 @@
Loading...
</div>
<div id="encoded">Voil&agrave;</div>
<p style="display:none;">I'm hidden.</p>
<p>I'm visible.</p>
<script>
setTimeout(function() {
var li = document.createElement('li')
......
/*global casper*/
/*jshint strict:false*/
casper.test.begin('visibility tests', 4, function(test) {
casper.test.begin('visibility tests', 5, function(test) {
casper.start('tests/site/visible.html', function() {
test.assert(this.visible('#img1'), 'Casper.visible() can detect if an element is visible');
test.assert(!this.visible('#img2'), 'Casper.visible() can detect if an element is invisible');
test.assert(!this.visible('#img1'), 'Casper.visible() can detect if an element is invisible');
test.assert(this.visible('#img2'), 'Casper.visible() can detect if an element is visible');
test.assert(!this.visible('#img3'), 'Casper.visible() can detect if an element is invisible');
test.assert(this.visible('img'), 'Casper.visible() can detect if an element is visible');
this.waitWhileVisible('#img1', function() {
test.pass('Casper.waitWhileVisible() can wait while an element is visible');
}, function() {
......
......@@ -117,7 +117,7 @@ casper.test.begin('waitForSelectorTextChange() tests', 1, function(test) {
});
});
casper.test.begin('waitUntilVisible() tests', 1, function(test) {
casper.test.begin('waitUntilVisible() tests', 2, function(test) {
casper.start('tests/site/waitFor.html');
casper.waitUntilVisible('li:nth-child(4)', function() {
......@@ -126,6 +126,12 @@ casper.test.begin('waitUntilVisible() tests', 1, function(test) {
test.fail('Casper.waitUntilVisible() waits for a selector being visible');
});
casper.waitUntilVisible('p', function() {
test.pass('Casper.waitUntilVisible() waits for a selector being visible');
}, function() {
test.fail('Casper.waitUntilVisible() waits for a selector being visible');
});
casper.run(function() {
test.done();
});
......