Status

()

enhancement
2 years ago
Last year

People

(Reporter: Fischer, Unassigned)

Tracking

(Depends on 8 bugs, Blocks 1 bug, {meta})

Firefox Tracking Flags

(relnote-firefox -)

Details

(Whiteboard: [photon-preference])

Reporter

Description

2 years ago
This is a meta bug to track all bugs that block the shipping of Preferences Search in Photon Preferences.
Reporter

Updated

2 years ago
Whiteboard: [photon-preferences]
Reporter

Updated

2 years ago
Whiteboard: [photon-preferences] → [photon-preference]
Reporter

Updated

2 years ago
Depends on: 1353954
Reporter

Updated

2 years ago
Depends on: 1335905
Reporter

Updated

2 years ago
Depends on: 1349747
Reporter

Updated

2 years ago
Depends on: 1352481
Reporter

Updated

2 years ago
Depends on: 1351867
Reporter

Updated

2 years ago
Depends on: 1357130
Reporter

Updated

2 years ago
Depends on: 1340643
Reporter

Updated

2 years ago
Depends on: 1353862
Reporter

Updated

2 years ago
Depends on: 1356500
Reporter

Comment 1

2 years ago
Thanks for the huge part of the search feature works done by Mike, Jared and the MSU students.
Since Photon Preferences plans to ship the search feature, we will join the development as well.
From now on the Photon Preferences will 
  - take on the bug 1357130 and 
  - fix follow-up bugs if reported and
  - continue the works of the bug 1349747, bug 1352481, and bug 1351867 if needed.

The below is the detail email from Mike about the current status and how to collaborate on the search works:
> On Tue, Apr 18, 2017 at 1:03 AM, Mike Conley <mconley@mozilla.com> wrote:
> Hey Fischer,
> 
> Thanks for the email.
> 
> The MSU students who have been working on this project are starting to
> wrap up now.
> 
> There's still some work in-flight for the search feature, but I suspect
> given the remaining time (the students are "pencil's down" on April
> 27th) it's unlikely to land in time, and we'll need to do some kind of
> hand-off.
> 
> Here are the big remaining bits:
> 
> * Bug 1349747 - The search box in about:preferences should show search
> suggestions
> ** MSU student manotejmeka has a proof-of-concept patch posted, but (at
> our suggestion), he's using hard-coded strings for things like "stop
> words". This patch might need some re-architecture, and certainly some
> clean-up.
> 
> * Bug 1352481 - Search should find matched strings within popups and
> subdialogs
> ** Similar to the above - manotejmeka has a proof-of-concept patch
> posted, but the code is definitely not in a state that is close to landing.
> 
> * Bug 1351867 - Show search matches in a tooltip pointing at buttons if
> the matched text will be revealed when clicking on the button
> ** Similar to the above. I'm not sure how useful manotejmeka's
> proof-of-concept patch will be as a starting point though, as he appears
> to be doing multiple synchronous reflows when adding the tooltips.
> 
> * Bug 1357130 - Preferences search highlight should match the spec
> ** This is one that hasn't been started yet (I just filed it), and could
> be started immediately by your team.
> 
> There are some smaller bugs blocking bug 1353954 that will likely be
> fixed before the MSU students are done (example: bug 1340643 and bug
> 1353862).
> 
> I should also point out that QA hasn't started to test search, as far as
> I know, so there might be some remaining functional issues that will
> need to be worked out as well that they will find.
> 
> My advice is to start immediately on bug 1357130, and start researching
> better approaches to bug 1349747, bug 1352481, and bug 1351867. We'll
> alert you when the MSU students have completed their term (next week),
> and we can then re-assign the bugs.
Reporter

Updated

2 years ago
Depends on: 1357352
Summary: [meta] Preferences Search → [meta] Preferences search
Target Milestone: --- → Firefox 56

Updated

2 years ago
Depends on: 1359306
Flags: qe-verify+
QA Contact: hani.yacoub
Depends on: 1360491
No longer depends on: 1349747
No longer depends on: 1351867
No longer depends on: 1352481

Updated

2 years ago
Depends on: 1360524

Updated

2 years ago
Depends on: 1360532
Depends on: 1351867
No longer depends on: 1360532
No longer depends on: 1359306
Depends on: 1363721
Depends on: 1363726
Depends on: 1363728

Updated

2 years ago
Depends on: 1363762

Updated

2 years ago
Depends on: 1365259
Depends on: 1365503
Depends on: 1364197
Depends on: 1365847

Updated

2 years ago
Depends on: 1365910
Depends on: 1366214

Updated

2 years ago
Depends on: 1366252

Updated

2 years ago
Depends on: 1366754

Updated

2 years ago
No longer depends on: 1366754
Depends on: 1352481
No longer depends on: 1366148
Keywords: meta

Updated

2 years ago
Depends on: 1366992

Updated

2 years ago
Depends on: 1367022

Updated

2 years ago
Depends on: 1367035

Updated

2 years ago
Depends on: 1367045

Updated

2 years ago
Depends on: 1367349

Updated

2 years ago
Depends on: 1368947

Updated

2 years ago
Depends on: 1368957
Depends on: 1369305

Updated

2 years ago
Depends on: 1369640

Updated

2 years ago
Depends on: 1369704

Updated

2 years ago
Depends on: 1372583

Updated

2 years ago
Depends on: 1373655

Updated

2 years ago
Depends on: 1374279
Depends on: 1374280

Updated

2 years ago
Depends on: 1374588
Depends on: 1375888

Updated

2 years ago
Depends on: 1377845

Updated

2 years ago
Depends on: 1378359

Updated

2 years ago
Depends on: 1378364
Depends on: 1377163
Depends on: 1383456
Depends on: 1383566

Updated

2 years ago
Depends on: 1384566
Depends on: 1384600

Updated

2 years ago
Depends on: 1385218
Depends on: 1385243

Comment 2

2 years ago
Release Note Request (optional, but appreciated)
[Why is this notable]:New feature 
[Affects Firefox for Android]:No
[Suggested wording]: Added search function under Preferences/ Options
[Links (documentation, blog post, etc)]:N/A

[Tracking Requested - why for this release]:
relnote-firefox: --- → ?

Comment 3

2 years ago
Bug 1296904 seems like a duplicate

Comment 4

2 years ago
Bug 1373447 probably should be added as a dependency ?

Updated

2 years ago
Duplicate of this bug: 1296904

Updated

2 years ago
Depends on: 1400420
The release note was already added in bug 1353954.

Updated

Last year
Depends on: 1446657
You need to log in before you can comment on or make changes to this bug.