[Bug 583797]

2019-01-31 Thread Gingerbread-man-2
*** Bug 1492598 has been marked as a duplicate of this bug. ***

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797] Re: "Warn on closing with multiple tabs open" not honored

2019-01-06 Thread Mathew Hodson
** Changed in: firefox (Ubuntu)
   Status: Invalid => Fix Released

** Bug watch removed: Mozilla Bugzilla #1515061
   https://bugzilla.mozilla.org/show_bug.cgi?id=1515061

** Bug watch removed: Mozdev Bugzilla #26384
   https://www.mozdev.org/bugs/show_bug.cgi?id=26384

** Bug watch removed: Mozilla Bugzilla #52821
   https://bugzilla.mozilla.org/show_bug.cgi?id=52821

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Siddhartha Sahu
Thanks for the reply. I opened
https://bugzilla.mozilla.org/show_bug.cgi?id=1515061.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Gijskruitbosch+bugs
(In reply to sh.siddhartha from comment #81)
> Does the fix not consider pinned tabs? I use multiple windows with many
> pinned tabs of websites I regularly use (mail, etc). Ctrl+Q only warns
> before quitting if there are at least 2 *unpinned* open tabs. Should it not
> consider pinned tabs in the count as well?

I think so. Please file a follow-up bug.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Siddhartha Sahu
Hi.

Does the fix not consider pinned tabs? I use multiple windows with many
pinned tabs of websites I regularly use (mail, etc). Ctrl+Q only warns
before quitting if there are at least 2 *unpinned* open tabs. Should it
not consider pinned tabs in the count as well?

I'm using FF Developer edition 65.0b4 on Linux.

Thanks.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Liquido2
Thanks a lot for fixing this at last!

I am gladly verifying that this is implemented in latest Nightly 65 on
Linux (x86_64)

Build ID20181205102000
User Agent  Mozilla/5.0 (X11; Linux x86_64; rv:65.0) Gecko/20100101 
Firefox/65.0

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Gijskruitbosch+bugs
Note: we're tweaking this in bug 1506173, so subject to change, but:

Release Note Request (optional, but appreciated)
[Why is this notable]: long-standing behavior changes subtly
[Affects Firefox for Android]: no
[Suggested wording]: The "You're closing multiple tabs" warning now applies to 
quitting Firefox with automatic session restore turned on.
[Links (documentation, blog post, etc)]: n/a

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Pascalc
Gijs, could you request an addition to our release notes? Thanks!

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Bugzilla-tf
*** Bug 1505729 has been marked as a duplicate of this bug. ***

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Ben-bucksch
Yay, thanks so much!

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Pulsebot
Pushed by gijskruitbo...@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/fd42da5a8e1a
also show close warnings when session restore is active, r=jaws,flod

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Gijskruitbosch+bugs
(In reply to Amy Lee [:amylee] UX from comment #65)
> Comment on attachment 9021147
> Bug 550559 - also show close warnings when session restore is active, r?jaws
> 
> Hi, 
> 
> I installed the mac version and I don't see an update to about:preferences
> under the Tabs section (“Warn you when quitting and closing multiple tabs”)

The checkbox is only visible when the user opts out of the warning. This
is the existing UX today. I'm happy to change it, but the comments there
implied that this was intentional, with the idea that the dialogs
themselves offer a checkbox to turn this off, at which point it can be
turned back on in the preferences.

Do you think the checkbox should always be visible?

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Gijskruitbosch+bugs
Amy, have you had a chance to check the build again? These changes
otherwise have r+ and are ready to land.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Amlee
(In reply to Amy Lee [:amylee] UX from comment #65)
> Comment on attachment 9021147
> Bug 550559 - also show close warnings when session restore is active, r?jaws
> 
> Hi, 
> 
> I installed the mac version and I don't see an update to about:preferences
> under the Tabs section (“Warn you when quitting and closing multiple tabs”)

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Amlee
Comment on attachment 9021147
Bug 550559 - also show close warnings when session restore is active, r?jaws

Checked and looks good.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Rgurzau
https://hg.mozilla.org/mozilla-central/rev/fd42da5a8e1a

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Pascalc
Added to 65 nightly release notes with this wording:
The "Warn you when quitting and closing multiple tabs" warning now applies to 
quitting Firefox with automatic session restore turned on

We are keeping the 'relnote?' flag set as this is for now a nightly only
note and the final wording may be different.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Gijskruitbosch+bugs
(In reply to Amy Lee [:amylee] UX from comment #69)
> (In reply to :Gijs (he/him) from comment #68)
> > (In reply to :Gijs (he/him) from comment #67)
> > > (In reply to Amy Lee [:amylee] UX from comment #65)
> > > > Comment on attachment 9021147
> > > > Bug 550559 - also show close warnings when session restore is active, 
> > > > r?jaws
> > > > 
> > > > Hi, 
> > > > 
> > > > I installed the mac version and I don't see an update to 
> > > > about:preferences
> > > > under the Tabs section (“Warn you when quitting and closing multiple 
> > > > tabs”)
> > > 
> > > The checkbox is only visible when the user opts out of the warning. This 
> > > is
> > > the existing UX today.
> > 
> > Specifically, this happened in bug 1063625, and there's some discussion
> > about it there.
> 
> Hi, 
> 
> I mean the warning text hasn't been updated to "Warn you when quitting and
> closing multiple tabs", right now it still reads as "Warn you when closing
> multiple tabs"

Uh, I don't really understand how that's possible.
https://hg.mozilla.org/try/rev/b7b641a61f5e#l3.14 shows the change to
the string - the old string is no longer present. I just checked the
builds produced by try myself, and I can see the new string. The string
identifier also changed, so I don't really understand how it's even
possible for it to show the old string. Are you sure you ran the .app
from the .dmg I linked? Can you try it with a new profile?

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Gijskruitbosch+bugs
(In reply to :Gijs (he/him) from comment #67)
> (In reply to Amy Lee [:amylee] UX from comment #65)
> > Comment on attachment 9021147
> > Bug 550559 - also show close warnings when session restore is active, r?jaws
> > 
> > Hi, 
> > 
> > I installed the mac version and I don't see an update to about:preferences
> > under the Tabs section (“Warn you when quitting and closing multiple tabs”)
> 
> The checkbox is only visible when the user opts out of the warning. This is
> the existing UX today.

Specifically, this happened in bug 1063625, and there's some discussion
about it there.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Gijskruitbosch+bugs
Comment on attachment 9021147
Bug 550559 - also show close warnings when session restore is active, r?jaws

(In reply to Amy Lee [:amylee] UX from comment #61)
> (In reply to :Gijs (he/him) from comment #59)
> > Amy, this bug is still assigned to you - is there more that needs to happen
> > design-wise or is this ready to be implemented?
> 
> The design is ready to be implemented. Can you please reassign the bug to
> someone that implement?

OK. I attached a patch. Here are some builds to test:

Linux64:
https://queue.taskcluster.net/v1/task/Zd3pnRF4QzatNAU9w4kgyw/runs/0/artifacts/public/build/target.tar.bz2

Mac:
https://queue.taskcluster.net/v1/task/fNm10L8cQ_6MS4FuBly_Ag/runs/0/artifacts/public/build/target.dmg

Windows: https://queue.taskcluster.net/v1/task/TsxrJ8xgQe-
pRSlpV7rgbg/runs/0/artifacts/public/build/target.zip

Can you let me know if this matches your expectations? Thank you!

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Gijskruitbosch+bugs
Amy, this bug is still assigned to you - is there more that needs to
happen design-wise or is this ready to be implemented?

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Amlee
(In reply to :Gijs (he/him) from comment #59)
> Amy, this bug is still assigned to you - is there more that needs to happen
> design-wise or is this ready to be implemented?

The design is ready to be implemented. Can you please reassign the bug
to someone that implement?

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Amlee
(In reply to :Gijs (he/him) from comment #68)
> (In reply to :Gijs (he/him) from comment #67)
> > (In reply to Amy Lee [:amylee] UX from comment #65)
> > > Comment on attachment 9021147
> > > Bug 550559 - also show close warnings when session restore is active, 
> > > r?jaws
> > > 
> > > Hi, 
> > > 
> > > I installed the mac version and I don't see an update to about:preferences
> > > under the Tabs section (“Warn you when quitting and closing multiple 
> > > tabs”)
> > 
> > The checkbox is only visible when the user opts out of the warning. This is
> > the existing UX today.
> 
> Specifically, this happened in bug 1063625, and there's some discussion
> about it there.

Hi,

I mean the warning text hasn't been updated to "Warn you when quitting
and closing multiple tabs", right now it still reads as "Warn you when
closing multiple tabs"

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Amlee
Comment on attachment 9021147
Bug 550559 - also show close warnings when session restore is active, r?jaws

Hi,

I installed the mac version and I don't see an update to
about:preferences under the Tabs section (“Warn you when quitting and
closing multiple tabs”)

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Gijskruitbosch+bugs
Created attachment 9021147
Bug 550559 - also show close warnings when session restore is active, r?jaws

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Gijskruitbosch+bugs
*** Bug 502908 has been marked as a duplicate of this bug. ***

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Rik
This bug is going to hit the standard edition... as soon as it become v63... 
and there is only 1 number left :-/
I think when the bug will hit the standard edition, support.mozilla.org will 
explode :-)

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Yumekui Neru
why is this taking a decade to fix?

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Mwalkington
@Gijs Okay, we can use the numerical version of numbers and NOT spell
them out.

**Confirm close**
You are about to close 7 tabs. Are you sure you want to continue?

Warn me when closing multiple tabs

Cancel  Close Tabs

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Amlee
Created attachment 8999195
Preferences - Close Tab Warning.png

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Amlee
Created attachment 8989553
Linux_Windows_Close.png

Reference for above comment

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Gijskruitbosch+bugs
(In reply to Meridel from comment #46)
> Prefs copy looks good and hasn't changed from what we have currently.
> 
> Edits to close warning dialogue:
> -Numbers less than ten should be spelled out. So if the user is closing 2 -
> 9 tabs, spell out the number of tabs.

This is really annoying to implement, and requires localizers to provide
translations for all these numbers. It also makes it harder to scan for
the numbers when reading the warning. It's also not clear to me that any
convention in (American?) English that you're using here applies in
other languages (eg I grew up in the Netherlands and the rule I got
taught was to write out numbers under 20, as well as any numbers
divisible by 10 (under 100) or 100 (under 1000), etc.). So it would be
"five thousand tabs" but "5341 tabs". I would be unsurprised if other
languages used other conventions still. It also doesn't seem related to
this bug. If you feel strongly that this is necessary, can you file a
separate bug for this, maybe with more details about how that should
work?

For the changes to the prefs:

(In reply to Amy Lee [:amylee] UX from comment #44)
> I've attached a spec for Preferences when Session Restore is Enabled and
> Disabled. The warning will not show up by default if Session Restore is
> enabled but I've added a pref that allows users to turn the warning message
> on when Session Restore is enabled if desired. 

I have a several concerns here.

First, now if users have session restore enabled there is 1 pref that
governs both window closing and quitting. Unfortunately, there isn't a
pref to turn on warnings for quit but not have warnings when closing
multiple tabs otherwise. This would be desirable because it's easy to
mis-hit cmd-q when trying to hit cmd-w (to close a tab). You have to try
pretty hard to mis-hit cmd-shift-w (to close a window) or to close
multiple tabs using the mouse. So I want a warning when quitting but not
when closing a window, but I can't have that either today or with the
implementation you're suggesting. That seems like something we should
fix.

Second, users who are used to the current behavior with session restore
enabled (esp. on Windows, where mis-hitting ctrl-shift-q is also not a
major concern) will start getting warning dialogs when we implement the
new behavior. That seems disruptive. But we can't turn off the warning
everywhere because then they won't get a warning when closing a window,
which they might have relied upon. Any thoughts about how to address
that?

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Gijskruitbosch+bugs
*** Bug 1499401 has been marked as a duplicate of this bug. ***

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Amlee
To clarify this confirmation should exist when quitting the browser in
all menus (not only when using shortcut keys)

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Gijskruitbosch+bugs
(In reply to Amy Lee [:amylee] UX from comment #49)
> (In reply to :Gijs (Not available 3-19 Aug; he/him) from comment #48)
> > Second, users who are used to the current behavior with session restore
> > enabled (esp. on Windows, where mis-hitting ctrl-shift-q is also not a major
> > concern) will start getting warning dialogs when we implement the new
> > behavior. That seems disruptive. But we can't turn off the warning
> > everywhere because then they won't get a warning when closing a window,
> > which they might have relied upon. Any thoughts about how to address that?
> 
> The default settings of Session Restore would be what we have currently (not
> having warnings).

That's not really what happens today. Today, there are warnings when
closing windows, or using "close tabs to the right", or using "close
other tabs" - but not when quitting, if and only if session restore is
enabled.

Specifically, the preference you're suggesting we surface "under"
Restore Previous Session already exists, and its default is true (ie
warn when closing tabs). However, there are hardcoded checks for "does
the user have session restore enabled?" in the code that avoid warning
when quitting, irrespective of whether the "warn when closing tabs"
checkbox is checked.

In your new design, we'd remove the hardcoded checks to ask session
restore in the code that actually implements warning, and cosmetically
we'd move the checkbox in about:preferences, and add some logic to
tick/untick (and set/unset the pref) when the user toggles restoring
their session. But that doesn't solve the conundrum here - the pref
would be turned on by default, and we'd remove the checks, so then the
user will start getting warnings.

We could have a one-time migration that turns off the "warn when closing
tabs" pref for people who have session restore turned on, but that also
seems like it'd potentially confuse users who would no longer see
warnings when closing windows / multiple tabs (e.g. via "close tabs to
the right") where they did see those before.

What would you prefer we do? Disable the warnings everywhere for session
restore users, or accept that we'll now start warning session restore
users who might not have expected this?


(In reply to Amy Lee [:amylee] UX from comment #49)
> I think we are getting very specific for an issue that I think affects a
> small population of users.

While I sympathise with the sentiment here, we already made a change in
bug 1438499 (to show the same "closing multiple tabs" warning when
quitting when session store is not enabled, removing a separate option
to show an alternative dialog that people had to opt in to via
about:config), and already people on Nightly are basically saying that
that removed a "fail-safe":

(In reply to Kestrel from bug 1438499 comment #47)
> browser.showQuitWarning was a good fail-safe to avoid data loss but now that
> it has been removed a suitable replacement is needed, hopefully Bug 550559
> can be expedited to avoid a regression.

... because the "closing multiple tabs" warning on quit is
enabled/disabled together with the same warning when closing
windows/other/right-hand tabs, there's no way to force the quit warning
to come up but not warn for closing windows / multiple tabs, and that's
burning people (it seems).

(In reply to Amy Lee [:amylee] UX from comment #49)
>  It starts to become overkill if we try to provide
> such specific checkbox options (ie. warn me when quitting, warn me when
> closing a window, etc.).

FWIW, I think the only requirement here is quitting vs. everything else,
and I think an about:config pref would be fine. I don't think this is
going to be a slippery slope. Quitting is by its nature *much* more
destructive than closing a single window, and as noted, easier to do by
accident (esp. on macOS/Linux) than all the other ways of closing
multiple tabs.

If you're convinced that we don't need to provide an option here, that
works, but I think we need to tread carefully and deliberately here.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Mdeboer
I think :Gijs is a good mentor for this as well. I think we'd both
happily take review and feedback requests!

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Rik
PS: are you sure this bug is the same that hide the dialog "Do you want
Firefox to save your tabs for the next time it starts?"?

Because I still see the question "You are about to close 2 tabs. Are you
sure you want to continue?" but I don't see the other dialog that once
allowed to save the tabs.

I've checked all the flag in about:config and nothing works anymore. The
question "Do you want Firefox to save your tabs for the next time it
starts?" is gone.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Rik
How is it going? In the meantime I've installed an add on to restore tabs. :-(
Hope this is not going to be passed into the standard version of FF, because at 
the moment the problem is only in the Developer Edition.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Amlee
I've attached a spec for Preferences when Session Restore is Enabled and
Disabled. The warning will not show up by default if Session Restore is
enabled but I've added a pref that allows users to turn the warning
message on when Session Restore is enabled if desired.

NI Meridel to review the copy. Thanks!

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Amlee
(In reply to :Gijs (Not available 3-19 Aug; he/him) from comment #48)
> (In reply to Meridel from comment #46)
> > Prefs copy looks good and hasn't changed from what we have currently.
> > 
> > Edits to close warning dialogue:
> > -Numbers less than ten should be spelled out. So if the user is closing 2 -
> > 9 tabs, spell out the number of tabs.
> 
> This is really annoying to implement, and requires localizers to provide
> translations for all these numbers. It also makes it harder to scan for the
> numbers when reading the warning. It's also not clear to me that any
> convention in (American?) English that you're using here applies in other
> languages (eg I grew up in the Netherlands and the rule I got taught was to
> write out numbers under 20, as well as any numbers divisible by 10 (under
> 100) or 100 (under 1000), etc.). So it would be "five thousand tabs" but
> "5341 tabs". I would be unsurprised if other languages used other
> conventions still. It also doesn't seem related to this bug. If you feel
> strongly that this is necessary, can you file a separate bug for this, maybe
> with more details about how that should work?
> 
> For the changes to the prefs:
> 
> (In reply to Amy Lee [:amylee] UX from comment #44)
> > I've attached a spec for Preferences when Session Restore is Enabled and
> > Disabled. The warning will not show up by default if Session Restore is
> > enabled but I've added a pref that allows users to turn the warning message
> > on when Session Restore is enabled if desired. 
> 
> I have a several concerns here.
> 
> First, now if users have session restore enabled there is 1 pref that
> governs both window closing and quitting. Unfortunately, there isn't a pref
> to turn on warnings for quit but not have warnings when closing multiple
> tabs otherwise. This would be desirable because it's easy to mis-hit cmd-q
> when trying to hit cmd-w (to close a tab). You have to try pretty hard to
> mis-hit cmd-shift-w (to close a window) or to close multiple tabs using the
> mouse. So I want a warning when quitting but not when closing a window, but
> I can't have that either today or with the implementation you're suggesting.
> That seems like something we should fix.

I think we are getting very specific for an issue that I think affects a
small population of users. It starts to become overkill if we try to
provide such specific checkbox options (ie. warn me when quitting, warn
me when closing a window, etc.).

> 
> Second, users who are used to the current behavior with session restore
> enabled (esp. on Windows, where mis-hitting ctrl-shift-q is also not a major
> concern) will start getting warning dialogs when we implement the new
> behavior. That seems disruptive. But we can't turn off the warning
> everywhere because then they won't get a warning when closing a window,
> which they might have relied upon. Any thoughts about how to address that?

The default settings of Session Restore would be what we have currently
(not having warnings). The user can turn on the warning when closing
multiple tabs in Prefs. If they accidentally close a window, it's less
of a destructive action because Session Restore is enabled so if that
does accidentally happen, their tabs are restored anyways when they
reopen. Again, I believe this affects a small population of users. If
they are that worried about accidental closing, they are given the
option to turn the warnings back on.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Jgilbert
I don't know. Matt? Can you route this to someone to provide direction
towards a solution, or establish WONTFIX?

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Amlee
Created attachment 8995330
Preferences - Close Tab Warning.png

Spec for Restore Previous Session Setting in Preferences

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread 2-gene
If you'd like to implement what Paul is talking about in Comment 38 you
can do this :

> The workaround that I've been using is to leave a tab pinned at all times 
> with code that uses the "onbeforeunload" javascript function.
> 
> https://github.com/gene1wood/sandbox/blob/master/docs/prevent_browser_quitting.html
> 
> With this tab open and pinned it forces the browser to prompt before shutting 
> down giving me a chance to click "Stay on page" and stop Firefox from exiting.

https://bugzilla.mozilla.org/show_bug.cgi?id=52821#c303

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Amlee
(In reply to :Gijs (Not available 3-19 Aug; he/him) from comment #50)
> (In reply to Amy Lee [:amylee] UX from comment #49)
> > (In reply to :Gijs (Not available 3-19 Aug; he/him) from comment #48)
> > > Second, users who are used to the current behavior with session restore
> > > enabled (esp. on Windows, where mis-hitting ctrl-shift-q is also not a 
> > > major
> > > concern) will start getting warning dialogs when we implement the new
> > > behavior. That seems disruptive. But we can't turn off the warning
> > > everywhere because then they won't get a warning when closing a window,
> > > which they might have relied upon. Any thoughts about how to address that?
> > 
> > The default settings of Session Restore would be what we have currently (not
> > having warnings).
> 
> That's not really what happens today. Today, there are warnings when closing
> windows, or using "close tabs to the right", or using "close other tabs" -
> but not when quitting, if and only if session restore is enabled.
> 
> Specifically, the preference you're suggesting we surface "under" Restore
> Previous Session already exists, and its default is true (ie warn when
> closing tabs). However, there are hardcoded checks for "does the user have
> session restore enabled?" in the code that avoid warning when quitting,
> irrespective of whether the "warn when closing tabs" checkbox is checked.
> 
> In your new design, we'd remove the hardcoded checks to ask session restore
> in the code that actually implements warning, and cosmetically we'd move the
> checkbox in about:preferences, and add some logic to tick/untick (and
> set/unset the pref) when the user toggles restoring their session. But that
> doesn't solve the conundrum here - the pref would be turned on by default,
> and we'd remove the checks, so then the user will start getting warnings.
> 
> We could have a one-time migration that turns off the "warn when closing
> tabs" pref for people who have session restore turned on, but that also
> seems like it'd potentially confuse users who would no longer see warnings
> when closing windows / multiple tabs (e.g. via "close tabs to the right")
> where they did see those before.
> 
> What would you prefer we do? Disable the warnings everywhere for session
> restore users, or accept that we'll now start warning session restore users
> who might not have expected this?

I would say accept that we'll now start warning session restore users
about quitting. They have the option on the warning message to disable
it if they want to. I've changed the wording slightly in the preference
to include "quitting" when enabling warning on closing multiple tabs. If
a user quits the browser with multiple tabs opened they should get this
warning as well cause technically they are still closing multiple tabs
when quitting. I've attached an updated spec.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Mozilla-org-8
Firefox warns on quit just fine if you have at least one active tab with
an unsubmitted form. That dialog lets the user abort a quit. Could we
use the same mechanism for this?

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread skierpage
(In reply to Linas from comment #27)
> Listed on bug 565567, there is
> https://addons.mozilla.org/en-US/firefox/addon/always-ask/ as workaround.

There's also https://addons.mozilla.org/en-US/firefox/addon/warn-before-
quit/ , they both work for me in Firefox 41 on Ubuntu 15.04

I have browser.showQuitDialog, browser.showQuitWarning, and
browser.warnOnQuit all set to true in about:config. It seems _insane_
that Firefox doesn't do what I tell it.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Mwalkington
Prefs copy looks good and hasn't changed from what we have currently.

Edits to close warning dialogue:
-Numbers less than ten should be spelled out. So if the user is closing 2 - 9 
tabs, spell out the number of tabs.
-Button copy should be capitalized
-The checkbox string should be in second person and consistent with language in 
Prefs

So, confirm close copy should be:

**Confirm close**
You are about to close seven tabs. Are you sure you want to continue?

Warn me when closing multiple tabs

Cancel  Close Tabs

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Amlee
Great! Thanks Meridel. Can we reassign this bug to someone that can
implement the changes? Currently this bug is assigned to me.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Amlee
After going through the thread and related bugs, it seems to make sense
to have a confirmation dialogue when quitting the browser. Also it looks
like a confirmation dialogue does appear in Windows/Linux when "closing"
multiple tabs and not when "quitting" which is confusing. There is no
confirmation at all in Mac. I've attached a screenshot of the
confirmation for when closing multiple tabs on Linux/Windows. This
should also exist for Mac when quitting/closing.

Rational for having a confirmation even if we have session restore:
1. Session Restore doesn't restore dev tools. Accidentally quitting and losing 
progress.
2. When bandwidth is limited and you don't want to reload your tabs after 
accidentally quitting.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Graham Perrin
For the Session Manager extension:

Bug 26384 – Not compatible with firefox 57+?
<https://www.mozdev.org/bugs/show_bug.cgi?id=26384>

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Beneficentone
(In reply to Ambrose Li from comment #31)
> In fact, session restore won't do the user any good if Firefox is running in
> Private Browsing mode. So the assumption that the warning can be turned off
> if we do session restores is inherently flawed.

Yes, that's true.  And Private Browsing mode is made still more
vulnerable by a bug where the 'close multiple tabs' warning doesn't work
- just filed as Bug 1096350, but an old issue.  Another bug (Bug
1096437) means no 'close multiple tabs' warning if you have a single
non-private window.  Just for good measure, the old quit dialog long
available via browser.showQuitWarning has finally been killed off.

Session Restore was a nice idea horribly implemented.  The answer was
always a simple one: add support for multiple sessions, as suggested by
me 3½ years ago (Bug 648081).  Not actually a huge undertaking.  A
session is just a set of files, and if you can save and restore one you
can accommodate 2 - or 20, come to that.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread 9-wojtek
While the add-ons work, this should be build-in to in the browser as
it's highly annoying!

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Mjd+mozilla-org
(In reply to wojtek from comment #34)

> this should be build-in to in the browser as it's highly annoying!

Stopped using Firefox, moved to Chromium.  That this issue should go
nowhere for 6½ years was one issue of many behind my reason to switch.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Linas-fi
Despite having browser.showQuitWarning and browser.warnOnQuit set to true (as 
well as browser.tabs.warnOnClose), pressing Ctrl+Q (or using File → Exit) 
closes the program with no confirmation.
This is very annoying when you were using the browser and just intended to 
close a tab instead (Ctrl+W), since it leaves you with no browser for 7 minutes 
(bug 806109) while it is busy closing a program you didn't want to close at 
all!.


Note: I'm duping here a bunch of related bugs.
Listed on bug 565567, there is 
https://addons.mozilla.org/en-US/firefox/addon/always-ask/ as workaround.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Gijskruitbosch+bugs
*** Bug 1025847 has been marked as a duplicate of this bug. ***

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Paul-oshannessy
Comment on attachment 594363
Don't disable warn-on-quit when session restore is enabled

Review of attachment 594363:
-

Sorry for the delay. I've had this on my mind for a while though.

My main problem here is that this doesn't really do what the end goal of
this bug is. This simply removes a check from the dialog path, but it
doesn't handle the keyboard case at all, which is what we really want.

So I think we should move discussion about this specifically to a new
bug. Since this is (mostly) only going to affect people who manually
gone & changed showQuitWarning to true & have sessionstore on, it
shouldn't be too disruptive. Though, we'll show the quit dialog which
asks about saving tabs when session restore is already on... so that's
not ok. Also, all the people quitting by closing the last window will
suddenly have a dialog in their face, event without showQuitWarning (I
think I take back "shouldn't be too disruptive"). We can discuss more in
a new bug if you file it

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread A-c-li
In fact, session restore won't do the user any good if Firefox is
running in Private Browsing mode. So the assumption that the warning can
be turned off if we do session restores is inherently flawed.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Linas-fi
Seems I don't have appropiate permissions. Someone able to do it may
like to dupe here bug  565567, bug 633504, bug 691297 and bug 702478.
Related but a bit different, there are also bug 298802 and bug 298803.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Jakub-g-opensource
*in the previous post, it should be: 'Default Firefox behavior
(*without* All-in-One Sidebar)'

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread M-wada-7
(In reply to Linas from comment #27)

If Mac OS X, because bug 938303 is fixed now, Command+Q is killed by "other 
shortcut assignment to Quit Firefox by OS" from Firefox 32, as it was possible 
unill Firefox 2. So, Mac OS X/Firefox user can freely press Command+W pretty 
safely. User can press Command+Q any time safely. Thus, severity of this bug 
will be changed to lower in Mac OS X after Firefox 32.
FYI.
Reason why all of bug 457973, bug 515395, bug 624881, bug 646362, bug 938303 
were hidden and killed :
All of these bugs were closed as dup of bug 429824.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Paul-oshannessy
Created attachment 621206
WIP for keyboard quit

This just adds a notification for when we quit via the keyboard. This
could be used to popup a new window & presumably spin the event loop or
something.

We can't quite do what (for example) Chrome is doing and capture
everything down in Cocoa-land because we construct the menu from XUL and
then pass everything straight back there without really processing the
event in Cocoa. The Cocoa part of this is for when quit is sent via some
other way, for example cmd+q while in cmd-tab, activity monitor.

This isn't a well formed idea yet but I had _something_ to put up. If
somebody (Frank?) wants to take this and run, feel free - I don't plan
on working on this in the near future.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Jgilbert
Huh, I guess I hadn't CC'd myself here.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Gavin Sharp
Comment on attachment 594363
Don't disable warn-on-quit when session restore is enabled

I think Paul gets to handle this one, since he really loves
quitting/prompting code.

(I apologize for the unreasonable delay in getting to this)

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Jakub-g-opensource
Let me add my two cents (scroll down for TLDR) (I'm a Windows user BTW).

I'm very close to @Ian Stirling opinion from #comment 8. I want to have
an option to *absolutely* ask on quit, whatever makes me do so
(shortcut, click, menu item); isn't there any 'onbeforequit'-like hook
in the codebase, whatever was the root cause that triggered it?

Let me explain the point.

For me, Firefox is The Centre Of Management of The Universe. I regularly have 
100+ tabs open, with their content being changed by me using some Greasemonkey 
scripts that add additional content when I click this and that (and this is 
gone after restart). I can't allow browser just to quit accidentally because of 
some misfortunate events happening. And there are zillions of things that can 
accidentally go wrong. I'm a power user, yet it still happens from time to time 
and you can't help:
- I can hit ALT-F4 while I was sure the focus was in the different app. Or hit 
ALT-F4 more than once, one of this happening in Firefox, because the OS 
switched the focus after closing previous app to FF which I didn't expect.
- I can have a different application (say Notepad) on top of Firefox on the 
desktop stack. And now two cases:
-- when both are maximized, I can hit 'X' twice by chance, because for instance 
my mouse is malfunctioning, or my touchpad too sensitive etc etc.
-- when at least one of them (say Notepad on top) is not maximized, I want to 
try to close the Notepad, but I move my mouse (or especially touchpad) 2 pixels 
too far on the right, and, voila, Firefox goes to hell.
-- I open a 'File' menu menu via LEFT_ALT, DOWN_ARROW, I see the options, I 
iterate them via DOWN_ARROW, UP_ARROW, and I push enter on exit, but I wanted 
one item higher in the menu (theoretical, but can happen).
-- Also, I may confuse shortcuts, because I'm overworked and simply tired!

All of the above are very unlikely, but still happen from time to time,
and each of those is a *DISASTER*, when you have 100+ tabs opened, and
(God prevent) you're out of mobile range / your mobile network provider
sucks / your phone-as-a-modem sucks etc. while Firefox still thinks
you're online.

It's especially painful when you buffered than damn 15-minutes YouTube
video to watch it offline, then you accidentally close Fx, and you're
screwed.

Also, I might be downloading a huge file from a server that doesn't
support restarts-in-the-middle. Default Firefox behavior (with All-in-
one-sidebar) is to close the actual window, and leave only the downloads
opened! This is ridiculous. I can now either wait 1h till the download
ends, or restart the Firefox, and screw the download. (it doesn't affect
me, as I use AiOS add-on; just pointing it).

When I tell the browser I want a warning, I REALLY want it, and don't be
overly clever to not show it to me when you think it's not necessary.
Period. Hope I'm convincing enough.

Please, always display me a warning when I want to quit Firefox and I
stated it in about:config. Actually I only quit Fx to install upgrades
/add-ons, or when the memory consumption is too big. I hibernate my
comps, and have Fx opened for whole days.

__
TL;DR:
- I may have 100+ tabs
- I may have slow connection (laptop, with mobile phone as a modem)
- I may sometimes go offline
- I may use dynamic content (Greasemonkey, Flash videos loaded) that will be 
lost after restart
- I might be tired and do something stupid by mistake! I'm a human after all

Then, session restore is only helpful in the way that you don't lose
URLs you were browsing.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Jgilbert
Created attachment 594216
Don't disable warn-on-quit when using session restore

I'm not that comfortable with this part of the codebase, but this seems
like a fairly trivial fix.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Limi-mozilla
Comment on attachment 594363
Don't disable warn-on-quit when session restore is enabled

You probably want Gavin to review the code; +1 from a UI standpoint from
me. If there's a tryserver build attached, I can help test and see if
there are any edge cases, since I can't read the code changes that well.
:)

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Bugzilla-mauve
It existed in 3.6, yes.
But for users who are trying migrating from multiple windows to tab groups, it 
will hit for the first time (as it did for me)

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Limi-mozilla
This is especially noticable on a Mac since ⌘Q and ⌘W are right next to
each other, but in general it violates the principle of least surprise:

We currently ask when you quit your browser whether that's what you
wanted to do, and give you an option to "never ask again".

However, when turning on Session Restore, this "never ask again"
behavior is automatically enabled, which is confusing. I appreciate the
logic ("you're not losing anything by quitting since session restore is
awesome"), but it still causes a long startup time if you do it
accidentally.

We could do this only when you quit using a keyboard shortcut too, but
that gets back into the "being too clever" side of things again, since
you would have two slightly different "never ask me again" settings.

Since it's so easy to check the "never ask me again" on the quit dialog,
we shouldn't assume that Session Restore implies that you don't want to
be asked. So let's please disable this behavior.


PS: When one of our developers doesn't even know about this behavior and 
creates an extension to fix it, it doesn't speak very well to its obviousness. 
;) https://addons.mozilla.org/en-US/firefox/addon/55824/
PPS: Yes, other applications & browsers have this same issue, but that doesn't 
mean we shouldn't make it better.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Limi-mozilla
Should probably be a softblocker if so. (Adding d? to get driver input,
let me know if this isn't the way to go about this :)

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Mozbugz-z
(In reply to comment #14)
> " Quitting the application by using the menu or using the window close button
> (Windows & Linux only) will not trigger any warning."
> 
> Absolute. garbage. 
> 
> This has made FF4 unusable for me. 

This bug is not about the feature change.  Your looking for bug 592822
and bug 629485.  You know, I get tired of hearing that session store
don't work, but most people never check Firefox to see how many ways
there are to restore sessions, or find out how to turn the dialogs on
which I'm not going to do here since this bug is different altogether.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Jgilbert
Comment on attachment 594216
Don't disable warn-on-quit when using session restore

This seems fairly simple, but I can currently only test on Windows. The
concerned platforms for this bug (IIRC) are mac and linux, so those need
a brief test.

Try push at: https://tbpl.mozilla.org/?tree=Try&rev=e6235a00b6db

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Chofmann
paul,  can you also make a call here on difficulty of a possible fix.
that would play into the possibility of this becoming a blocker.  It
needs to get out of the nomination queue, one way or the other.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Gavin Sharp
I don't think we want to show a quit warning when we're restarting, e.g.
for an extension install. I think that case is covered by
browser.sessionstore.resume_session_once==true, which you're removing.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Mbeltzner
This still relevant now that we've changed the quit dialog to be ... not
so much there ...?

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Jgilbert
Created attachment 594363
Don't disable warn-on-quit when session restore is enabled

Thanks Gavin. That looks like the case.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Limi-mozilla
Refining title, Ctrl-Q is present on Mac, Cmd-Q on OS X. As far as I
know, Ctrl-Q doesn't work on Windows, so it's not a problem there.

Alt-F4 closes a window on Windows, so it's not going to be accidentally
triggered when closing a tab (Ctrl-W) — so we don't need to handle that
in any way (unless there's an Alt-F3 or Alt-F5 shortcut I don't know
about).

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Mickrussom
" Quitting the application by using the menu or using the window close button
(Windows & Linux only) will not trigger any warning."

Absolute. garbage.

This has made FF4 unusable for me. Good luck keeping up with the
competition by de-feature-izing this browser into yet another Chrome
clone.

At this point, I'd rather start putting up with Chrome, since it seems
to be adding things, not taking them away.

not warning when there is more than one tab open is insane, and due to
the ridiculous amount of time it takes to restart FF, its even more
annoying.

What is wrong with this project? Make it use less CPU, make it use less
memory, but dont sit there are rip features out.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Paul-oshannessy
I think the keyboard stuff is relevant, but the dialog stuff less so.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Bugzilla-mauve
Echoing comment 3.
(I'm on linux)
On comment 4 - '* Quitting the application by using the menu or using the 
window close button
(Windows & Linux only) will not trigger any warning. (Restoring your previous
session should be even more discoverable once the new about:home design lands,
which seems to be on track to make Firefox 4.0.)'

First, a general point.
What is the rationale between the close button not closing FF window with 40 
tabs, only if there is another window which may only have one tab?

Addressing this feature specifically.

The above comment to me seems to miss several usability issues for those
with disadvantaged net connections.

If the web were static, CPU and bandwidth were free, there isn't really
any issue.

Unfortunately, when you get away from this ideal case, as is common in
many parts of the world, restoring a large number of tabs gets painful.

To go into the usability issues I've hit.

A) A fair amount of content may not be cached - especially if the
browser session has been up for some time. This can take a long time,
cost money, or lose data that you were looking at in dynamic pages that
have now changed.

Yesterday, I was out, and accidentally closed FF, and had to buy wifi
access, rather than just use the data in my open tabs.

B) Network unreliability.
If the network isn't available all the time, then many of the tabs come back 
broken.

C) Stateful pages.
I'm a member of a subscription news site that limits my downloads to 20 per 
day, and sets the no-cache header. This means that when FF restarts, my tabs 
that I've opened as reference over the last 3-4 days all load at once, and take 
me over my 20 per day limit, meaning I need to wait a day.

D) CPU - I don't have a very fast CPU, even neglecting the above issues,
it would take a while to load.

A-C could in principle be addressed by by making it possible to set the
cache up in such a way that prerequisites of pages are always cached,
and on restoring state, the state is restored solely from cache, not
touching the network at all.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Limi-mozilla
To reduce confusion (people on IRC said it was still a bit unclear),
here's what we're proposing:

* On Mac & Linux, since they have the Cmd-Q shortcut to quit the
browser, we want to warn users that they're about to close all their
tabs (or quit) when they do it using the keyboard shortcut *only*.

  * In that dialog, *do* offer the option to not ask again, for the
people who just want this to quit automatically.

* Quitting the application by using the menu or using the window close
button (Windows & Linux only) will not trigger any warning. (Restoring
your previous session should be even more discoverable once the new
about:home design lands, which seems to be on track to make Firefox
4.0.)

Let me know if anything is missing or unclear.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Mars-martian+bugmail
Limi:
should we also change the behaviour in Private Browsing Mode? Currently if you 
cmd+Q while in PBM, no quit dialog is ever shown. Based on 
http://blog.mozilla.com/metrics/2010/08/23/understanding-private-browsing/ it 
doesn't seem like the user would be losing much data if we preserved the 
current behaviour.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Limi-mozilla
OK, I guess I changed my mind slightly here after living with it for a
while — I now think we should ask when trying to quit using the keyboard
(still with the "Do not ask me again" shortcut, but not when quitting
using the menu or (on Windows) the close button.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797] Re: "Warn on closing with multiple tabs open" not honored

2019-01-02 Thread Bug Watch Updater
** Changed in: firefox
   Status: Invalid => Fix Released

** Bug watch added: Mozdev Bugzilla #26384
   https://www.mozdev.org/bugs/show_bug.cgi?id=26384

** Bug watch added: Mozilla Bugzilla #52821
   https://bugzilla.mozilla.org/show_bug.cgi?id=52821

** Bug watch added: Mozilla Bugzilla #1515061
   https://bugzilla.mozilla.org/show_bug.cgi?id=1515061

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Hskupin
With the removal of the Quit warning dialog on OS X this bug is kinda
annoying because shortcuts to close tabs and the browser itself are
directly beneath each other. It happens so often for me that I
accidentally close the whole browser when I'm trying to close a tab.
With a lot of tabs open it takes a long time to shutdown and restart, in
most cases I loose about 4-5 minutes.

I will ask for blocking so we can sort out what should happen for Fx4
and what not.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Mbeltzner
This still doesn't block; existed in 3.6, will continue to exist in 4.0.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2019-01-02 Thread Paul-oshannessy
My current guess is not easy. That's just going off what I remember when
I looked into this briefly before.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797] Re: "Warn on closing with multiple tabs open" not honored

2018-12-28 Thread Mathew Hodson
** Bug watch removed: github.com/nigelbabu/warn-before-quit/issues #16
   https://github.com/nigelbabu/warn-before-quit/issues/16

** Bug watch added: Mozilla Bugzilla #550559
   https://bugzilla.mozilla.org/show_bug.cgi?id=550559

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2018-12-14 Thread Shtetldik
I can confirm, that setting browser.sessionstore.warnOnQuit = true fixes
this for me. The browser won't close down without a warning when hitting
Ctrl+Q! Thanks for finally fixing it.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797] Re: "Warn on closing with multiple tabs open" not honored

2018-12-12 Thread Olivier Tilloy
Closing as invalid to reflect upstream bug report.

** Changed in: firefox (Ubuntu)
   Status: Triaged => Invalid

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797] Re: "Warn on closing with multiple tabs open" not honored

2018-12-12 Thread Bug Watch Updater
** Changed in: firefox
   Status: Confirmed => Invalid

** Bug watch added: github.com/nigelbabu/warn-before-quit/issues #16
   https://github.com/nigelbabu/warn-before-quit/issues/16

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2018-12-12 Thread Soupdragon90
Really? This bug is still present 8 years after I quit when it wasn't
fixed? Why am I not surprised? Glad I switched but am tired of getting
emails related to this bug. I thought I unsubbed to them all but I guess
there's more.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2018-12-12 Thread Gijskruitbosch+bugs
(In reply to Kestrel from comment #65)
> The fix for Bug 550559 did not end up resolving this issue,
> browser.warnOnQuit = true still does not work with browser.tabs.warnOnClose
> = false.

After bug 1506173, there's now a separate pref,
browser.sessionstore.warnOnQuit , which you can use to have a quit
warning if you have session restore enabled. There's also a checkbox in
the prefs/options for this feature.

If session restore is not enabled, browser.tabs.warnOnClose still
governs whether you get a confirmation warning (if there are multiple
tabs).

We don't intend to add a third, separate pref/warning for people who
*do* want to have a quit warning, *don't* turn on session restore, and
*don't* want warnings when closing windows / multiple tabs (ie making
browser.tabs.warnOnClose = false + no automatic session restore +
browser.warnOnQuit = true show a dialog is not something we're
interested in doing. The "point" of the warning is to avoid losing tabs
that you can't currently see / may have forgotten you had; from that
perspective, wanting a warning for quitting but not for closing a window
with multiple tabs doesn't really make sense).

Keeping this bug open doesn't really make sense given we've fixed part
of it and don't want to make further changes as implied by the bug. But
we don't have a "partly fixed" resolution value. So to close this, I
have to pick an imperfect resolution value. I'm going to go with "works
for me", because the more recent comments here, as well as the proposed
patch, are all about the case where people *do* have session restore
enabled, which now has an option (that is also exposed in the
preferences/options, not just about:config) to do what this bug wants.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2018-12-12 Thread Plurtu
The fix for Bug 550559 did not end up resolving this issue,
browser.warnOnQuit = true still does not work with
browser.tabs.warnOnClose = false.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2018-12-12 Thread Shtetldik
The bug is currently unassigned, so it means that no one from Mozilla or
contributors is working on upstreaming the patch. May be someone should
acknowledge first, that the patch is acceptable.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2018-12-12 Thread Gabe25
we need to save on close (In some cases, not always)
We are not interested to turn in the options and each time select or deselect 
the option.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2018-12-12 Thread Gijskruitbosch+bugs
Comment on attachment 9004028
-fixquit.patch make warn=true warn again

Thanks for the suggested patch.

There's a plan and UX design in bug 550559, and that'll involve more
work than just changing this check. I'd take an updated patch in that
bug (I'll dupe this over in a sec). It looks like you'd need to also
update the label in the preferences ( https://searchfox.org/mozilla-
central/rev/c3fef66a5b211ea8038c1c132706d02db408093a/browser/components/preferences
/in-content/main.xul#88-89 - change the identifier for l10n there, and
change the identifier and string in https://searchfox.org/mozilla-
central/rev/c3fef66a5b211ea8038c1c132706d02db408093a/browser/locales/en-
US/browser/preferences/preferences.ftl#179-181 ), and there may be some
automated tests that need adjusting.

I'm also not sure what branch you wrote this patch against - but it
wouldn't apply against nightly, because the code has changed and also
checks the warnOnClose pref now (which it should probably keep doing, at
least for now).

Finally, for your next patch, please could you post something generated
by `hg export` or `git show` instead of just the diff, so that the
commit properly credits you and includes a summary of the change (ie
commit message). Thank you!

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2018-12-12 Thread Rik
I can't see any difference. Still not working.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2018-12-12 Thread Gijskruitbosch+bugs
*** This bug has been marked as a duplicate of bug 550559 ***

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2018-12-12 Thread Yumekui Neru
Created attachment 9004028
-fixquit.patch make warn=true warn again

found the bug
there was a second OR-condition that cancelled out the warning message if 
session restore is turned on, instead instantly closing the browser
removing this condition fixes the issue

maybe we finally can put this 10-20 year old issue behind us now c:

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 583797]

2018-12-12 Thread Josh Matthews
Comment on attachment 9004028
-fixquit.patch make warn=true warn again

Redirecting to someone who works on the Firefox frontend.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/583797

Title:
  "Warn on closing with multiple tabs open" not honored

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/583797/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

  1   2   >