Skip to content

Commit c18c86b

Browse files
committed
Merge pull request #6 from proll/hide-remove-selectors-all
Hide remove selectors all
2 parents a2d82d5 + 900d2f5 commit c18c86b

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

capture/genBitmaps.js

+7-3
Original file line numberDiff line numberDiff line change
@@ -80,17 +80,21 @@ function capturePageSelectors(url,grabConfigs,viewports,bitmaps_reference,bitmap
8080
//HIDE SELECTORS WE WANT TO AVOID
8181
grabConfig.hideSelectors.forEach(function(o,i,a){
8282
casper.evaluate(function(o){
83-
document.querySelector(o).style.visibility='hidden';
83+
Array.prototype.forEach.call(document.querySelectorAll(o), function(s, j){
84+
s.style.visibility='hidden';
85+
});
8486
},o);
8587
});
8688

8789
//REMOVE UNWANTED SELECTORS FROM RENDER TREE
8890
grabConfig.removeSelectors.forEach(function(o,i,a){
8991
casper.evaluate(function(o){
90-
document.querySelector(o).style.display='none';
92+
Array.prototype.forEach.call(document.querySelectorAll(o), function(s, j){
93+
s.style.display='none';
94+
});
9195
},o);
9296
});
93-
97+
9498
//CREATE SCREEN SHOTS AND TEST COMPARE CONFIGURATION (CONFIG FILE WILL BE SAVED WHEN THIS PROCESS RETURNS)
9599
grabConfig.selectors.forEach(function(o,i,a){
96100
var cleanedSelectorName = o.replace(/[^a-zA-Z\d]/,'');//remove anything that's not a letter or a number

0 commit comments

Comments
 (0)