Closed Bug 190129 Opened 22 years ago Closed 22 years ago

don't show ldap addressbooks in "addressbook" Sender picker

Categories

(SeaMonkey :: MailNews: Message Display, defect)

Product:
SeaMonkey
Component:
MailNews: Message Display
Type:
defect
Priority:
Not set
Severity:
major

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
mozilla1.3beta

People

(Reporter: esther, Assigned: sspitzer)

References

Details

(Whiteboard: [mailviews])

Attachments

(1 file)

patch
1.25 KB, patch
sspitzer
: superreview+
asa
: approval1.3b+
Details | Diff | Splinter Review 
Using trunk build 20030119 on winxp, mac osx and linux the drop down list for
address books within the Customize maillist should not show LDAP directories.

1. Launch app, launch mail
2. In Address book add an LDAP directory
3. Select the "People I know" default view and select Edit
4. Expand the list to select which address book you want to use

Result: The LDAP directory is listed 
Expected: The LDAP directory should not be listed per developer.
QA Contact: laurel → esther
fix in hand, I'll attach.
Keywords: nsbeta1+
Target Milestone: --- → mozilla1.3beta
Attached patch patch — — Splinter Review 
hide mailing lists and remote (LDAP) addressbooks.

note, this hides mailing lists and LDAP addressbooks from the UI in filters,
search, and mailviews, which is what we want
Attachment #113019 - Flags: review?(cavin)
Status: NEW → ASSIGNED
QA Contact: esther → nbaca
Summary: [mailviews]- Customize, Address book drop down should not show LDAP directories → [mailviews] don't show ldap addressbooks in "addressbook" Sender picker
Summary: [mailviews] don't show ldap addressbooks in "addressbook" Sender picker → don't show ldap addressbooks in "addressbook" Sender picker
Whiteboard: [mailviews]
Comment on attachment 113019 [details] [diff] [review]
patch

this has r/sr=bienvenu over aim.
Attachment #113019 - Flags: superreview+
Comment on attachment 113019 [details] [diff] [review]
patch

seeking approval
Attachment #113019 - Flags: approval1.3b?
Comment on attachment 113019 [details] [diff] [review]
patch

a=asa (on behalf of drivers) for checkin to 1.3beta.
Attachment #113019 - Flags: approval1.3b? → approval1.3b+
fixed.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Attachment #113019 - Flags: review?(cavin)
using trunk build 20030205 on winxp, maco osx and linux this is fixed, verified
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: