Opened 7 weeks ago

Closed 2 weeks ago

#6431 closed change (fixed)

Remove old options page from adblockpluschrome

Reported by: greiner Assignee: sebastian
Priority: P3 Milestone: Adblock-Plus-for-Chrome-Opera-Firefox-next
Module: User-Interface Keywords:
Cc: wspee, saroyanm, agiammarchi, kzar, sebastian Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

https://codereview.adblockplus.org/29738601

Description (last modified by sebastian)

Background

As part of #6403 we have started using the new options page for all platforms that are supported by adblockpluschrome. That means that we can now safely remove any remnants of the old options page.

What to change

In adblockpluschrome:

  • Remove these files:
    • desktop-options.html
    • desktop-options.js
  • Remove these strings (note that the string "filters_subscription_lastDownload_connectionError" is also used for the issue reporter and should therefore not be removed):
    • add_domain_button
    • add_filter_button
    • add_your_own_filters
    • apply_changes
    • disabled_for_these_domains
    • edit_filters_raw
    • enable_only_the
    • filter_lists
    • filter_must_follow_syntax
    • filters_acceptableAds2_label
    • filters_addSubscription_label
    • filters_addSubscriptionAdd_label
    • filters_addSubscriptionOther_label
    • filters_readMore_label
    • filters_subscription_delete_label
    • filters_subscription_enabled_label
    • filters_subscription_lastDownload_checksumMismatch
    • filters_subscription_lastDownload_inProgress
    • filters_subscription_lastDownload_invalidData
    • filters_subscription_lastDownload_invalidURL
    • filters_viewList_label
    • found_a_bug
    • general
    • global_remove_subscription_warning
    • global_subscription_invalid_location
    • last_updated_at
    • last_updated_at_today
    • options
    • out_of_date_lists
    • overlay_shownotifications_label
    • remove_button
    • show_block_element_menu
    • show_devtools_panel
    • subscriptionSelection_location_label
    • subscriptionSelection_subscriptionSelector_label
    • subscriptionSelection_title_label
    • update_now
    • whitelisted_domains
  • Remove "composer.quoteCSS" messaging endpoint from lib/filterComposer.js

Change History (6)

comment:1 Changed 7 weeks ago by kzar

Well it's finally happening after all these years! Cool!

comment:2 Changed 7 weeks ago by greiner

  • Description modified (diff)

comment:3 Changed 3 weeks ago by sebastian

  • Description modified (diff)
  • Owner set to sebastian

It seems the string disabled_for_these_domains was also only used by the old options page. I added it to the list.

comment:4 Changed 3 weeks ago by sebastian

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

comment:5 Changed 2 weeks ago by abpbot

A commit referencing this issue has landed:
Issue 6431 - Removed old options page

comment:6 Changed 2 weeks ago by sebastian

  • Milestone set to Adblock-Plus-for-Chrome-Opera-Firefox-next
  • Resolution set to fixed
  • Status changed from reviewing to closed
Note: See TracTickets for help on using tickets.