Hidden elements #1
Comments
ghost
assigned 
|
I think I'll start off with the simplest approach to this; properties = ele.style.match /(display|visibility)\s*:\s*[a-z]+/gi
visible = yes
if properties?
for property in properties
visible = not /(none|hidden)\s*$/i.test propertyThis should, at least for now, detect obvious hidden elements. |
neocotic
added a commit
that referenced
this issue
Dec 10, 2012
neocotic
added a commit
that referenced
this issue
Dec 10, 2012
|
This works as expected and I'll be sure to unit test it as part of #3. |
|
You, sir, are a BOSS.. BOSS |
|
sqrt(4096) SSdkIGxpa2UgdG8gY2hhdCB3aXRoIHlvdS4gQ291bGQgeW91IGRyb3AgbWUgYSBtYWls aWFtQGdteC5pdA== |
neocotic
added a commit
that referenced
this issue
Mar 12, 2018
* switched to ES5 to avoid having to compile using babel * simplified import for europa-core * roll 4.0.0-alpha9 after big rewrite to be compatible with changes to europa-core * node v6 is now a minimum requirement due to jsdom dependency * roll 4.0.0-alpha10 after bumping europa-core changes * roll 4.0.0-alpha11 * roll 4.0.0-alpha12 * roll 4.0.0-alpha13 * roll 4.0.0-alpha14 * roll 4.0.0
neocotic
added a commit
that referenced
this issue
Mar 13, 2018
* switched to ES5 to avoid having to compile using babel * fixed bug with previous Map method reference * moved to !ninja and another big refactor of code * added slack notifications to travis * roll 4.0.0-alpha9 * roll 4.0.0-alpha10 * fixed bug where window not passed to DOMUtilites.isVisible and rolled 4.0.0-alpha11 * roll 4.0.0-alpha12 after fixing issue with detecting styles on detached DOM * roll 4.0.0-alpha13 after finding bug with inherited visibility on temp wrapper for detatched elements * simplified plugins and presets * changed load point for default preset to better support bundlers like rollup * roll 4.0.0-alpha16 after renaming destroy method to release * roll 4.0.0
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Although detached or generated elements cannot have their styles computed and therefor visibility determined, the
style.displayproperty should at least be checked.Also, it's probably worth while checking the
visibilityproperty.The text was updated successfully, but these errors were encountered: