Bug 138789
Summary: | [PATCH] URLs opened from outside Firefox open in new windows, regardless of UI setting | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Craig Box <craig> | ||||||
Component: | firefox | Assignee: | Christopher Aillon <caillon> | ||||||
Status: | CLOSED RAWHIDE | QA Contact: | |||||||
Severity: | medium | Docs Contact: | |||||||
Priority: | medium | ||||||||
Version: | rawhide | CC: | cdavis, cefrodrigues, chaghi, chris.ricker, davidf, mattdm, mitr, notting, ondrej, wtogami | ||||||
Target Milestone: | --- | ||||||||
Target Release: | --- | ||||||||
Hardware: | All | ||||||||
OS: | Linux | ||||||||
Whiteboard: | |||||||||
Fixed In Version: | 1.0.2-2 | Doc Type: | Bug Fix | ||||||
Doc Text: | Story Points: | --- | |||||||
Clone Of: | Environment: | ||||||||
Last Closed: | 2005-03-25 18:00:52 UTC | Type: | --- | ||||||
Regression: | --- | Mount Type: | --- | ||||||
Documentation: | --- | CRM: | |||||||
Verified Versions: | Category: | --- | |||||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||
Cloudforms Team: | --- | Target Upstream Version: | |||||||
Embargoed: | |||||||||
Bug Depends On: | |||||||||
Bug Blocks: | 136450, 136452 | ||||||||
Attachments: |
|
Description
Craig Box
2004-11-11 06:07:00 UTC
This wont be easy due to the inflexibility of the gconf-based system used to launch the preferred applications. I noticed this too. Why must it be "openurl($URL,new-window)" and not "openurl($URL)"? I'm not seeing the problem with doing this... That is unacceptable to me. I happen to LIKE new windows opening on links from other programs, because it is impossible to open multiple links because it replaces the current content of your browser window. The only way to fix this is to extend the reach of the preference. Anything less is unacceptable. Well, when Firefox has "open links in a new tab in the most recent window", the contents aren't replaced, as the preference says. Besides, "openurl($URL)" does not impose any particular setting, it honors the preference defined in firefox (new window/new tab/most recent window). With "openurl($URL,new-window)", the preference is ignored, which _is_ a bug. *** Bug 141564 has been marked as a duplicate of this bug. *** Ok, I think it would be acceptable to make this change to /usr/bin/firefox ONLY IF our package default preference be changed to "new window" rather than the current window. Then advanced users like yourself can change it to their preferred behavior. caillon, your thoughts? That would be just fine. *** Bug 142775 has been marked as a duplicate of this bug. *** First, sorry for the duplicate 142775. As Firefox is new in FC3, I didn't look for the bug in "Devel" version. Having read all these comments, I don't understand why we have to wait for a "fix" in the package default preference to fix this bug. Firefox has a preference so the user can choose if he/she likes a new window, a new tab or the same window/tab. With the current behaviour, /usr/bin/firefox *overrides* the user preference, which is very annoying and frustrating (nobody is going to look into /usr/bin/firefox, specially a newbie). With the proposed fix in /usr/bin/firefox, *all of us* (advanced users or not) can have firefox do whatever they choose in the preferences. We have to cater to both advanced and newbie users. The default behavior of "open in the same window" is absolutely terrible. For this reason I suggest to the package maintainer to change that default option while also changing /usr/bin/firefox. If somebody submits a simple patch to change that default behavior, maybe this can happen sooner. For reference, the config option is "browser.link.open_external" and the options are accessible under Edit->Preferences->Advanced->Tabbed Browsing: "Open links from other applications in:" (2) a new window (3) a new tab (1) the most recent tab/window (default) The number given is the value of the preference for that option. Created attachment 108881 [details]
Patch to firefox.js to set default pref for opening windows
Added a patch to mozilla/browser/app/profile/firefox.js to set the default
preference to open links from external applications in a new window
Created attachment 108882 [details]
patch to firefox.sh.in to not pass "new-window" to firefox
this just removes the ",new-window" parameter that is passed to firefox when
opening a new URL
Attached the neccessary patches. The prefs patch needs to be applied to mozilla/browser/app/profile/firefox.js inside the tarball so it would need a change to the spec file, or it could be added to another patch. The other patch is to firefox.sh.in which is included as a source file in the RPM. These are untested but should work by principle, hopefully someone can check them out and apply them. caillon: his patches should work, but we are forbidden from patching firefox.js caillon: use firefox-redhat-default-preferences.js caillon: firefox.js is reserved for upstream Chris said he'll fold this into FC4 soon along with many other patches. It will be tested there for a short while, then backported into a FC3 firefox update. *** Bug 144981 has been marked as a duplicate of this bug. *** caillon said this is going in before FC4test2. In tomorrow's rawhide. Thanks for the patch, David, and sorry for the delay here. This patch just saved my sanity. :) Glad this has gone in, and to be saving people's sanity ... Could someone report here if/when this goes into a FC3 update? Could someone please report if there are plans on an update to FC3 or if we will have to go to FC4 to get this fixed? |