142148 - Sidebar Address Book, sort widget remains in descending order

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Address Book & Contacts, defect)

Description

[Ninoschka Baca]

Branch build 2002-05-03: WinMe, Mac 10.1.3, Linux RH 7.1

Overview: In the Sidebar Address Book select the Name column and the sort widget
changes from ascending to descending order and remains in the descending state.

Steps to reproduce:
1. Open a profile that has address book entries
2. Add the Address Book to the Sidebar
3. Select the Name column and it should change the sort
4. Select the Name column multiple times to change the sort order

Actual Results: The cards are resorted but the sort widget remains in descending
order.

Expected Results: The directin of the widget should reflect the actual state of
the cards

Comment 1

[Ninoschka Baca]

Marking nsbeta1 for RTM consideration even though I realize that this is
probably a low priority. It would be nice for the widget to reflect the actual
sort order of the cards.

Comment 2

[scottputterman]

nsbeta1- per mail triage.

Comment 3

[Ninoschka Baca]

Marking nsbeta1 so that the Sidebar AB widgets reflect the true
ascending/descending state.

Comment 4

[jglick]

I still this behavior. 2002121608, windows.

Comment 5

[Samir Gehani]

Mail triage team: nsbeta1+/adt2

Comment 6

[Cavin Song]

Reassigning.

Comment 7

[Cavin Song]

Reassigning.

Comment 8

[shliang]

Attachment: patch

Comment 9

[Cavin Song]

Comment on attachment 113609 [details] [diff] [review]
patch

r=cavin.

Comment 10

[(not reading, please use [e-mail] instead)]

Comment on attachment 113609 [details] [diff] [review]
patch

the id of "addressbookList" is used by both the addressbook sidebar panel and
the tree in the select addresses dialog.  (the one you get from the compose
window if you hit the "Address" toolbar button)

it looks like that dialog has the same bug as the addressbook sidebar panel.

from abCommon.js:

function InitCommonJS()
{
  dirTree = document.getElementById("dirTree");
  abList = document.getElementById("addressbookList");
  gAbResultsTree = document.getElementById('abResultsTree');
}

Can you just do:

-  if (dirTree && gAbView) {
+  if ((dirTree || abList) && gAbView) {

while your fix would work, this removes the extra call to getElementById().

Can you verify that it fixes both the sidebar panel and the select addresses
dialog?

(you should look for this bug on the select addresses dialog and take it, as
you'll be fixing it, too.)

Comment 11

[shliang]

Attachment: changing to abList

Comment 12

[shliang]

and it does fix both the sidebar panel and select addresses dialog

Comment 13

[(not reading, please use [e-mail] instead)]

Comment on attachment 114291 [details] [diff] [review]
changing to abList

r/sr=sspitzer

nice work.

asking for 1.3 final approval on shuehan's behalf.

shuehan, before you check in, could you add a comment to this code indicating
why we need to check both dirTree and abList?

Comment 14

[Asa Dotzler [:asa]]

Comment on attachment 114291 [details] [diff] [review]
changing to abList

a=asa (on behalf of drivers) for checkin to 1.3 final.

Comment 15

[shliang]

marking fixed.

Comment 16

[Ninoschka Baca]

Trunk build 2003-03-03: WinXP, Linux RH 8, Mac 10.1.5
Verified Fixed. 

Note: Could only check the Sidebar Address Book in the Browser since it is no
longer available in any Mail component.