Closed Bug 1774765 Opened 2 years ago Closed 2 years ago

Add support for pager, fax, mobile, ... types for vCard telephone numbers in the edit UI

Categories

(Thunderbird :: Address Book, enhancement)

enhancement

Tracking

(thunderbird_esr102 verified, thunderbird102 affected)

VERIFIED FIXED
103 Branch
Tracking Status
thunderbird_esr102 --- verified
thunderbird102 --- affected

People

(Reporter: TbSync, Assigned: darktrojan)

Details

Attachments

(2 files)

Attached image contact.png

The edit UI only has "Home", "Work" and "None" in the dropdown type selector, but the overview UI correctly displays the additional type info.

Assignee: nobody → geoff
Status: NEW → ASSIGNED

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/3b73aa35ddad
Add mobile, fax and pager types to Address Book card editor. r=#thunderbird-reviewers,aleca

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 103 Branch

Comment on attachment 9282305 [details]
Bug 1774765 - Add mobile, fax and pager types to Address Book card editor. r=#thunderbird-reviewers

[Approval Request Comment]
Regression caused by (bug #): new Address Book
User impact if declined: can't make a phone number marked as mobile or fax
Testing completed (on c-c, etc.): landed last week
Risk to taking this patch (and alternatives if risky): no

Attachment #9282305 - Flags: approval-comm-beta?

Comment on attachment 9282305 [details]
Bug 1774765 - Add mobile, fax and pager types to Address Book card editor. r=#thunderbird-reviewers

Wrong flag.

Attachment #9282305 - Flags: approval-comm-beta? → approval-comm-esr102?

Comment on attachment 9282305 [details]
Bug 1774765 - Add mobile, fax and pager types to Address Book card editor. r=#thunderbird-reviewers

[Triage Comment]
Approved for esr102

Let's explicitly QA/test address book in RC2, to make sure there are no regressions or ill effects

Flags: needinfo?(wls220spring)
Flags: needinfo?(bugzilla2007)
Attachment #9282305 - Flags: approval-comm-esr102? → approval-comm-esr102+

Verified testing the 102.0 rc and release on Windows and Fedora Linux

Status: RESOLVED → VERIFIED
Flags: needinfo?(wls220spring)

I have continuously tested the new AB before and after release and filed bugs against it.

Tried a fully filled contact card from TB 91 today and updated that to TB 102. Quite time-consuming...

  • pager, fax, mobile etc. all migrated perfectly (this bug 1774765, VERIFIED FIXED)

More findings:

  • web addresses without protocol like http:// not shown in display - odd. Also blocks saving other edits like name to the card - worse.
  • Filed Bug 1779567 - Reversed order of primary street address and extended address after update to 102
  • Birthday of 29 Feb (without year, from TB91) displays as 1 March. Also blocks saving other edits. Bug 1779100 territory.

Tested and same result for 102.0.2 (64-bit) and 104.0a1 (2022-07-13) (64-bit), Win10.

Flags: needinfo?(bugzilla2007)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: