Mailing List Dialog for OS X address books doesn't list addresses and isn't read-only

VERIFIED FIXED in Thunderbird 3.0a3

Status

P2
normal
VERIFIED FIXED
10 years ago
10 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

(Depends on: 1 bug)

Trunk
Thunderbird 3.0a3
All
Mac OS X
Dependency tree / graph
Bug Flags:
blocking-thunderbird3 +
in-testsuite ?
in-litmus ?

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [litmus test required])

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

10 years ago
When examining the properties of a mailing list which is in a linked Mac OS X Address Book, there are two problems:

1) The list of addresses doesn't appear.
2) The dialog appears to be read-write, which it isn't. It should be read-only like we have the main card dialog.

Blocking-TB 3 as this is a highlighted feature.
Flags: blocking-thunderbird3+
(Assignee)

Updated

10 years ago
Assignee: nobody → bugzilla
(Assignee)

Updated

10 years ago
Priority: -- → P2
(Assignee)

Comment 1

10 years ago
Created attachment 327403 [details] [diff] [review]
[checked in] Provide read-only detection for mailing list dialogs.

I wanted to come up with something for this before TB 3 alpha 2. This I think is the best I can do without writing a separate read-only dialog, or rewriting how most of it is working.

All the elements are read-only, apart from the list box which is disabled. We can always improve it later if we want to, but I'm hoping we're going to improve the whole dialog, not just the read-only part ;-)
Attachment #327403 - Flags: superreview?(bienvenu)
Attachment #327403 - Flags: review?(bienvenu)

Comment 2

10 years ago
Comment on attachment 327403 [details] [diff] [review]
[checked in] Provide read-only detection for mailing list dialogs.

Looks good - one comment nit:

read-only? if so,

should be "If so", not "if so".
Attachment #327403 - Flags: superreview?(bienvenu)
Attachment #327403 - Flags: superreview+
Attachment #327403 - Flags: review?(bienvenu)
Attachment #327403 - Flags: review+
(Assignee)

Updated

10 years ago
Attachment #327403 - Attachment description: Provide read-only detection for mailing list dialogs. → [checked in] Provide read-only detection for mailing list dialogs.
(Assignee)

Comment 3

10 years ago
The fix for displaying addresses in the list box is in bug 437903.
Depends on: 437903
(Assignee)

Comment 4

10 years ago
The second part of this bug (actually listing the addresses in the mailing list dialog) has been fixed by bug 437903 attachment 328102 [details] [diff] [review].

Therefore this bug is also completely fixed.
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.1a1
Mark, sorry if I'm wrong but this bug isn't fully fixed. The listbox isn't read-only. I can still use the keyboard to add/modify/delete entries. Ok, after clicking on Ok the changed values weren't applied but IMO the listbox should be disabled.

I'll reopen the bug until you can give a clear statement. It's bad to have two separate issues here.
Status: RESOLVED → REOPENED
Hardware: PC → All
Resolution: FIXED → ---
(Assignee)

Comment 6

10 years ago
Created attachment 330001 [details] [diff] [review]
Fix initialisation of directories/mailing lists

So the problem with the code at the moment, is that the maintained lists (of cards) are only filled in when GetChildCards is called. This is giving the effect that if you don't show the properties of the list before you send the email (aka call GetChildCards), then it won't send because it doesn't know about any items in that list.

This patch fixes that by moving the initialisation to the Init function (how original!), and doing the appropriate returns in GetChildCards. I think its still sensible to leave the GetChildCards function sorting out what is actually in the list for queries as that's how the rest of the address book types do it.
Attachment #330001 - Flags: superreview?(bienvenu)
Attachment #330001 - Flags: review?(bienvenu)
(Assignee)

Comment 7

10 years ago
Comment on attachment 330001 [details] [diff] [review]
Fix initialisation of directories/mailing lists

Opps, wrong bug. I'd better move it across.
Attachment #330001 - Attachment is obsolete: true
Attachment #330001 - Flags: superreview?(bienvenu)
Attachment #330001 - Flags: review?(bienvenu)
Product: Core → MailNews Core

Updated

10 years ago
Target Milestone: mozilla1.9.1a1 → Thunderbird 3.0b1
(Assignee)

Updated

10 years ago
Depends on: 452192
(Assignee)

Comment 8

10 years ago
(In reply to comment #5)
> Mark, sorry if I'm wrong but this bug isn't fully fixed. The listbox isn't
> read-only. I can still use the keyboard to add/modify/delete entries. Ok, after
> clicking on Ok the changed values weren't applied but IMO the listbox should be
> disabled.

I've filed bug 452192 on this issue, that I'll look at after b1 alongside some other changes I am thinking of.

Therefore closing this bug down.
Status: REOPENED → RESOLVED
Last Resolved: 10 years ago10 years ago
Resolution: --- → FIXED
Verified with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b1pre) Gecko/20080830025944 Shredder/3.0b1pre ID:20080830025944
Status: RESOLVED → VERIFIED
Flags: in-testsuite?
Flags: in-litmus?

Comment 10

10 years ago
This FIXED bug is flagged with in‑testsuite?   It would be great if assignee or someone else can clear the flag if a test is not appropriate.  And if appropriate, create a test and plus the flag to finish off the bug.
(Assignee)

Updated

10 years ago
Whiteboard: [litmus test required]
You need to log in before you can comment on or make changes to this bug.