https://github.com/MoonchildProductions ... ue.js#L685
I do not get it, what is the purpose of that, can it be some bug? Becauase of that browser.showQuitWarning not works, but there are users who want use this setting.
I removed that code, and browser.showQuitWarning began do properly, and seems no other things has broken. Let that code to be removed from main code.
Why is there that code?
Forum rules
Please keep everything here strictly on-topic.
This board is meant for Pale Moon source code development related subjects only like code snippets, patches, specific bugs, git, the repositories, etc.
This is not for tech support! Please do not post tech support questions in the "Development" board!
Please make sure not to use this board for support questions. Please post issues with specific websites, extensions, etc. in the relevant boards for those topics.
Please keep things on-topic as this forum will be used for reference for Pale Moon development. Expect topics that aren't relevant as such to be moved or deleted.
Please keep everything here strictly on-topic.
This board is meant for Pale Moon source code development related subjects only like code snippets, patches, specific bugs, git, the repositories, etc.
This is not for tech support! Please do not post tech support questions in the "Development" board!
Please make sure not to use this board for support questions. Please post issues with specific websites, extensions, etc. in the relevant boards for those topics.
Please keep things on-topic as this forum will be used for reference for Pale Moon development. Expect topics that aren't relevant as such to be moved or deleted.
Re: Why is there that code?
No. Stop telling us what to do about things you don't understand. The feature works as intended.
-
- Pale Moon guru
- Posts: 35640
- Joined: 2011-08-28, 17:27
- Location: Motala, SE
Re: Why is there that code?
If # pages < 2, then don't show quit warning == correct behavior.
"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
Re: Why is there that code?
Yes, by default it do that way exactly.
But when users enabled browser.showQuitWarning they want get quit warning on one page too, is that wrong?
But when users enabled browser.showQuitWarning they want get quit warning on one page too, is that wrong?
-
- Pale Moon guru
- Posts: 35640
- Joined: 2011-08-28, 17:27
- Location: Motala, SE
Re: Why is there that code?
You clearly don't understand the preferences involved. Stop messing with them.Fedor2 wrote:Yes, by default it do that way exactly.
But when users enabled browser.showQuitWarning they want get quit warning on one page too, is that wrong?
"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
Re: Why is there that code?
So i want to understand.
If you want to get quit warning on closing browser with one page, what you are to tweak in the prefs?
I found these related:
One is false, and with that you get quit warning on two and greater pages, so i think if to enable the last, i shall get quit warning regardless page count, but with check in the code mentioned above, execution cannot reach to process showQuitWarning if opened only one page.
If you want to get quit warning on closing browser with one page, what you are to tweak in the prefs?
I found these related:
Code: Select all
browser.tabs.warnOnClose;true
browser.tabs.warnOnCloseOtherTabs;true
browser.warnOnQuit;true
browser.showQuitWarning;false