Opened on 02/28/2018 at 02:41:34 PM
Closed on 04/04/2018 at 01:57:31 AM
Last modified on 05/10/2018 at 07:24:50 PM
#6431 closed change (fixed)
Remove old options page from adblockpluschrome
Reported by: | greiner | Assignee: | sebastian |
---|---|---|---|
Priority: | P3 | Milestone: | Adblock-Plus-3.1-for-Chrome-Opera-Firefox |
Module: | User-Interface | Keywords: | |
Cc: | wspee, saroyanm, agiammarchi, kzar, sebastian | Blocked By: | |
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | yes | Confidential: | no |
Tester: | Ross | Verified working: | yes |
Review URL(s): |
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
Attachments (0)
Change History (7)
comment:1 Changed on 02/28/2018 at 02:48:56 PM by kzar
comment:3 Changed on 03/31/2018 at 07:30:06 PM 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 on 03/31/2018 at 07:31:12 PM by sebastian
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:5 Changed on 04/04/2018 at 01:56:30 AM by abpbot
A commit referencing this issue has landed:
Issue 6431 - Removed old options page
comment:6 Changed on 04/04/2018 at 01:57:31 AM by sebastian
- Milestone set to Adblock-Plus-for-Chrome-Opera-Firefox-next
- Resolution set to fixed
- Status changed from reviewing to closed
comment:7 Changed on 05/10/2018 at 07:24:50 PM by Ross
- Tester changed from Unknown to Ross
- Verified working set
Done. Hasn't caused any regressions. Have not noticed any missing strings.
ABP 3.0.4.2042
Firefox 59 / 55 / 51 / Windows 10
Chrome 66 / 58 / 49 / Windows 7
Opera 52 / 45 / 36 / Windows 10
Well it's finally happening after all these years! Cool!