'Hidden' text contributes to score

poblish / bannedList-Plugin

#BannedList Plugin

3 stars 1 forks source link

'Hidden' text contributes to score #28

Open poblish opened 12 years ago

poblish commented 12 years ago

See: http://planet.jboss.org/post/8_2_million_messages_second_with_specjms ... and click the 'Search' tab to see the hidden text.

It's confusing to show a high score when several banned terms are hidden - unfortunately I haven't found a way of determining that those terms are 'hidden'. All the usual checks (e.g. http://stackoverflow.com/a/1343350/954442) report that they are visible.

Annoying, but not a big deal.

poblish commented 12 years ago

www.theaustralian.com.au is also a big pain in this regard.

poblish commented 12 years ago

Fix might be a little over-zealous, but I'm pretty happy with it.

poblish commented 12 years ago

Reopened for Firefox because a nasty FF/JQuery bug in the theJQNode.css('display') === 'none' check completely breaks that version. Have had to comment the check out for now, but it should be restored as and when safe.

See http://bit.ly/JZfHlp