Closed Bug 594081 Opened 14 years ago Closed 14 years ago

"Check online presence (AOL)" should be removed from address book tools menu

Categories

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

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1b1

People

(Reporter: iav, Assigned: sgautherie)

References

()

Details

Attachments

(1 file)

"Check online presence AOL" should be removed from address book tools menu
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED
Flags: in-testsuite-
OS: Windows XP → All
Summary: "Check online presence AOL" should be removed from address book tools menu → "Check online presence (AOL)" should be removed from address book tools menu
Depends on: 295726
(In reply to comment #0)
> "Check online presence AOL" should be removed from address book tools menu

Why? Last I checked it was still working (alright that was a year or two ago), but if you're going to file a bug you should give the reason as to why.
No longer depends on: 295726
Maybe, I am wrong... I not see this feature work since Netscape Coppunicator bundled with AOL client. Then I think it's just obsoleted non-functional legacy.

But, if it still usable - ok, let it be.
Comment on attachment 473069 [details] [diff] [review]
Remove outdated code from SeaMonkey and Thunderbird
[Checked in: Comment 10]

So from a SeaMonkey perspective, the only bits that I can see that are currently broken are the image to display the online presence (if the menu option is checked). Namely:

http://ncmail.netscape.com/include/nc/images/offline.gif
http://ncmail.netscape.com/include/nc/images/online.gif

do not exist any more (they used to resemble little people).

So they should either be fixed, or this code removed. That decision is up to Karsten or Neil.

In the Thunderbird case, I think we can loose that code. However, I'd prefer that to go on a Thunderbird bug so that we can track it more easily and keep it separate from the SeaMonkey decision that needs making here.
Attachment #473069 - Flags: review?(bugzilla) → review-
Comment on attachment 473069 [details] [diff] [review]
Remove outdated code from SeaMonkey and Thunderbird
[Checked in: Comment 10]

(In reply to comment #4)
> http://ncmail.netscape.com/include/nc/images/offline.gif
> http://ncmail.netscape.com/include/nc/images/online.gif
> do not exist any more (they used to resemble little people).

Fwiw, that's the only thing I had checked before starting to create my patch :-|

> So they should either be fixed, or this code removed. That decision is up to
> Karsten or Neil.

Asking for feedback...

NB:
Iiuc, the feature is useless if we don't have/fix some kind of working UI for it.
Fwiw, I think this feature was/is nice but should be an extension nowadays (if someone wants to do one).

> In the Thunderbird case, I think we can loose that code. However, I'd prefer
> that to go on a Thunderbird bug so that we can track it more easily and keep it
> separate from the SeaMonkey decision that needs making here.

I did TB part too because I assumed abCardViewOverlay.xul was shared between the two apps and I didn't wan't to try to "ifdef" that code...
Was I wrong? Would there be a solution not to remove the whole feature (if that's what we want) from both SM and TB at the same time?
Attachment #473069 - Flags: feedback?(neil)
Attachment #473069 - Flags: feedback?(mnyromyr)
Ftr,
http://api.oscar.aol.com redirects to http://dev.aol.com/aim
{
Going forward, we're shifting our focus to select partnership opportunities that will help us move the needle in the communications space.
}
Comment on attachment 473069 [details] [diff] [review]
Remove outdated code from SeaMonkey and Thunderbird
[Checked in: Comment 10]

Since we removed most remnants of our commercial inheritance anyway by now (eg AIM buddy icons), I don't think there's much justification left to actually include AIM notifications... (And it's not even open source. ;-) )
Neil, do you see any higher reason to keep it?

f+=me (not a formal review)
Attachment #473069 - Flags: feedback?(mnyromyr) → feedback+
Comment on attachment 473069 [details] [diff] [review]
Remove outdated code from SeaMonkey and Thunderbird
[Checked in: Comment 10]

No point keeping a broken feature if there's nobody around to fix it.
Attachment #473069 - Flags: feedback?(neil) → feedback+
Comment on attachment 473069 [details] [diff] [review]
Remove outdated code from SeaMonkey and Thunderbird
[Checked in: Comment 10]

(In reply to comment #4)

> So they should either be fixed, or this code removed. That decision is up to
> Karsten or Neil.

Then, both agree to remove the whole feature.

> In the Thunderbird case, I think we can loose that code. However, I'd prefer
> that to go on a Thunderbird bug so that we can track it more easily and keep it
> separate from the SeaMonkey decision that needs making here.

Unless the answer to my comment 5 question(s) is that SM and TB cases can be separated, you can move this very bug to TB if you prefer.
Attachment #473069 - Flags: review?(bugzilla)
Attachment #473069 - Flags: review?(bugzilla) → review+
Comment on attachment 473069 [details] [diff] [review]
Remove outdated code from SeaMonkey and Thunderbird
[Checked in: Comment 10]

http://hg.mozilla.org/comm-central/rev/80df3f094abc
Attachment #473069 - Attachment description: Remove outdated code from SeaMonkey and Thunderbird → Remove outdated code from SeaMonkey and Thunderbird [Checked in: Comment 10]
Attachment #473069 - Flags: review-
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b1
(In reply to comment #9)
> Unless the answer to my comment 5 question(s) is that SM and TB cases can be
> separated, you can move this very bug to TB if you prefer.

Ftr, actually they could have, but I figured it out after the fact (because TB still packages the file):
I filed bug 599726.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: