Closed Bug 220694 Opened 16 years ago Closed 14 years ago

Location should appear in alarm dialog

Categories

(Calendar :: Sunbird Only, enhancement)

x86
All
enhancement
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: lduperval, Assigned: robin.edrenius)

References

Details

Attachments

(1 file, 2 obsolete files)

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030624
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030624

Whe the alarm shows up, it give the time of the meeting or event, but it dosn't
show the location. In orter to view, you have to open the event. I think it
would be better to show the location in the dialog (or at least, allow it to be
configured).

Reproducible: Always

Steps to Reproduce:
1. Enter an event with a location
2. Enable alarm
3. When alarm appears, location isn't mentioned
I would like to "Second" the request for this feature. But I would like to add
support for also sending the location via email. Currently if you send an email
as an alarm then the location isn't included in the message.
Ditto on the whole thing.  This is the reason I switched back to outlook for my
calendar.  A pop-up alarm or e-mail that does not tell me the conference room
for a meeting or the phone number for the teleconference is completely useless.
Sorry for the bugspam, can we get this added to the alarm requirements list and
the associated tracker bug please?
Status: UNCONFIRMED → NEW
Component: Calendar General → Calendar Front End
Ever confirmed: true
OS: Linux → All
Blocks: cal-alarms
If people (like me) include the location in the "note", having an extra line
taken up for the "location" would waste valuable space in the alarm notification
window. 

I therefore suggest that fields that *can* appear in the alarm notification
window should only appear *if* they are not blank.

BTW: The title & note texts currently appear too far to the right in the alarm
notification window. There is a whole area in the center that is unused, and
should be used to increase the amount of possible text. I'll attach a screenshot
(I know this should be another bug, but i don't have time to file one right now,
and i hope it can be easily fixed by someone here.)
Attached image Screenshot showing some random other bug (obsolete) —
QA Contact: gurganbl → sunbird
Assignee: mostafah → robin.edrenius
Status: NEW → ASSIGNED
Attachment #200511 - Flags: first-review?(mvl)
Comment on attachment 200511 [details] [diff] [review]
Adds Location to the alarm-dialog

This will have to wait. Anything that is not a regression fix or fixing a major blocker will have to wait until after 0.3a1.
Made a change that was suggested by mvl per IRC.
Attachment #200511 - Attachment is obsolete: true
Attachment #200545 - Flags: first-review?(mvl)
Attachment #200511 - Flags: first-review?(mvl)
Attachment #152589 - Attachment description: Screenshot showing wasted space in alarm → Screenshot showing some random other bug
Attachment #152589 - Attachment is obsolete: true
Can you provide a screenshot of the alarm dialog with the location filled in? We don't want to make the dialog too cluttered.
Comment on attachment 200545 [details] [diff] [review]
remove un-necessary variable.

I tried it, and it doesn't look to bad.
(Although I do think that the dialog wants a redesign)

r=mvl
Attachment #200545 - Flags: first-review?(mvl) → first-review+
patch checked in
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.