Closed
Bug 555694
Opened 15 years ago
Closed 5 years ago
autocomplete should not suggest switching to the current tab
Categories
(Firefox :: Address Bar, defect, P3)
Firefox
Address Bar
Tracking
()
People
(Reporter: elmar.ludwig, Assigned: mak)
References
(Blocks 2 open bugs)
Details
(Whiteboard: [switch-to-tab][fxsearch])
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.3a4pre) Gecko/20100329 Minefield/3.7a4pre
Bug 480350 made it so that autocomplete suggests switching to a tab whose URL matches the one currently typed in the location bar. While this may be useful, it will also suggest switching to the current tab, which is kind of pointless, I think.
1. Visit a common URL, like https://bugzilla.mozilla.org/
2. Type "bugzilla" into the location bar
3. Firefox suggests to switch to the current tab
Displaying the text "Current Tab" instead of "Switch to Tab" would be more informative to the user.
Updated•14 years ago
|
Summary: autocomplete suggests switching to the current tab → autocomplete should not suggest switching to the current tab
Comment 6•14 years ago
|
||
case which is useless and annoying to me, might also confuse normal users:
1. open new empty tab
2. start typing "about" (or less, of course) in location bar
-> always suggests to switch to current tab, which is about:blank
Comment 7•14 years ago
|
||
After all this time, I still find this annoying, doubly so on a new tab. Can we block on this?
blocking2.0: --- → ?
Comment 8•14 years ago
|
||
I don't think this blocks. Mildly curious behavior that doesn't come close to causing real problems does not a release hold back. blocking-.
blocking2.0: ? → -
This bug makes it impossible to remove a page from the history quickly. Very annoying.
Assignee | ||
Comment 10•14 years ago
|
||
in bug 582703 we are moving the switch to tab APIs to the locationbar implementation. At this point could be easy to add a API to register the current topmost tab, so that locationbar can discard it.
Depends on: 582703
Comment 11•14 years ago
|
||
I encountered this ridiculous bug today.
I was at domain.com/this_page.htm and knew of other pages I'd been to on that domain so I deleted this_page.htm and Firefox's top suggestion to me was to switch to...my current tab.
Not only does this waste my time and effort, but it does not show the URL and so one may not recognize that this is their current tab and try to switch to it. Some users may be quite confused "that nothing happened".
Changing the text to "Current Tab" is informational like telling someone the sky's blue. Technically accurate, but obvious and unhelpful.
Comment 13•14 years ago
|
||
Would it be hard to fix? It seems like such a small change Because to be honest its constantly annoying. I very often have to step one more than expected. Plus its totally useless. And the bug is approaching its first birthday...
Comment 14•14 years ago
|
||
It also prevents from deleting the url from history as I mentioned in Bug 636992
So fixing this bug will actually fix two bugs. Kill two birds with one stone.
Comment 16•14 years ago
|
||
Marco, can you give slightly more detail about what this needs/where this needs to be fixed? I don't have much experience with the larger Firefox codebase, but this seems like a good first bug (I could use a mentor, though).
Assignee | ||
Comment 17•14 years ago
|
||
(In reply to comment #16)
> Marco, can you give slightly more detail about what this needs/where this
> needs to be fixed?
I've sent you a mail with some information.
Assignee | ||
Comment 18•14 years ago
|
||
Fwiw, this is not a good-first-bug, indeed it's pretty tricky to figure out a strategy, but you're free to try or others could suggest one.
In the mail I've suggested better bugs you could have fun fixing, thus why I didn't directly post here the content of the mail, it would have been off-topic.
Comment 19•14 years ago
|
||
Yes, thank you! Apparently the salient problem is figuring out a strategy for having the autocomplete backend know what the current tab is. It seems to me there would be two possible ways of implementing that: (a) either the browser could call into the backend every time the active tab is switched, or (b) the backend should have a way of querying the browser what the active tab's location is. Supposedly it's implemented the way it is because we don't want the autocompleter backend know about the browser? In which case (b) is undesirable, leaving (a). So I suppose that, similar to registerOpenPage() there could be something like registerActivePage()?
Assignee | ||
Comment 20•14 years ago
|
||
(In reply to comment #19)
> (a) either the
> browser could call into the backend every time the active tab is switched
this could be tricky with multiple windows.
> or (b) the backend should have a way of querying the browser what the active
> tab's location is
maybe, but it's usually wrong to have toolkit query browser for something. toolkit components can be used in different applications than browsers.
I was thinking of the possibility to filter this out somehow in the locationbar popup (e.g. present it as a common entry even if it is returned as a special entry), but again that popup is driven by toolkit autocomplete.xml
Comment 21•14 years ago
|
||
(In reply to comment #20)
> this could be tricky with multiple windows.
Ah. So is there some kind of window identifier we can send along?
Comment 22•14 years ago
|
||
(In reply to comment #20)
> this could be tricky with multiple windows.
Not really. You can have code run when you gain/lose focus.
Comment 23•13 years ago
|
||
Build ID: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:8.0a1) Gecko/20110815 Firefox/8.0a1
Is there an update to this bug? The issue (with the steps in the Description) is still reproducible on all the channels including the latest Nightly.
Comment 24•13 years ago
|
||
Why so serious ... bug this is? I read the problems above and I still think it should not be hard to fix (for people who know the code). Just decide on one thing and fix please! I infer from the above discussion that suggestion (a) is probably not that dangerous.
Comment 27•12 years ago
|
||
AFAICS, the current behaviour is:
- when entering an (in another tab/window already opened URL) you are suggested the swtich
- when the "swtich to" is selected, but Alt+Enter is used,.. a new tab is opened (as requested by this bug).
I would suggest that the above is just switched:
I.e. default = open new, Alt+Enter = switch
This sounds even more reasonable IMHO, because when a user would want to reuse the already existing tab/window,.. he could also just click/Alt+Tab/etc to it, probably with even not more keystrokes or clicks needed.
Which is why as of now, entering an already opened address in the bar, rather duplicates normal window/tab switching functionally, for which it is not meant to.
So the "switch to" function should better be the non-standard use-case i.e. the one with a modifier like Alt.
Assignee | ||
Updated•8 years ago
|
Priority: -- → P3
Whiteboard: [switch-to-tab] → [switch-to-tab][fxsearch]
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Blocks: qb-results-papercuts
Assignee | ||
Updated•5 years ago
|
Points: --- → 2
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee: nobody → mak
Status: NEW → ASSIGNED
Iteration: --- → 75.2 - Feb 24 - Mar 8
Assignee | ||
Comment 32•5 years ago
|
||
Comment 33•5 years ago
|
||
Pushed by mak77@bonardo.net:
https://hg.mozilla.org/integration/autoland/rev/559ec6828d6d
Address bar should not suggest switching to the current tab. r=harry
Comment 34•5 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox75:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 75
You need to log in
before you can comment on or make changes to this bug.
Description
•