Closed Bug 1532109 Opened 5 years ago Closed 5 years ago

composition contacts sidebar does not display until all contacts are loaded

Categories

(Thunderbird :: Message Compose Window, enhancement)

enhancement
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 68.0

People

(Reporter: aceman, Assigned: aceman)

Details

Attachments

(2 files)

In Thunderbird's compose window, if you open the Contacts sidebar (F9), it shows all contact cards from the last used addressbook. That may take a while if you have thousands of cards. Until all are fetched the sidebar contents are not drawn and the space is empty.

Attached patch 1532109.patchSplinter Review
Assignee: nobody → acelists
Status: NEW → ASSIGNED
Attachment #9059661 - Flags: review?(mkmelin+mozilla)
Type: defect → enhancement
Comment on attachment 9059661 [details] [diff] [review]
1532109.patch

Review of attachment 9059661 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM thx, r=mkmelin
Attachment #9059661 - Flags: review?(mkmelin+mozilla) → review+

Thanks.

Keywords: checkin-needed

Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/8d373d4dbcb6
show contacts sidebar before searching contacts for the matching AB. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 68.0
Attached patch 1532109-2.patchSplinter Review

Sadly this broke the test test-send-button.js::test_send_enabled_address_contacts_sidebar on all debug platforms.
This should fix it.

Try run:
https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=b32e430dfef89ed4987a1505517a87ea30990800

Attachment #9061153 - Flags: review?(jorgk)
Comment on attachment 9061153 [details] [diff] [review]
1532109-2.patch

Thanks. Sadly especially Linux64 debug still looks like a mine field :-(
Attachment #9061153 - Flags: review?(jorgk) → review+
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/2ff21f9b6206
Follow-up: fix test of contact sidebar results. r=jorgk DONTBUILD
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: