Consider removing in-content search box code
Categories
(Firefox :: Search, task, P3)
Tracking
()
People
(Reporter: bugzilla, Unassigned)
References
(Blocks 4 open bugs)
Details
(Whiteboard: [proton-cleanups])
As of bug 1699834, handoff is enabled on all versions of Firefox, replacing the in-content search on about:home and about:privatebrowsing. We might want to consider removing in-content search if we have no plans to use it. Bug 1699834 also modified some tests so that they continue to test in-content search. These tests could be removed as well.
Comment 1•4 years ago
|
||
Clarifying title to indicate this is the just the code, not the box on the new tab page.
We should totally do this once this once it is released (89). Whilst I remember, there is probably also some tidy up that can be done around the hand-off code - I think it doesn't need all the messages that are passed (it inherited some from the in-content search box).
Updated•4 years ago
|
Updated•4 years ago
|
| Reporter | ||
Comment 2•4 years ago
|
||
I'm making this depend on bug 1713827. That bug will need the "Search the Web" string that might otherwise be removed in this bug.
Updated•4 years ago
|
Very strongly opposed.
I am very of the opinion that urls and search terms should be strictly separated and handled via separate widgets. Attempting to handle them via the same widget just opens up whole classes of privacy leaks and phishing vectors that don't need to exist. It also promotes confusion between urls and search terms among unsophisticated users. Accordingly, the "awesome bar" should not even exist in the first place. If you feel you must absolutely have an "Awesome bar," then handoffToAwesomebar should be under user control and should default to false.
(Yes, I know I'm very late to this party.)
| Comment hidden (advocacy) |
| Comment hidden (off-topic) |
| Comment hidden (advocacy, metoo) |
Comment 7•4 years ago
|
||
Bug 1727319 was cleaning up the styles on new tab, but it was made a bit more complicated needing to consider that this in-content search behavior is still available via about:config toggle.
Description
•