Opened on 10/02/2015 at 02:44:49 PM

Closed on 09/05/2019 at 05:24:57 PM

#3154 closed change (duplicate)

Extend handling of app.open message in adblockplusui

Reported by: greiner Assignee:
Priority: P3 Milestone:
Module: User-Interface Keywords:
Cc: Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by greiner)

Background

The message responder already contains an "app.open" message for opening pages such as the options page so it'd make sense to get rid of handling of "add-subscription" and extend "app.open" instead. Furthermore the current options page introduced the "focus-section" message as part of #3024 which could also be added the same way.

What to change

In messageResponder.js:

  • Remove the handling of "add-subscription"
  • Let "app.open" accept the additional message properties "focus" and "payload" where "payload" is optional data that should be forwarded to the specified page
  • Currently, the following values are in use:
    • "add-subscription" with the payload containing a "url" and "title" property describing the subscription that should be added
    • "configure-notifications"

Notes for dependency update

This change removes the handling of the "add-subscription" and "focus-section" messages with "app.open". Therefore any occurence of those messages needs to be adapted accordingly.
Furthermore the optional callback parameter for ext.showOptions() can be removed since this was the only way it was used so far.

Attachments (0)

Change History (3)

comment:1 Changed on 10/13/2015 at 12:09:54 PM by greiner

  • Priority changed from Unknown to P3

comment:2 Changed on 01/04/2016 at 04:02:38 PM by greiner

  • Description modified (diff)

comment:3 Changed on 09/05/2019 at 05:24:57 PM by greiner

  • Resolution set to duplicate
  • Status changed from new to closed

Closing this ticket in favor of webext#76.

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 (none).
 
Note: See TracTickets for help on using tickets.