Recurrence dialog ignores recurrence units

RESOLVED FIXED

Status

Calendar
Lightning Only
RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: Stephan Schaefer, Assigned: Stephan Schaefer)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

12 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8) Gecko/20051111 Firefox/1.5
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8) Gecko/20051111 Firefox/1.5

The recurrence units (repeat for n days/weeks/months) are not respected. As there is no useful internal represenation possible for the unit it should just read 'occurences'.

Reproducible: Always

Steps to Reproduce:
1. Create a weekly recurring event
2. Let it repeat for 2 month
3. 

Actual Results:  
It is repeated only twice.

Expected Results:  
Repeated 8 times.
(Assignee)

Comment 1

12 years ago
Created attachment 210611 [details] [diff] [review]
Removes recurrence units menu.

As it is quite difficult to properly restore the unit upon editing the recurrence rule it should always be treated as a plain count. The patch removes the units menu and replaces it with a simple label:
Repeat for: n occurrence(s)
Assignee: nobody → stephan.schaefer
Status: NEW → ASSIGNED
Attachment #210611 - Flags: first-review?(jminta)

Comment 2

12 years ago
Comment on attachment 210611 [details] [diff] [review]
Removes recurrence units menu.

It was a mildly interesting concept to have units there, but this seems much simpler/better.  Nice work. r=jminta
Attachment #210611 - Flags: first-review?(jminta) → first-review+

Comment 3

12 years ago
Patch checked in. :-)
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.