Filter UI: Action section is missing descriptive text

VERIFIED FIXED in mozilla0.9.1

Status

MailNews Core
Filters
P4
normal
VERIFIED FIXED
18 years ago
10 years ago

People

(Reporter: laurel, Assigned: gerv)

Tracking

Trunk
mozilla0.9.1

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(3 attachments)

(Reporter)

Description

18 years ago
Using may24 m16 commercial build

refer to mail filter spec:
http://gooey/client/5.0/specs/mail/filters/Filters.html

The filter rules dialog's Action section needs to have descriptive text, current
spec revision 4/19/00 shows it to be:

"When messages match the above criteria perform the following action:"
(Reporter)

Updated

18 years ago
QA Contact: lchiang → laurel

Updated

18 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → M17

Comment 1

18 years ago
mass moving to M18 and adding nsbeta3 keyword
Keywords: nsbeta3
Target Milestone: M17 → M18

Updated

18 years ago
Whiteboard: [nsbeta3-]
Target Milestone: M18 → Future

Updated

18 years ago
Assignee: alecf → gayatrib
Status: ASSIGNED → NEW

Comment 2

18 years ago
reassigning my filter bugs to gayatrib..

Comment 3

17 years ago
Got a fix in my tree for this - taking.
Assignee: gayatrib → hwaara
Keywords: nsbeta3
Whiteboard: [nsbeta3-]
Target Milestone: Future → mozilla0.9.1

Updated

17 years ago
Priority: P3 → P4
hwaara - will this do? Could you review, please?

Gerv
Assignee: hwaara → gervase.markham
Re-ccing hwaara. Hwaara - I attached a patch when you weren't looking ;-)

Gerv

Comment 8

17 years ago
Thanks for fixing this. r=hwaara
Keywords: approval, patch
thanks for working on this.  your wording doesn't match the spec.

http://www.mozilla.org/mailnews/specs/filters/

Comment 10

17 years ago
What Seth said. Just replace "criteria" with "conditions" and you're done.
It's nice to have an accessible URL for the spec at last. Silly me for looking
at the bug instead of hunting it down ;-)

Seriously, another patch coming right up.

Gerv
sr=sspitzer

note:  you might not need to do orient="vertical".   I think the titlebox should
orient that way by default.

Comment 14

17 years ago
my r= persists. gerv, do you want me to check this in for you?
Sure. If you want to check out whether seth is correct and modify the patch 
accordingly, feel free :-) I'm currently in the wrong OS, and in the middle of 
some Distributed Systems revision ;-)

Gerv
I checked - orient=vertical is necessary. The default for <titlebox> is
horizontal.

Checked in by hwaara. Marking fixed.

Gerv
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
thanks for checking about the default for orientation before checking in.

(perhaps it is box that is vertical by default?)
(Reporter)

Comment 18

17 years ago
OK using may17 commercial trunk build
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.