Firefox

17302 readers

543 users here now

A place to discuss the news and latest developments on the open-source browser Firefox

founded 4 years ago

MODERATORS

[email protected]

201

Google rolls back reCaptcha update to fix Firefox issues (www.bleepingcomputer.com)

submitted 4 months ago by pro_grammer to c/[email protected]

23 comments fedilink hide all child comments

you are viewing a single comment's thread
view the rest of the comments

[–] [email protected] 15 points 4 months ago* (last edited 4 months ago) (2 children)

The script attempted to modify a div's background color using document.body.removeChild, but as the script was loaded in the HTML head, the DOM had not loaded

Isn't that how it works/always worked? When i was learning html/js ages ago i had to use some event listener (DOMContentLoaded i think) or put the script just before </body> (for any code that should run on load and interacts with the DOM).

And how do you change the background by removing a child?

[–] [email protected] 3 points 4 months ago

And how do you change the background by removing a child?

The removed child could also have a background color, and it could span the entire area of the parent element.
But it's weird because the quote says "modify a div's background color", and this way you don't actually modify that, but only it's appearance.

Or maybe it's done with some CSS trickery, looking for a specific child in it's selector?

[–] [email protected] 3 points 4 months ago

Yeah, they should be listening for an event, not just YOLOing it in the head, that's just racey...