Allow making reminder alert popups modal (config option)

ASSIGNED
Assigned to

Status

Calendar
Alarms
ASSIGNED
9 years ago
6 months ago

People

(Reporter: Hans Deragon, Assigned: Merouane Atig, Mentored)

Tracking

Details

(Whiteboard: [good first bug][lang=js])

Attachments

(1 attachment)

(Reporter)

Description

9 years ago
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9b5) Gecko/2008050509 Firefox/3.0b5
Build Identifier: Lightning 0.8, Build 2008033118

The reminder alert popups are not modal.  Thus when a reminder pops up, it remains behind other windows and often goes unnoticed.  Of course, this render reminders useless.

They should be system wide modal.  Or at least, this should be the default option with the option of disabling it in the preferences if people prefer to have the choice.

Reproducible: Always
(Reporter)

Comment 1

9 years ago
Moved this bug to 'critical'.  The #1 feature of a Calandar program is to remind people about events.  If alerts show up under other windows, the whole usefulness of the application is lost.
Severity: normal → critical
Flags: blocking-calendar0.9+
Version: unspecified → Trunk

Comment 2

9 years ago
This request is similar Bug 250137, Bug 267925, Bug 313645, Bug 321910, Bug 377806, Bug 388073. 

In Bug 202620 it was decided to not force the window on top but use the platform dependent notification system (e.g. flash the taskbar, bounce the dock icon). 

I also proposed an alarm slider that displays on top of all windows but that proposal was requested (Bug 329676).
Severity: critical → normal
Flags: blocking-calendar0.9+
Version: Trunk → unspecified
(Reporter)

Comment 3

9 years ago
Well, under Ubuntu 08.04, the current solution is unacceptable.

As I mentioned before, make it the default but configurable in the Preferences.

We need a solution that remains on top of all windows and require the user to
click on it to remove it.  I understand the problem with modal windows, but it
is still, of all solutions, the best under Ubuntu 08.04.  I, as a user, want a
modal window that disturb me in what I am doing.  At least, give me this
option.

I started with Lightning last Friday and today I will revert back to Korganizer because of this issue.  Even if I have to manually enter the invitations into Korganizer, at least because of its modal alerts I will not miss any meetings.  I will continue to follow Lightning development and hope that a reliable solution will emerge.

Can someone switch the status of this bug report to 'NEW'?
Component: Lightning Only → Alarms
QA Contact: lightning → alarms
I doubt we should make this a user visible pref, but I can imagine a hidden pref in about:config. Maybe the prefname being:

calendar.alarms.showModal
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: Reminder alert popups are not modal → Allow making reminder alert popups modal (config option)
Whiteboard: [good first bug]

Comment 5

7 years ago
Likewise for me reminders are nearly useless because of this.  Running Fedora 14 KDE.

Comment 6

7 years ago
This problem makes the alarms basically worthless.  There is a about:config option (on Lightning under Postbox on MacOS) that is called ui.use_native_popup_windows and defaults to false.  However, changing this to true just screws up all popups.
Whiteboard: [good first bug] → [good first bug][mentor=Fallen][lang=js]
Duplicate of this bug: 724895

Comment 8

4 years ago
Why isn't this getting any attention? Everyone is right, the alert is useless if you aren't being alerted. For this reason alone I have moved to using Chrome. I love FF for firebug and would love to use it more - I will not until this is fixed.

Comment 9

4 years ago
This apparently change in TB 24 (at least in ubuntu precise), now the reminders list keep in forefront FOR EVER, the problem is that you can not send it to back, so now you can only use any other window of TB (main window, new message, ...) moving the reminder window around and around...

Comment 10

4 years ago
(In reply to Hernan from comment #9)
> This apparently change in TB 24 (at least in ubuntu precise), now the
> reminders list keep in forefront FOR EVER, the problem is that you can not
> send it to back, so now you can only use any other window of TB (main
> window, new message, ...) moving the reminder window around and around...

Exact same issue. Why on earth is it desirable to make this modal? Sigh...
Because it is meant to be a list of reminders: either you snooze them, or you dismiss them. If you just leave them open it turns into yet another todo list. Anyway, both methods are valid.

The change in tb24 is probably related to the window manager or the Mozilla Platform. We didn't change any code that would affect this, but possibly something further down is influencing this.

Comment 12

4 years ago
(In reply to Philipp Kewisch [:Fallen] from comment #11)
> Because it is meant to be a list of reminders: either you snooze them, or
> you dismiss them. If you just leave them open it turns into yet another todo
> list. Anyway, both methods are valid.
> 
> The change in tb24 is probably related to the window manager or the Mozilla
> Platform. We didn't change any code that would affect this, but possibly
> something further down is influencing this.

Fair enough. But an option to restore the non-modal behavior would be desirable. Thanks for the explanation. ;-)
Mentor: philipp@bugzilla.kewis.ch
Whiteboard: [good first bug][mentor=Fallen][lang=js] → [good first bug][lang=js]

Comment 13

2 years ago
I willing to start work on the reminders if there is some support.  Please see
https://bugzilla.mozilla.org/show_bug.cgi?id=1092552#c4

Updated

2 years ago
Flags: needinfo?(dauge)

Updated

2 years ago
Flags: needinfo?(david)

Updated

2 years ago
Flags: needinfo?(etiffany)

Updated

2 years ago
Flags: needinfo?(hernan)

Updated

2 years ago
Flags: needinfo?(khaos119)

Updated

2 years ago
Flags: needinfo?(orion)

Updated

2 years ago
Flags: needinfo?(orion)

Comment 14

2 years ago
I'm currently using TB on the Mac and not seeing the same behavior, so not sure I can add much info. Still, if you need something, happy to help where I can. Should I clear the needinfo flag?
Flags: needinfo?(david)
Please don't needinfo everyone just to get a comment, I've added the right needinfo on the bug you linked.
Flags: needinfo?(khaos119)
Flags: needinfo?(hernan)
Flags: needinfo?(etiffany)
Flags: needinfo?(dauge)

Comment 16

2 years ago
(In reply to Philipp Kewisch [:Fallen] from comment #15)
> Please don't needinfo everyone just to get a comment, I've added the right
> needinfo on the bug you linked.

This bug is also affected if work on bug-id 1092552 is started.  So, it's reasonable to have also some feedback of people interested in this bug.
In that case a comment asking folks specific questions should be sufficient, they will all get email because they are on CC. You can needinfo "anyone" if you like. I guess its just formalities, lets not get hung up on it.

Anyway, I think its quite clear what we need in this bug: add a pref that allows making the alarm dialog modal, default to off. I'm not yet sure if this should be a user-visible pref, but we can reiterate this question when the code for the hidden pref is ready.
Assignee: nobody → info
Status: NEW → ASSIGNED

Comment 18

2 years ago
Haven't used Firefox in over a year for this reason. Also, as a web developer, it's too slow for me. Removing myself from this list. Good luck!
(Assignee)

Updated

2 years ago
Duplicate of this bug: 919168
(Assignee)

Comment 20

2 years ago
I would like to work on this issue. Is it OK to reassign this ticket to me?

I already isolated the part where the alarm window is open as a dialog: https://dxr.mozilla.org/comm-central/source/calendar/base/src/calAlarmMonitor.js#126

I now have to create a hidden pref and set the dialog parameter accordingly. I'm thinking about calling it "calendar.alarms.modal" or "calendar.alarms.alwaysontop".
Flags: needinfo?(philipp)
(Assignee)

Comment 21

2 years ago
Created attachment 8640131 [details] [diff] [review]
433531_v1.patch
Attachment #8640131 - Flags: review?(philipp)
Comment on attachment 8640131 [details] [diff] [review]
433531_v1.patch

Review of attachment 8640131 [details] [diff] [review]:
-----------------------------------------------------------------

Does dialog=yes actually make it modal? IIUC then it was this way before, and for me on mac the reminder dialog is not modal. I'd appreciate if we could make sure the behavior is the same on all platforms. I'd expect this:


showModal = true: When the dialog shows up, the parent calendar window is not actionable until all reminders are dismissed
showModal = false: When the dialog shows up, it appears on top at first, but it can be switched to the main window with alt+tab

In that case, the default for showModal should be false, because this is how it has been for me before. Is this also what you would expect?

::: calendar/base/src/calAlarmMonitor.js
@@ +124,5 @@
>              this.mWindowOpening = Services.ww.openWindow(
>                  null,
>                  "chrome://calendar/content/calendar-alarm-dialog.xul",
>                  "_blank",
> +                "chrome,dialog=" + isDialog +",all,resizable",

nit: space after second +
Attachment #8640131 - Flags: review?(philipp) → feedback+
Assignee: info → yahoocbien-dev
Flags: needinfo?(philipp)
You need to log in before you can comment on or make changes to this bug.