Opened 2 years ago

Closed 18 months ago

Last modified 9 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 18 months ago by greiner

  • Owner set to greiner

comment:2 Changed 18 months ago by greiner

  • Description modified (diff)

comment:3 Changed 18 months ago by greiner

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

comment:5 Changed 16 months ago by greiner

  • Blocking 6936 added

comment:6 Changed 11 months ago by abpbot

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

comment:7 Changed 9 months ago by ukacar

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