1779034 - Remove obsolete chat services from SeaMonkey address book

Status: RESOLVED FIXED

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

Description

[Frank-Rainer Grahl (:frg)]

AIM and now Google Talk are dead. We still have list in the address book.

Comment 1

[Frank-Rainer Grahl (:frg)]

Attachment: 1779034-removeaim-25314.patch

We need to keep some definitions because of shared mailnews code. I also removed the IM button in the ab. It is useless. TB allows to open the internal chat but we only have support for irc and even then I fail to see the value of it. As a follow-up maybe add the matrix nickname to the chat name page.

[Approval Request Comment]
Regression caused by (bug #): --
User impact if declined: Usmess names in adress book
Testing completed (on m-c, etc.): 2.53.14
Risk to taking this patch (and alternatives if risky): trivial
String changes made by this patch: various removals.

Comment 2

[Ian Neal]

Comment on attachment 9284974 [details] [diff] [review]
1779034-removeaim-25314.patch

>+++ b/suite/mailnews/components/addrbook/content/abCardViewOverlay.js

>     // Chat section
>-    visible = cvSetNodeWithLabel(data.cvGtalk, zGtalk,
>-                                 card.getProperty("_GoogleTalk"));
>-    visible = cvSetNodeWithLabel(data.cvAIM, zAIM,
>-                                 card.getProperty("_AimScreenName")) || visible;
>     visible = cvSetNodeWithLabel(data.cvYahoo, zYahoo,
>                                  card.getProperty("_Yahoo")) || visible;
The || visible; needs removing from this line (see GoogleTalk line).

Is it worth amending the classic icons to remove the now unused icon(s)?

r/a=me with those fixed / addressed.

Comment 3

[Frank-Rainer Grahl (:frg)]

Attachment: 1779034-removeaim-v1_1-25314.patch

There was an error in the first patch which caused some header breakage. Given that it was just a matter of removing a call to no longer existing setFromBuddyIcon() I am keeping the r/a+ from IanN. Also some globals removed which were only used by setFromBuddyIcon().

I thought about fixing the classic icon strip but think can go in a follow-up because probably lots of css changes. Also maybe we can reuse them for something else later.

Comment 4

[Frank-Rainer Grahl (:frg)]

Attachment: 1779034-2-cleanup-25314.patch

Some cleanups var to let conversion and initializing.

[Approval Request Comment]
Regression caused by (bug #): --
User impact if declined: JJust not so nice code
Testing completed (on m-c, etc.): 2.53.14b1 pre
Risk to taking this patch (and alternatives if risky): trivial
String changes made by this patch: --

Comment 5

[Ian Neal]

Comment on attachment 9285754 [details] [diff] [review]
1779034-2-cleanup-25314.patch

[Triage Comment]
Good clean up r/a=me

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/b14a487f22ed
Remove dead AIM and Google Talk from Address book. r=IanN
https://hg.mozilla.org/comm-central/rev/642810cc383d
Clean up DisplayCardViewPane a bit. r=IanN

Comment 7

[Frank-Rainer Grahl (:frg)]

https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/af7bb6848193c9219349c91a67bd1434942b68d5
Remove dead AIM and Google Talk from Address book. r=IanN a=IanN
https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/81ef8184997917310423c5475b0b7823878f6d6a
Clean up DisplayCardViewPane a bit. r=IanN a=IanN