Opened on 10/29/2018 at 12:25:37 PM

Closed on 10/31/2018 at 06:23:30 PM

#7077 closed defect (fixed)

Critical notification animation does not stop when dismissed

Reported by: Ross Assignee: kzar
Priority: P2 Milestone: Adblock-Plus-3.5-for-Chrome-Opera-Firefox
Module: Platform Keywords:
Cc: sebastian, kzar, greiner, Ross Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by kzar)


Adblock Plus (also 3.3.2)
Chrome 70 / Chrome 49 / Windows 10

How to reproduce

  1. Trigger a critical notification, for example by appending this snippet to the end of adblockpluschrome/lib/notificationHelper.js and browsing to
  id: new Date() | 0,
  type: "critical",
  title: "Notification title",
  message: "<a>Open contribution page</a>",
  links: ["contribute"],
  urlFilters: ["||^$document"]

Observed behaviour

The critical notification is displayed. However if the notification is dismissed via the desktop notification (instead of via the popup), the ! icon animation continues to play.

Expected behaviour

The ! animation to be stopped when the notification is dismissed.

Hints for testers

This change tweaks the logic which handles what happens when the desktop notification is clicked (either on a button, or elsewhere) or closed. Therefore, it's important to test that things work correctly there, for various different types of notifications, on different supported browsers, and on different operating systems which handle notifications differently.

Attachments (0)

Change History (5)

comment:1 Changed on 10/29/2018 at 12:25:56 PM by Ross

  • Summary changed from Critical notification to Critical notification animation does not stop when dismissed

comment:2 Changed on 10/29/2018 at 02:11:45 PM by kzar

  • Component changed from Unknown to Platform
  • Description modified (diff)
  • Priority changed from Unknown to P2
  • Ready set

I can also reproduce this with Chrome 70 on Linux. Investigating now, but I don't think this should block the upcoming release.

comment:3 Changed on 10/29/2018 at 04:44:15 PM by kzar

  • Owner set to kzar
  • Review URL(s) modified (diff)

comment:4 Changed on 10/29/2018 at 04:44:32 PM by kzar

  • Status changed from new to reviewing

comment:5 Changed on 10/31/2018 at 06:23:30 PM by kzar

  • Cc Ross added
  • Description modified (diff)
  • Milestone set to Adblock-Plus-for-Chrome-Opera-Firefox-next
  • Resolution set to fixed
  • Status changed from reviewing to closed

A commit (incorrectly!) referencing this issue has landed:
Issue 7707 - Stop icon animation after notification is clicked

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from kzar.
Note: See TracTickets for help on using tickets.