Night Mode Addon compatible with 27+?
Moderators: FranklinDM, Lootyhoof
- Pallid Planetoid
- Knows the dark side
- Posts: 4279
- Joined: 2015-10-06, 16:59
- Location: Los Angeles CA USA
Re: Night Mode Addon compatible with 27+?
Hi JustOff --
Suggestion: What do you think of providing users with an option to turn on/off "Nite Mode" during a specific set time-range?
What I would perhaps be doing myself is have "Nite Mode" off from say 600 to 1800 hours (day-time) and the remaining 24 hours 1800 to 600 hours the next day (night-time) have "Nite Mode" on.
Even better yet also include a "daylight savings time adjustment" option so the time-ranges adjust an hour either way back-and-forth accordingly to accommodate the two time adjustments that occur during the year.
Of course this last suggestion would have to be optional (possibly adjustable as well in the event the dates change however as of now all States that practice this are identical as illustrated in the screen-shot) since this is applicable only to the U.S. and not all States apply as well (i.e. Arizona and Hawaii): This extension has gotten very powerful with additional handy options -- adding this would provide even more usable/practical options...
Suggestion: What do you think of providing users with an option to turn on/off "Nite Mode" during a specific set time-range?
What I would perhaps be doing myself is have "Nite Mode" off from say 600 to 1800 hours (day-time) and the remaining 24 hours 1800 to 600 hours the next day (night-time) have "Nite Mode" on.
Even better yet also include a "daylight savings time adjustment" option so the time-ranges adjust an hour either way back-and-forth accordingly to accommodate the two time adjustments that occur during the year.
Of course this last suggestion would have to be optional (possibly adjustable as well in the event the dates change however as of now all States that practice this are identical as illustrated in the screen-shot) since this is applicable only to the U.S. and not all States apply as well (i.e. Arizona and Hawaii): This extension has gotten very powerful with additional handy options -- adding this would provide even more usable/practical options...
Current Pale Moon(x86) Release | WIN10 | I5 CPU, 1.7 GHz, 6GB RAM, 500GB HD[20GB SSD]
Formerly user Pale Moon Rising - to provide context involving embedded reply threads.
Good judgment comes from experience and a lot of that comes from bad judgment. - Will Rogers
Knowing Pale Moon is indisputably #1 is defined by knowing the totality of browsers. - Pale Moon Rising
Formerly user Pale Moon Rising - to provide context involving embedded reply threads.
Good judgment comes from experience and a lot of that comes from bad judgment. - Will Rogers
Knowing Pale Moon is indisputably #1 is defined by knowing the totality of browsers. - Pale Moon Rising
Re: Night Mode Addon compatible with 27+?
If it would be getting the time from the OS, that shouldn't be necessary. Your OS should adjust the time accordingly for daylight savings time changes.Pale Moon Rising wrote:Even better yet also include a "daylight savings time adjustment" option
Nichi nichi kore ko jitsu = Every day is a good day.
-
- Fanatic
- Posts: 137
- Joined: 2017-09-11, 14:28
Re: Night Mode Addon compatible with 27+?
Daylight saving time is unfortunately not a USA-only thing. Anyway I don’t think it would be a useful option as we usually use computers indoors with artificial lighting.
- Pallid Planetoid
- Knows the dark side
- Posts: 4279
- Joined: 2015-10-06, 16:59
- Location: Los Angeles CA USA
Re: Night Mode Addon compatible with 27+?
Hmm, based on your conclusion there would then be no need for the actual extension ("Nite Mode") itself.... (Note: for purposes of clarity -- I bolded the part myself in the post that is relevant to my reply)tooshorttoolong wrote:Daylight saving time is unfortunately not a USA-only thing. Anyway I don’t think it would be a useful option as we usually use computers indoors with artificial lighting.
In other words you're making the argument that the extension is superfluous...
Edit: thanks to tooshorttoolong pointer -- I've corrected my improper use of the word "your" which should have been the contraction "you're" (a bad habit of mine).
Last edited by Pallid Planetoid on 2018-06-23, 19:24, edited 1 time in total.
Current Pale Moon(x86) Release | WIN10 | I5 CPU, 1.7 GHz, 6GB RAM, 500GB HD[20GB SSD]
Formerly user Pale Moon Rising - to provide context involving embedded reply threads.
Good judgment comes from experience and a lot of that comes from bad judgment. - Will Rogers
Knowing Pale Moon is indisputably #1 is defined by knowing the totality of browsers. - Pale Moon Rising
Formerly user Pale Moon Rising - to provide context involving embedded reply threads.
Good judgment comes from experience and a lot of that comes from bad judgment. - Will Rogers
Knowing Pale Moon is indisputably #1 is defined by knowing the totality of browsers. - Pale Moon Rising
- Pallid Planetoid
- Knows the dark side
- Posts: 4279
- Joined: 2015-10-06, 16:59
- Location: Los Angeles CA USA
Re: Night Mode Addon compatible with 27+?
Excellent point! (I had to think about that for a moment )Nigaikaze wrote:If it would be getting the time from the OS, that shouldn't be necessary. Your OS should adjust the time accordingly for daylight savings time changes.Pale Moon Rising wrote:Even better yet also include a "daylight savings time adjustment" option
Addendum: but then I was thinking in terms the longer days -- but then that changes as time progresses in and out of the two time settings so nothing is going to be perfect -- and with that in mind your point remains a good one.
Last edited by Pallid Planetoid on 2018-06-23, 18:41, edited 1 time in total.
Current Pale Moon(x86) Release | WIN10 | I5 CPU, 1.7 GHz, 6GB RAM, 500GB HD[20GB SSD]
Formerly user Pale Moon Rising - to provide context involving embedded reply threads.
Good judgment comes from experience and a lot of that comes from bad judgment. - Will Rogers
Knowing Pale Moon is indisputably #1 is defined by knowing the totality of browsers. - Pale Moon Rising
Formerly user Pale Moon Rising - to provide context involving embedded reply threads.
Good judgment comes from experience and a lot of that comes from bad judgment. - Will Rogers
Knowing Pale Moon is indisputably #1 is defined by knowing the totality of browsers. - Pale Moon Rising
-
- Fanatic
- Posts: 137
- Joined: 2017-09-11, 14:28
Re: Night Mode Addon compatible with 27+?
You have a very flawed logic. (And also, it’s written "you’re", not "your".)Pale Moon Rising wrote:In other words your making the argument that the extension is superfluous...
- Pallid Planetoid
- Knows the dark side
- Posts: 4279
- Joined: 2015-10-06, 16:59
- Location: Los Angeles CA USA
Re: Night Mode Addon compatible with 27+?
Off-topic:
That said, I would like to know where my logic is flawed as well... I'm certainly capable of being wrong as already proven -- so I welcome the help regarding my alleged improper logic. Thanks for the tip --- I fixed the post accordingly and I appreciate you pointing it out (is "you" alright there? -- I also often have the habit of leaving off the "r" in "your" even if I use the word correctly)tooshorttoolong wrote:You have a very flawed logic. (And also, it’s written "you’re", not "your".)Pale Moon Rising wrote:In other words your making the argument that the extension is superfluous...
Last edited by Pallid Planetoid on 2018-06-23, 19:27, edited 1 time in total.
Current Pale Moon(x86) Release | WIN10 | I5 CPU, 1.7 GHz, 6GB RAM, 500GB HD[20GB SSD]
Formerly user Pale Moon Rising - to provide context involving embedded reply threads.
Good judgment comes from experience and a lot of that comes from bad judgment. - Will Rogers
Knowing Pale Moon is indisputably #1 is defined by knowing the totality of browsers. - Pale Moon Rising
Formerly user Pale Moon Rising - to provide context involving embedded reply threads.
Good judgment comes from experience and a lot of that comes from bad judgment. - Will Rogers
Knowing Pale Moon is indisputably #1 is defined by knowing the totality of browsers. - Pale Moon Rising
Re: Night Mode Addon compatible with 27+?
Pale Moon Rising,
You are assuming (kind of) that people want to use Night Mode addon for a specifc period in a day and then switch it off for the rest of the day. I don't think that's how most people use it. People use it because they like dark theme on web pages (and their OS, editor etc.) and it does not have anything to do with the time of the day. I guess this is the reason for your argument with tooshorttoolong.
You are assuming (kind of) that people want to use Night Mode addon for a specifc period in a day and then switch it off for the rest of the day. I don't think that's how most people use it. People use it because they like dark theme on web pages (and their OS, editor etc.) and it does not have anything to do with the time of the day. I guess this is the reason for your argument with tooshorttoolong.
-
- Fanatic
- Posts: 137
- Joined: 2017-09-11, 14:28
Re: Night Mode Addon compatible with 27+?
I mean that the current time, or even position of the sun (like Redshift and F.lux use), is not really a good way to automate enabling a dark theme, because what matters is the ambient light, which will be different depending if it’s cloudy or not, if you’re inside or outside, if the curtains are pulled, if there is artificial light. Other factors could be if your eyes hurt and which web page you’re looking at. Ideally we’d need a light sensor that measures the light coming from behind the monitor.
- Pallid Planetoid
- Knows the dark side
- Posts: 4279
- Joined: 2015-10-06, 16:59
- Location: Los Angeles CA USA
Re: Night Mode Addon compatible with 27+?
Yes you're absolutely correct regarding my assumption (in some cases that is my conclusion -- which is why it would be an "optional" function) and certainly your alternative perspective is appreciated as well. That said, I would point out that the extensions name is after all "Nite Mode" which suggests that my assumption is arguably applicable. And as an "optional" function users are welcome to use the function or not as they see fit -- wouldn't you agree?. As I see it if a user wants to use the extension as a "Nite Mode" function which again is after all the name of the extension and as such an additional "optional" automatic "Nite Mode" optional function for the add-on seems to me to be a reasonable suggestion. (if you think about it a "Theme" is generally considered how a browser GUI is rendered and does not generally have to do with how websites get rendered (or am I wrong here?) -- this add-on involves changing the way web sites are rendered that is specifically in a more subdued way (reducing the frequent stark-white page rendering that is typical) and as such provides the user with a "Nite Mode" website alternative as I see it. (I may be misconstruing this -- and if so my apologies to all).adesh wrote:Pale Moon Rising,
You are assuming (kind of) that people want to use Night Mode addon for a specifc period in a day and then switch it off for the rest of the day. I don't think that's how most people use it. People use it because they like dark theme on web pages (and their OS, editor etc.) and it does not have anything to do with the time of the day. I guess this is the reason for your argument with tooshorttoolong.
Last edited by Pallid Planetoid on 2018-06-24, 11:15, edited 2 times in total.
Current Pale Moon(x86) Release | WIN10 | I5 CPU, 1.7 GHz, 6GB RAM, 500GB HD[20GB SSD]
Formerly user Pale Moon Rising - to provide context involving embedded reply threads.
Good judgment comes from experience and a lot of that comes from bad judgment. - Will Rogers
Knowing Pale Moon is indisputably #1 is defined by knowing the totality of browsers. - Pale Moon Rising
Formerly user Pale Moon Rising - to provide context involving embedded reply threads.
Good judgment comes from experience and a lot of that comes from bad judgment. - Will Rogers
Knowing Pale Moon is indisputably #1 is defined by knowing the totality of browsers. - Pale Moon Rising
- Pallid Planetoid
- Knows the dark side
- Posts: 4279
- Joined: 2015-10-06, 16:59
- Location: Los Angeles CA USA
Re: Night Mode Addon compatible with 27+?
Good points! -- that said, I'm using my computer right now (just as you are) and right now it is 3:52AM in my case and dark out including the room I'm in right now. Without "Nite Mode" enabled this page happens to be jarringly bright white on the other hand with "Nite Mode" enabled the age is far more subdued in this dark room and hence far easier on the eyes. Try it yourself in a dark room and you'll perhaps see what I mean (I typically leave the lights off when up using the computer as I find no need to have bright lights on -- I have a few very subdued night lights that produce very minimal light). So for people with my habits -- would you concede that perhaps a automatic "Nite Time" mode that coincides to day/night periods of a 24 hour day could be a welcome additional "optional' function? (and of course my suggestion is relative to the time-of-day and not relative to "Redshift and F.lux" -- whatever that is.... I think we are over-thinking the complexity of what my suggestion represents. It's a far more simple consideration that simply involves the difference between night and day .... but thanks for the feedback.tooshorttoolong wrote:I mean that the current time, or even position of the sun (like Redshift and F.lux use), is not really a good way to automate enabling a dark theme, because what matters is the ambient light, which will be different depending if it’s cloudy or not, if you’re inside or outside, if the curtains are pulled, if there is artificial light. Other factors could be if your eyes hurt and which web page you’re looking at. Ideally we’d need a light sensor that measures the light coming from behind the monitor.
Last edited by Pallid Planetoid on 2018-06-24, 11:04, edited 2 times in total.
Current Pale Moon(x86) Release | WIN10 | I5 CPU, 1.7 GHz, 6GB RAM, 500GB HD[20GB SSD]
Formerly user Pale Moon Rising - to provide context involving embedded reply threads.
Good judgment comes from experience and a lot of that comes from bad judgment. - Will Rogers
Knowing Pale Moon is indisputably #1 is defined by knowing the totality of browsers. - Pale Moon Rising
Formerly user Pale Moon Rising - to provide context involving embedded reply threads.
Good judgment comes from experience and a lot of that comes from bad judgment. - Will Rogers
Knowing Pale Moon is indisputably #1 is defined by knowing the totality of browsers. - Pale Moon Rising
Re: Night Mode Addon compatible with 27+?
Sorry, but this feature does not seem to me essential.Pale Moon Rising wrote:What do you think of providing users with an option to turn on/off "Nite Mode" during a specific set time-range?
- Pallid Planetoid
- Knows the dark side
- Posts: 4279
- Joined: 2015-10-06, 16:59
- Location: Los Angeles CA USA
Re: Night Mode Addon compatible with 27+?
^ Thanks for considering it though...
Current Pale Moon(x86) Release | WIN10 | I5 CPU, 1.7 GHz, 6GB RAM, 500GB HD[20GB SSD]
Formerly user Pale Moon Rising - to provide context involving embedded reply threads.
Good judgment comes from experience and a lot of that comes from bad judgment. - Will Rogers
Knowing Pale Moon is indisputably #1 is defined by knowing the totality of browsers. - Pale Moon Rising
Formerly user Pale Moon Rising - to provide context involving embedded reply threads.
Good judgment comes from experience and a lot of that comes from bad judgment. - Will Rogers
Knowing Pale Moon is indisputably #1 is defined by knowing the totality of browsers. - Pale Moon Rising
-
- Astronaut
- Posts: 650
- Joined: 2015-07-30, 20:29
- Location: Vaughan, ON, Canada
Re: Night Mode Addon compatible with 27+?
On my homebrew 32-bit PM28 builds ANM is on-again, off-again, every couple of "git clone" + builds that I do. With PM28 beta3 it's on again, for which my eyes are very greatful. Thank you for this wonderful extension.
There's a right way
There's a wrong way
And then there's my way
There's a wrong way
And then there's my way
-
- Astronaut
- Posts: 650
- Joined: 2015-07-30, 20:29
- Location: Vaughan, ON, Canada
Re: Night Mode Addon compatible with 27+?
This is getting "curiouser and curiouser". It seems that ANM works fine on my bog-standard PM 28 builds. However, it does not work on my customized builds (different mozconfig). Same build machine+environment, same source code. Now that I've finally gotten builds working on CentOS 6.10, I can concentrate on this problem. I'll go with the heavy handed approach... changing 1 line at a time in my mozconfig until it stops working. The poor laptop will have to do several consecutive builds.Walter Dnes wrote:On my homebrew 32-bit PM28 builds ANM is on-again, off-again, every couple of "git clone" + builds that I do.
There's a right way
There's a wrong way
And then there's my way
There's a wrong way
And then there's my way
-
- Astronaut
- Posts: 650
- Joined: 2015-07-30, 20:29
- Location: Vaughan, ON, Canada
Re: Night Mode Addon compatible with 27+?
That took less time than I thought. Testing shows that adding ac_add_options --disable-devtools to the mozconfig disables ANM. @JustOff, can you confirm that ANM has a dependancy on something in devtools?
There's a right way
There's a wrong way
And then there's my way
There's a wrong way
And then there's my way
Re: Night Mode Addon compatible with 27+?
ANM does not have a hard dependency on devtools and works just fine if they are not available. Moreover, ANM has not yet been updated to use the new path to gDevTools.jsm required for UXP, so it acts the same way as if they are not included, even if they are.
Do you see any errors in the console when ANM is failed to load in your build?
Do you see any errors in the console when ANM is failed to load in your build?
-
- Astronaut
- Posts: 650
- Joined: 2015-07-30, 20:29
- Location: Vaughan, ON, Canada
Re: Night Mode Addon compatible with 27+?
I respectfully disagree. ANM works on my homebrew 32-bit builds with devtools enabled, and does not work with them disabled.JustOff wrote:Moreover, ANM has not yet been updated to use the new path to gDevTools.jsm required for UXP, so it acts the same way as if they are not included, even if they are.
I see multiple repetitions of...JustOff wrote:Do you see any errors in the console when ANM is failed to load in your build?
Code: Select all
console.error:
Object
- message = Component returned failure code: 0x80040111 (NS_ERROR_NOT_AVAILABLE) [nsIXPCComponents_Utils.import]
- fileName = undefined
- lineNumber = 18
- stack = @undefined:18:NaN|@resource://gre/modules/commonjs/sdk/loader/sandbox.js:18:22|@resource://gre/modules/commonjs/sdk/content/sandbox.js:19:37|@resource://gre/modules/commonjs/sdk/content/worker-child.js:14:27|@resource://gre/modules/commonjs/sdk/content/page-mod.js:14:25|@resource://gre/modules/commonjs/sdk/remote/child.js:81:3|emitOnObject@resource://gre/modules/commonjs/sdk/event/core.js:112:9|emit@resource://gre/modules/commonjs/sdk/event/core.js:89:38|messageReceived@resource://gre/modules/commonjs/sdk/remote/child.js:67:37|
- toString = () => toString
There's a right way
There's a wrong way
And then there's my way
There's a wrong way
And then there's my way
Re: Night Mode Addon compatible with 27+?
This error means that you not only disabled the devtools client (--disable-devtools), but also disabled the devtools server (--disable-devtools-server) that is required for Add-ons SDK, and I guess you patched the configure script, which should not allow you to do this. Thus, no Add-ons SDK based add-ons will work, including ANM, regardless of whether it uses devtools or not.
Last edited by JustOff on 2018-07-31, 13:58, edited 3 times in total.
Re: Night Mode Addon compatible with 27+?
And this, my friends, is why we ask to see your build configuration.
"Sometimes, the best way to get what you want is to be a good person." -- Louis Rossmann
"Seek wisdom, not knowledge. Knowledge is of the past; wisdom is of the future." -- Native American proverb
"Linux makes everything difficult." -- Lyceus Anubite
"Seek wisdom, not knowledge. Knowledge is of the past; wisdom is of the future." -- Native American proverb
"Linux makes everything difficult." -- Lyceus Anubite