Opened on 01/26/2018 at 02:39:59 PM

Closed on 09/25/2018 at 11:56:50 AM

Last modified on 07/10/2019 at 12:14:56 PM

#6323 closed change (fixed)

Move composer.html related files to adblockplusui

Reported by: greiner Assignee: greiner
Priority: P3 Milestone:
Module: User-Interface Keywords:
Cc: kzar, sebastian 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)


The filter composer dialog belongs to the User Interface module (albeit, it's still listed as "block.html") but the code for it is still located in adblockpluschrome. Therefore we want to move all its code to adblockplusui to be able to offer a more streamlined interface with the Web Extension team.

See also ui#203.

What to change

  • Move the following files from adblockpluschrome to adblockplusui:
    • composer.html
    • composer.js
  • Replace images in composer.html with skin/abp-logo.svg
  • Move styles in composer.html to new file at skin/composer.css

Notes for dependency update

  • Remove the following files from adblockpluschrome:
    • icons/detailed/abp-32.png.
    • composer.html
    • composer.js
  • Add mappings for the following files:
    • skin/composer.css
    • composer.html
    • composer.js

Attachments (0)

Change History (9)

comment:1 Changed on 09/21/2018 at 02:10:04 PM by greiner

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

Actually, we now have various icons in adblockplusui that we could use instead so there should be no need anymore to move them over from adblockpluschrome.

comment:2 Changed on 09/21/2018 at 02:17:31 PM by greiner

  • Description modified (diff)

Added link to GitLab issue.

comment:3 Changed on 09/21/2018 at 02:50:06 PM by greiner

  • Description modified (diff)

comment:4 Changed on 09/24/2018 at 12:06:14 PM by greiner

  • Owner set to greiner

comment:5 Changed on 09/24/2018 at 12:19:17 PM by greiner

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

comment:6 Changed on 09/24/2018 at 01:46:41 PM by greiner

  • Description modified (diff)

Updated ticket description to reflect decision that was made in review to drop jQuery UI.

comment:8 Changed on 05/20/2019 at 10:58:27 AM by abpbot

A commit referencing this issue has landed:
Issue 6323 - Moved composer from adblockpluschrome

comment:9 Changed on 07/10/2019 at 12:14:56 PM by ukacar

  • Verified working set

Add Comment

Modify Ticket

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