Closed
Bug 246274
Opened 20 years ago
Closed 20 years ago
Firefox advanced browsing option "open links in the background" misleading
Categories
(Firefox :: Tabbed Browser, defect)
Firefox
Tabbed Browser
Tracking
()
VERIFIED
FIXED
People
(Reporter: tmeader, Assigned: tmeader)
Details
Attachments
(1 file)
3.29 KB,
patch
|
mconnor
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040608 Firefox/0.8.0+ Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040608 Firefox/0.8.0+ This is an analog to the Mozilla Suite bug I've been working on, #111842 To sum up, the preference under General Browser in Firefox's Advanced options labeled "Load links in background" is EXTREMELY confusing. It was slightly better (though still misleading) in the Mozilla Suite since it was under Tabbed Browsing preferences. Under General Browser in Firefox, it makes it seem as if the browser is "precaching" pages that are linked to on the current page. In no way does it specify that it has something to do with the manner in which new Tabs (and Bookmarks) are opened. I've created a patch that his been approved (but waiting for sr, neither have gotten back to me for the past five days though) which basically gives a new text for the option, as well as automatically reverse the meaning of the pref in the backend. Shouldn't break anything existing that depends on this pref. Reproducible: Always Steps to Reproduce: I would normally set this to "Trivial" but this is a VERY confusing option. I believe it's rather important to clear up for 1.0 exactly what's happening when people enable or disable the preference. I'll try to get the patch submitted tonight.
Comment 1•20 years ago
|
||
this is a dupe, iirc
Assignee | ||
Comment 2•20 years ago
|
||
Mike, there is one for Mozilla Suite that I'm also working on, but a search for the pref text in Bugzilla reports no duplicates that I can find. Please let me know if I'm wrong. Changing summary text to better reflect it.
Summary: browser prefs text "load links in the background" misleading → Firefox advanced browsing text "load links in the background" misleading
Assignee | ||
Updated•20 years ago
|
Summary: Firefox advanced browsing text "load links in the background" misleading → Firefox advanced browsing option "load links in the background" misleading
Assignee | ||
Comment 3•20 years ago
|
||
mconnor, I'd written you a little over a week ago about possibly getting this into Firefox as well as the Suite. Here's my first crack at it. This is patched against today's AVIARY branch.
Assignee | ||
Comment 4•20 years ago
|
||
Comment on attachment 150499 [details] [diff] [review] Firefox port of the Mozilla Suite patch Setting to mconnor for review
Attachment #150499 -
Flags: review?(mconnor)
Comment 5•20 years ago
|
||
Bug 215441 is what I'm thinking about, the wording changed when it moved to the advanced panel. I think I like this wording much better, but I'm sleep-deprived now :(
Assignee | ||
Updated•20 years ago
|
Summary: Firefox advanced browsing option "load links in the background" misleading → Firefox advanced browsing option "open links in the background" misleading
Updated•20 years ago
|
Assignee: bugs → tmeader
Status: UNCONFIRMED → NEW
Ever confirmed: true
Updated•20 years ago
|
Status: NEW → ASSIGNED
Comment 6•20 years ago
|
||
Comment on attachment 150499 [details] [diff] [review] Firefox port of the Mozilla Suite patch This is definitely better wording. rlk, please note the wording change for Help
Attachment #150499 -
Flags: review?(mconnor) → review+
Comment 7•20 years ago
|
||
Thanks mconnor! Fix checked in on FIREFOX_HELP_1_0_BRANCH on mozdev.
Assignee | ||
Comment 8•20 years ago
|
||
I appreciate assigning it to me, if nothing else than to get it off Ben's plate... but I unfortunately don't have CVS checkin ability. If everyone thinks this patch is fine, please mconnor, when you get the chance, go ahead and check this in. Thanks in advance.
Comment 9•20 years ago
|
||
Fix checked in on the trunk. I'll check it into aviary soon.
Comment 10•20 years ago
|
||
Fix checked in on the aviary branch.
Comment 11•20 years ago
|
||
I'd've gone for "Automatically switch to newly opened tabs", but that's fine too.
Assignee | ||
Comment 12•20 years ago
|
||
Unfortunately that doesn't specify that it only applies to tabs opened from links. If you click new tab, that is ALWAYS selected, there's no option for that. Anyhow, gonna download today's and make sure it's working alright. Will close after that if so.
Comment 13•20 years ago
|
||
I hate to be a nag over such a trivial issue, but is "Focus" a better choice than "Select"? It's shorter, and it doesn't seem as awkward to me as "Select" does. To me "Select" implies a conscious choice among alternatives (but there's no conscious choice at the moment the tab's opened), while "Focus" simply says the tab will be displayed. If this doesn't make sense, ignore or reply as you wish - this is trivial enough that it's not worth the time to explain why my suggestion doesn't make as much sense as the original suggestion.
Comment 14•20 years ago
|
||
Jeff, "focus" is a term that isn't a) 100% specific, since we're choosing a different tab, not just focusing it, and b) isn't necessarily clear (the concept of "focus" is more abstract than selection to users). Tell your grandma to focus the tab and she won't know what you mean, 9 times out of 10. Fixed branch and trunk, resolving.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Comment 15•20 years ago
|
||
Verified, Firefox 0.9. However, as a Firefox localizer I am strongly opposed to such changes between 0.9 RC and 0.9 final. :-(
Status: RESOLVED → VERIFIED
Comment 16•20 years ago
|
||
(In reply to comment #14) > Jeff, "focus" is a term that isn't a) 100% specific, since we're choosing a > different tab, not just focusing it, and b) isn't necessarily clear (the concept > of "focus" is more abstract than selection to users). Tell your grandma to > focus the tab and she won't know what you mean, 9 times out of 10. "Switch to"?
You need to log in
before you can comment on or make changes to this bug.
Description
•