198663 - [Patch] Switch multiweek and month's "F" commands.

Status: RESOLVED FIXED

(Calendar :: General, defect)

Description

[Pham]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.0.2) Gecko/20021216
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.0.2) Gecko/20021216

Since bug 198493 is fixed, you'll probably want to change

Multiweek F4
Month F3

to Multiweek F3
Month F4

Reproducible: Always

Steps to Reproduce:
1.
2.
3.

Comment 1

[Daniel Hrbac]

I agree, It would be more logical.

Comment 2

[Charles Mak]

Attachment: calendar.xul

Small patch (changes to lines 150 and 151) that makes the requested change.

Not sure if everyone actually wanted this change, but I just started trying to
figure out how calendar works and used this bug as a quick way to jump into the
code. Figured I might as well attach the code.

Comment 3

[Brant Gurganus]

Comment on attachment 120679 [details] [diff] [review]
calendar.xul

This patch is not a patch. A patch is created by using cvs diff -u (filename)
where filename is the path to the file checked out from CVS.  To learn about
CVS, see  http://www.mozilla.org/cvs.html

Comment 4

[Charles Mak]

Attachment: calendar.xul patch

Sorry about that, and thanks for pointing me in the right direction. How about
this attachment?

Comment 5

[Brant Gurganus]

That's the correct patch format.  Now, just request review from mikep and he
will take a look and add it to the repository if he approves.

Comment 6

[Charles Mak]

Ok, thanks. Adding "patch" to summary. Requesting review of patch.

Comment 7

[Mike Potter]

Approved and checked into cvs.
Thanks for the patch Charles!

Comment 8

[Reed Loden [:reed]]

The bugspam monkeys have been set free and are feeding on Calendar :: General. Be afraid for your sanity!