Opened 4 years ago

Closed 14 months ago

#4856 closed change (duplicate)

Reimplement getDocLink assignment in adblockplusUI

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

Description (last modified by greiner)


We do use getDocLink function to assign redirectable, localized links. Usage of that function, both in new-options.js and firstRun.js is messy and inefficient due to unnecessary repetition.

What to change

Use the data-doclink attribute to assign links that use to anchor tags on the page (similar to data-i18n).

Note that as soon as the messaging API supports it, we could make use of the initialization code to aggregate all "app.get" messages into a single one to avoid unnecessary message passing.

Change History (3)

comment:1 Changed 4 years ago by saroyanm

  • Description modified (diff)

I created this as a part of current discussion:

  • Are changes fine with you @Thomas ?
  • I didn't make this blocker for #2706, let me know if you think it should be.
Last edited 4 years ago by saroyanm (previous) (diff)

comment:2 Changed 4 years ago by greiner

  • Description modified (diff)
  • Priority changed from Unknown to P3
  • Ready set

I'm fine with the proposed changes but I'd suggest tackling the efficiency issue separately. The reason for that being that multiple input and output values were initially part of the messaging protocol but got rejected. Therefore we first need to agree on an appropriate way on how we can minimize the communication overhead between background page and other pages - requesting multiple values at once is only one possible approach.

comment:3 Changed 14 months ago by greiner

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

Closing this ticket as duplicate of ui#421.

Note: See TracTickets for help on using tickets.