Opened 3 years ago

Closed 23 months ago

Last modified 13 months ago

#6324 closed change (fixed)

Move messageResponder.js to lib directory

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

Description (last modified by greiner)


We want to establish a more organized file structure for the adblockplusui repository. For that we want to clearly separate code that's imported into the background page from logic in UI pages or mock logic. The former should be located in the already existing "lib" directory and the only outlier to that rule is "messageResponder.js" which is why we could easily make this consistent.

See also ui#204.

What to change

  • Move messageResponder.js to lib/ directory
  • Updated reference in background.html

Notes for dependency update

  • Mapping for messageResponder.js needs to be updated to point to adblockplusui/lib/messageResponder.js

Change History (7)

comment:1 Changed 23 months ago by greiner

  • Owner set to greiner

comment:2 Changed 23 months ago by greiner

  • Description modified (diff)

comment:3 Changed 23 months ago by greiner

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

comment:5 Changed 21 months ago by greiner

  • Blocking 6936 added

comment:6 Changed 15 months ago by abpbot

A commit referencing this issue has landed:
Issue 6324 - Moved message responder to lib directory

comment:7 Changed 13 months ago by ukacar

  • Verified working set
Note: See TracTickets for help on using tickets.