Open Bug 1704156 Opened 3 years ago Updated 1 year ago

Consider removing in-content search box code

Categories

(Firefox :: Search, task, P3)

task
Points:
3

Tracking

()

People

(Reporter: bugzilla, Unassigned)

References

(Depends on 1 open bug, 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.

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).

Summary: Consider removing in-content search box → Consider removing in-content search box code
Severity: -- → N/A
Priority: -- → P3
Blocks: 1529979
Blocks: 1483244
Blocks: 1478048
Blocks: 1394387

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.

Depends on: 1713827
Whiteboard: [proton-cleanups]
See Also: → 1713953

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.)

For reasons already mentioned by cwbussard I also don't support the seperate search bar, despite the fact that some users don't seem to understand the difference between a search term and a webpage address and the combination is convenient for them. Combining both into one appears to me like being allowed to write something as "nearest grocery store" on the envelope of a letter if you don't know the address of the store. That being said, I do like some of the address bar intelligence, mainly not having to type the "https://" or "http://" and the autocompletion of the most common top level domains (if one is missing at all).
Concidering the handling of search terms, I really think the feature should still not be turned on by default. - But this is just my opinion on the subject, it might appear outdated to others.

Additional note:
The confusion for unsophisticatet users seems to be caused partly because correctly entered addresses can end up in a search result instead of the existing webpage being loaded. - I noticed that behaviour several times when supporting users. This may happen on mobile devices but also on desktop browsers depending on the specific circumstances - probably because the search results can be prioritized over other entries (appear above them) and the user just hitting enter instead of selecting the correct item. On an Android device of someone I even witnessed a website being not accessible at all just because Google search interfered and the newly created website had not been indexed by Google at that time!

My conclusion:
Using the address bar for search terms should rather be considered as a bug instead of as a feature.

(In reply to cohaller from comment #4)

... I also don't support the seperate search bar,...

Correction: ... I also don't support removing the seperate search bar, ...

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.

See Also: → 1727319
Depends on: 1809591
You need to log in before you can comment on or make changes to this bug.