Fix #134 #135 #136

[alebianco-doxee]

This PR contains a:

• bugfix
• new feature
• code refactor
• test update
• typo fix
• metadata update

Motivation / Use-Case

Fixes NoSleep not working in recent iOS versions with the solution found by @z-x
#135
#136

Also fixes NoSleep not working on Samsung Browser due to permission denied error
#134

[z-x]

Thanks!

[Moongazer]

It would be amazing if this fix is merged soon into a 0.12.1 patch version or so. We've also quite a lot of iOS users recently complaining that the screen goes black while using our App.
Thanks for your work guys!

[CodeCommander]

I can confirm this fixed NoSleep.js on iOS 15.1 in my app.

[zamarawka]

@richtr Any way to merge it?

[Romander]

Hey guys, it will be perfect if it will be merged 🙌

[dcbarans]

@zamarawka it looks like @richtr has made zero contributions this year.. I wonder if this is a dead repo :(

[Uriopass]

Since the owner seems dead, I republished it as another package under @uriopass/nosleep.js with version 0.12.1

[timtribers]

@Uriopass @alebianco-doxee @z-x @Moongazer @huppfi @CodeCommander @zamarawka @Romander @dcbarans - please, please, please will one of you kind people consider taking this on and creating an "official" fork which we can all come together around? The functionality of nosleep is simply too important not to have a maintained solution (and @richtr definitely seems to have disappeared :( ) - otherwise we are going to end up with a 1000 forks!

If it helps, I am sure some sponsor/patreon/etc. income could be generated for an active repo.

[kylemacfarlane]

The fix from #135 breaks Firefox if the script is loaded before the body tag. Appending to body needs to be delayed with something like:

document.addEventListener("DOMContentLoaded", function() { 
  document.querySelector("body").append(this.noSleepVideo);
});

[ezekiel747]

Since the owner seems dead, I republished it as another package under @uriopass/nosleep.js with version 0.12.1

Awesome, thank you!
Could you please change the installation instructions in the readme to include the updated package name?

[Uriopass]

Since the owner seems dead, I republished it as another package under @uriopass/nosleep.js with version 0.12.1

Awesome, thank you! Could you please change the installation instructions in the readme to include the updated package name?

Done :)

[ezekiel747]

I've checked and the Readme file still contains:
This package is published to npm as nosleep.js and can be installed with:
npm install nosleep.js

[Uriopass]

Ah sorry you're talking about the npm page! I'll fix this

EDIT: Done

[Noitidart]

Isn't this not safe? On new Samsungs it might work. Is it possible to execute it, and in case the promise rejects, then we should be like "ok this samsung doesn't allow it, so then fallback"?

[Noitidart]

Does this need to go into document.onload? Or should we check if document.readyState === 'complete'?

[SUCHiDEV]

@Uriopass will you update your version?

[Uriopass]

I don't see what's there to update

[MasterInQuestion]

    "position: absolute; top: -100%" alone should suffice.
    Demo (bookmarklet): javascript: void document.getElementsByTagName( "body" )[0].appendChild( document.createRange().createContextualFragment( "<div style='position: absolute; top: -100%; background-color: #F00'>XXXXXXXX</div>" ) );

    Another potential workaround: "overflow: hidden; position: absolute; top: 0; margin: 0; padding: 0; border-style: none; outline-style: none; width: 1px; height: 1px" (if the above failed to work)