Event dialog: Reminder List contains too many items

RESOLVED FIXED in 1.0b2

Status

RESOLVED FIXED
12 years ago
9 years ago

People

(Reporter: chris.j.bugzilla, Assigned: Fallen)

Tracking

Trunk
1.0b2

Details

(Whiteboard: [needed beta][has l10n impact])

Attachments

(1 attachment)

(Reporter)

Description

12 years ago
The current Reminder contains too many items:
It should be reduced to a readable number of items:

No Reminder
-----------------
5 min before 
10 min before 
30 min before
 ------------------
1 hour before
2 hours before
15 hours before
 -------------------
1 day before
2 days before
7 days before
--------------------
Edit Custom Reminders...
What would be a readable (and useful) number of items?

  15 minutes before
  1  hour befor
  1  day before
  Custom Reminder...
What about making the default reminder values configurable? 

Instead of hard-coding the default reminders into the xul file there could be e.g. a preference containing as comma separated list of default values. For example "15, 60, 1440". Upon opening the dialog this would be parsed to "15 minutes", "1 hour" and "1 day". 

This would make it easier to change the default set of reminders or to customize the default reminders to the users requirements e.g. via about:config or extension.
Component: General → Theme
Product: Calendar → Firefox
Target Milestone: --- → Firefox 3
Version: unspecified → Trunk
Oops, sorry, my bad, returning to the right component, awfully sorry about stomping your target milestone, please send hatemail to this address :(
Component: Theme → General
Product: Firefox → Calendar
Target Milestone: Firefox 3 → ---
Version: Trunk → Sunbird 0.7

Comment 4

11 years ago
> The current Reminder contains too many items

> What about making the default reminder values configurable?
> e.g. via about:config or extension

Why not combine your ideas and make the list of available reminders in the dropdown a mirror of the list in the Custom dialog?  This way it would be easy for us to customize the choices in the dropdown to our own needs and we wouldn't need to open the Custom dialog as often.

In this case I would change the word in the dropdown from "Custom..." to "Customize..." to emphasize that the purpose would be to customize the values in the dropdown.

Updated

10 years ago
Assignee: nobody → brennan.vincent
OS: Windows XP → All
Hardware: x86 → All
Summary: [Proto] Event dialog: Reminder List contains too many items → Event dialog: Reminder List contains too many items
Component: General → Dialogs
QA Contact: general → dialogs
(Assignee)

Comment 5

9 years ago
(In reply to comment #4)
> Why not combine your ideas and make the list of available reminders in the
> dropdown a mirror of the list in the Custom dialog?  This way it would be easy
> for us to customize the choices in the dropdown to our own needs and we
> wouldn't need to open the Custom dialog as often.
Given the "Custom" option has changed to now show multiple reminders, I think we should leave this to an extension.

It would make a very nice extension. I might actually do this soon.


Do we want to further shorten the list? I'd suggest the following:



No Reminder
-----------------
0 min before 
5 min before
15 min before 
30 min before
 ------------------
1 hour before
2 hours before
12 hours before
 -------------------
1 day before
2 days before
1 week before
--------------------
Edit Custom Reminders...



(Brennan, I hope its ok I reassign this bug to myself?)
Assignee: brennan.vincent → philipp
Status: NEW → ASSIGNED
Version: Sunbird 0.7 → Trunk
(Assignee)

Comment 6

9 years ago
I've just uploaded the addon previously promised to
https://addons.mozilla.org/en-US/thunderbird/addon/97408/

Comment 7

9 years ago
Thanks Philipp, the extension to edit the list of custom alarms works like a charm.  The only problem is that the rows in the list don't highlight when I click them (but the separator rows do).  I'm referring to "Tools > Addons > Custom Alarms > Options".  I'm using WinXP ("Classic Theme" if it matters) and TB 3.0.1 official.
(Assignee)

Comment 8

9 years ago
Pete, likely a windows-only issue. I'll see what I can do!
Comment on attachment 437808 [details] [diff] [review]
Fix - v1

This patch takes care, based on my above suggestion in comment 5.

Requesting ui-review to make sure its the right set.
Attachment #437808 - Flags: ui-review?(clarkbw)
Attachment #437808 - Flags: review?(Mozilla)
Comment on attachment 437808 [details] [diff] [review]
Fix - v1

patch looks good and works as expected.

r=markus
Attachment #437808 - Flags: review?(Mozilla) → review+
Comment on attachment 437808 [details] [diff] [review]
Fix - v1

looks good
Attachment #437808 - Flags: ui-review?(clarkbw) → ui-review+
Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/ccabde3874e3>
-> FIXED
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.0b2
(Assignee)

Updated

9 years ago
Whiteboard: [needed beta][has l10n impact]
You need to log in before you can comment on or make changes to this bug.