Closed Bug 974146 Opened 11 years ago Closed 6 years ago

Update socialapi tests to use PopupNotifications.getNotification()

Categories

(Firefox Graveyard :: SocialAPI, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: Dolske, Unassigned)

References

Details

Bug 967349 adds an API so that things using doorhangers (PopupNotifications) can simply add a "Learn More" link by specifying a URL (vs manually jamming bits into the DOM, or using <popupnotification> markup to add one). That patch removes the <label id="servicesInstall-learnmore-link" class="text-link"/> from the services, but keeps the otherwise-empty <popupnotification> due to a _bunch_ of tests doing |let panel = document.getElementById("servicesInstall-notification");|. These should be switched to PopupNotifications.getNotification(id,browser), but given the scale of the changes I didn't want to block on that. Once that's done we can remove the <popupnotification id="servicesInstall-notification">
deprecation in fx51
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
What does "deprecation" mean exactly? Can we now remove the <popupnotification> element from popup-notifications.inc without worrying about the 3 remaining tests or should we just remove the tests as well?
Flags: needinfo?(mixedpuppy)
Most of the socialapi functionality was deprecated/removed, but share remains, and install notification also needs to remain. I suppose this should be reopened.
Status: RESOLVED → REOPENED
Flags: needinfo?(mixedpuppy)
Resolution: WONTFIX → ---
SocialAPI was removed from Firefox 57 and is no longer available in any current release.
Status: REOPENED → RESOLVED
Closed: 8 years ago6 years ago
Resolution: --- → WONTFIX
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.