Opened 5 months ago

Closed 3 months ago

Last modified 3 months ago

#6077 closed change (fixed)

Create script to modify the list of search engines dynamically

Reported by: jwangenheim Assignee: jwangenheim
Priority: P2 Milestone: Adblock-Browser-for-Android-next
Module: Adblock-Browser-for-Android Keywords:
Cc: Blocked By: #6076
Blocking: Platform: Adblock Browser for Android
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

https://codereview.adblockplus.org/29633555/
https://codereview.adblockplus.org/29633713/

Description (last modified by diegocarloslima)

Background

The way how search engines are selected and sorted changed in Firefox 55. Due to this change we had to remove the old logic of ordering search engines for the ABB merge with Fennec 56. On top of that, we need a script to modify list.json (which contains the logic for ordering search engines for all locales) to sort the engines with our custom ordering.

What to change

Write a script to sort the list of search engines with our custom ordering. Use the order from transform_locales.py and add ecosia.xml to adblockbrowser/file/abb_fennec_56_0_merge/mobile/locales/searchplugins
Information about the correct sorting can be also found here: https://issues.adblockplus.org/ticket/5518

Change History (9)

comment:1 Changed 5 months ago by diegocarloslima

  • Description modified (diff)

comment:2 Changed 5 months ago by diegocarloslima

  • Platform changed from Unknown / Cross platform to Adblock Browser for Android

comment:3 Changed 5 months ago by jwangenheim

  • Blocking 6076 removed

comment:4 Changed 5 months ago by jwangenheim

  • Blocked By 6076 added

comment:5 Changed 4 months ago by diegocarloslima

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

comment:6 Changed 4 months ago by diegocarloslima

  • Milestone set to Adblock-Browser-for-Android-next

comment:7 Changed 3 months ago by abpbot

comment:8 Changed 3 months ago by jwangenheim

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:9 Changed 3 months ago by abpbot

Note: See TracTickets for help on using tickets.