1919471 - with bigger fonts, items in address book list overflow into eachother

Status: RESOLVED FIXED

(Thunderbird :: Address Book, defect)

Description

[seb128]

Attachment: address.png

Using thunderbird 130.0b2 or 128.2.2esr, I'm using bigger fonts in my config and the addressbook isn"t handling that correctly and cutting the items as shown on the screenshot

Comment 1

[Richard Marti (:Paenglab)]

Attachment: Bug 1919471 - Update AB row height/density to be font size dependant. r=#thunderbird-front-end-reviewers

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/473d0c6cb531
Update AB row height/density to be font size dependant. r=tobyp,micahilbery

Comment 3

[Geoff Lankow (:darktrojan)]

This breaks the row height in the table layout. It doesn't change with the density settings, and that's causing test failures. I'm going to back it out.

Comment 4

[Geoff Lankow (:darktrojan)]

Backed out:
https://hg.mozilla.org/comm-central/rev/7992a6a3731fdf3e2a72ef5db5a8912764d95870

(No need to respond to the needinfo request BTW, it's just for visibility.)

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/dd0ee25ca036
Update AB row height/density to be font size dependant. r=aleca,micahlibery,tobyp

Comment 6

[Richard Marti (:Paenglab)]

Comment on attachment 9426083 [details]
Bug 1919471 - Update AB row height/density to be font size dependant. r=#thunderbird-front-end-reviewers

[Approval Request Comment]
User impact if declined: Address Book contact lists not font sensitive
Testing completed (on c-c, etc.): on c-c
Risk to taking this patch (and alternatives if risky): low

Comment 7

[Corey Bryant]

Comment on attachment 9426083 [details]
Bug 1919471 - Update AB row height/density to be font size dependant. r=#thunderbird-front-end-reviewers

[Triage Comment]
This is already in comm-beta via yesterday's merge.