Opened on 08/25/2017 at 09:46:10 AM

Closed on 09/26/2017 at 08:21:02 PM

#5574 closed change (fixed)

Update to current jQuery and jQuery UI versions

Reported by: trev Assignee: trev
Priority: P1 Milestone: Adblock-Plus-3.0-for-Firefox
Module: User-Interface Keywords:
Cc: sebastian, greiner Blocked By:
Blocking: Platform: Firefox
Ready: yes Confidential: no
Tester: Ross Verified working: yes
Review URL(s):

https://codereview.adblockplus.org/29527615/

Description (last modified by sebastian)

Background

Mozilla requires at least jQuery 2.0 for extensions, our jQuery 1.7.1 is considered unsafe. However, when updating jQuery, jQuery UI has to be updated as well, since our version (jQuery UI 1.8) is incompatible with jQuery 1.9 and newer versions.

What to change

Update to current jQuery and jQuery UI versions, make the necessary adjustments to our code.

Hints for testers

This change effects the (old) options page and the "Block element" dialog. Some visual minor changes are to be expected, as long as they don't impair the usability. Also make sure that all UI functionality is still working as before, and that no JavaScript errors are logged in the developer tools.

Attachments (0)

Change History (9)

comment:1 Changed on 08/25/2017 at 09:49:08 AM by trev

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:2 Changed on 08/25/2017 at 10:58:25 AM by trev

  • Component changed from Platform to User-Interface

comment:3 Changed on 08/25/2017 at 11:22:29 AM by greiner

  • Cc greiner added
  • Platform changed from Unknown / Cross platform to Firefox
  • Priority changed from Unknown to P1
  • Ready set

comment:4 Changed on 08/25/2017 at 11:37:39 AM by abpbot

A commit referencing this issue has landed:
Issue 5574 - Update to current jQuery and jQuery UI versions

comment:5 Changed on 08/25/2017 at 11:38:57 AM by trev

  • Milestone set to Adblock-Plus-for-Chrome-Opera-next
  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:6 Changed on 08/25/2017 at 01:19:23 PM by sebastian

  • Description modified (diff)

comment:7 Changed on 09/11/2017 at 01:18:43 PM by Ross

  • Tester changed from Unknown to Ross
  • Verified working set

Done. UI still works as expected and no errors appear to occur.

ABP 1.13.13.1838
Chrome 49 / 60 / Windows 7
Opera 36 / 46 / Windows 7

comment:8 Changed on 09/26/2017 at 02:51:06 AM by sebastian

  • Milestone changed from Adblock-Plus-1.13.4-for-Chrome-Opera to Adblock-Plus-for-Chrome-Opera-next
  • Resolution fixed deleted
  • Status changed from closed to reopened

This change has been backed out, for now, due to #5748.

comment:9 Changed on 09/26/2017 at 08:21:02 PM by sebastian

  • Resolution set to fixed
  • Status changed from reopened to closed

This change was only backed out in master prior to the Adblock plus 1.13.4 release. It was never backed out in next wich now became the new master. Therefore closing again.

Add Comment

Modify Ticket

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