Closed
Bug 924461
Opened 11 years ago
Closed 4 years ago
customize search engines should allow the user to specify the order of engines
Categories
(Firefox for Android Graveyard :: General, defect, P5)
Tracking
(fennec+)
RESOLVED
INCOMPLETE
Tracking | Status | |
---|---|---|
fennec | + | --- |
People
(Reporter: blassey, Assigned: arcturus, Mentored)
References
Details
(Whiteboard: [lang=java])
No description provided.
Comment 2•11 years ago
|
||
This isn't final art by any means, but one simple pattern we could apply is to add a 'grippy' graphic to the right of the list items, to show that items can be dragged up and down the list.
http://cl.ly/image/113g1F410P1u
The one thing to consider here would be how to handle the default search provider at the top of the list. A couple of ways to handle it could be
1. Items that are dragged above it become the default automatically
or
2. Items cannot be dragged above the default provider
Flags: needinfo?(ibarlow)
Reporter | ||
Comment 3•11 years ago
|
||
In the interest of giving users as much control as possible, I prefer option 1.
Comment 4•11 years ago
|
||
Works for me
Updated•11 years ago
|
tracking-fennec: ? → +
Comment 6•10 years ago
|
||
As a first step, we should just add the same "Change order" context menu item that we use for the home panel settings.
As we've seen in bug 974983, making preference items draggable is hard.
Mentor: liuche
Whiteboard: [lang=java]
Assignee | ||
Comment 7•10 years ago
|
||
Would like to work on this with some help from the mentor :)
Comment 8•10 years ago
|
||
(In reply to Francisco Jordano [:arcturus] [:francisco] from comment #7)
> Would like to work on this with some help from the mentor :)
Hi Francisco! Chenxia is on vacation, so I can help you out.
You should start by taking a look at the code we use to create the "Change order" context menu for the home panel settings. Here's where you can find some of that logic:
http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/preferences/PanelsPreference.java?force=1#167
You'll then want to add similar logic to the search preferences, likely somewhere in here:
http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/preferences/SearchEnginePreference.java?force=1#115
In order to actually change the order of the search engine, you'll need to send a message to JS, since we use the toolkit search engine manager, which is written in JS. You can follow a patten similar to what we use to change the default search engine:
http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/preferences/SearchPreferenceCategory.java#60
http://mxr.mozilla.org/mozilla-central/source/mobile/android/chrome/content/browser.js#7054
Let me know if you have any questions!
Assignee: nobody → francisco
Assignee | ||
Comment 9•10 years ago
|
||
Great, this weekend I'll take a look to this following the instructions!
Assignee | ||
Comment 10•10 years ago
|
||
(In reply to :Margaret Leibovic from comment #8)
>
> You'll then want to add similar logic to the search preferences, likely
> somewhere in here:
>
> http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/
> preferences/SearchEnginePreference.java?force=1#115
>
Hei Margaret after taking a look to the code, you think is better to kind of repeat the logic from PanelsPreferences into SearchEnginePreference, or do you think that makes sense to try to generalise into CustomListPreference, making it be 'reorderable'?
What are your thoughts?
Flags: needinfo?(margaret.leibovic)
Comment 11•10 years ago
|
||
(In reply to Francisco Jordano [:arcturus] [:francisco] from comment #10)
> (In reply to :Margaret Leibovic from comment #8)
> >
> > You'll then want to add similar logic to the search preferences, likely
> > somewhere in here:
> >
> > http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/
> > preferences/SearchEnginePreference.java?force=1#115
> >
>
> Hei Margaret after taking a look to the code, you think is better to kind of
> repeat the logic from PanelsPreferences into SearchEnginePreference, or do
> you think that makes sense to try to generalise into CustomListPreference,
> making it be 'reorderable'?
>
> What are your thoughts?
Sorry for the delayed response! I was the one on vacation last week :)
I think that generalizing the logic here makes sense, good catch.
Chenxia has also done a lot of work in this part of the code, so she's a good person to ask questions if you run into any issues. Feel free to post a WIP patch for feedback!
Flags: needinfo?(margaret.leibovic)
Comment 13•10 years ago
|
||
This doesn't need to track a specific release, so it's a +, but it would be really nice to have.
Updated•10 years ago
|
tracking-fennec: ? → +
Updated•10 years ago
|
Comment 15•6 years ago
|
||
Re-triaging per https://bugzilla.mozilla.org/show_bug.cgi?id=1473195
Needinfo :susheel if you think this bug should be re-triaged.
Comment 16•4 years ago
|
||
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INCOMPLETE
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•