2009/1/13 Mike Hommey <[email protected]>:
>> I think the proper fix (or workaround, depending on your PoV of the
>> issue) is to use the chrome key since it is an iceweasel regression

>> and, even though the urgency setting is lost, I think the impact is a
>> lot smaller than these windows moving around like crazy and imepeding
>> work.

>> There a way to add system wide this setting by adding it to
>> /etc/iceweasel/pref/iceweasel.js
>>
>> browser.tabs.loadDivertedInBackground = true
>
> The problem is that this pref does more than working around metacity's
> problem, and changes the behaviour for some categories of tab opening.

I know, that's why I said:

>> I think the impact is a
>> lot smaller than these windows moving around like crazy and imepeding
>> work.

... although I wasn't clear that I was aware that all tabs would be
opened in the background.

> Which means instead of getting reports about firefox stealing focus,
> we'll then get reports about firefox opening tabs in the background in
> some cases.
>
> I'm not sure which users will be more annoyed (and more annoying ;) ).

I think that users in my own category will be more annoyed and annoying
since the iceweasel window would move all over the place for this behaviour,
while for the other they would need to do and extra click to reach the new tab.


Maybe the urgency shouldn't be set if the link is opened from an external link?
How hard would it be to disable the urgency stuff for everybody. I know, I know,
that's overkill, too, but I really fear the reviews and impressions from people
getting annoyed at this bug after the upgrade.


Meanwhile I looked at the source and prepared a small NMU and the diff is
attached - although probably useless since you're active and I am not a DD.

-- 
Regards,
EddyP
=============================================
"Imagination is more important than knowledge" A.Einstein
commit 5c267ca5aac640b76833facdae054fa207460cef
Author: Eddy Petrisor <[email protected]>
Date:   Tue Jan 13 20:26:17 2009 +0200

    prepare an NMU for #486570

diff --git a/iceweasel-3.0.5/debian/changelog b/iceweasel-3.0.5/debian/changelog
index fa1a8e0..f4ad9d2 100644
--- a/iceweasel-3.0.5/debian/changelog
+++ b/iceweasel-3.0.5/debian/changelog
@@ -1,3 +1,11 @@
+iceweasel (3.0.5-1.2) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * do not move the iceweasel window to the current desktop when opening
+    a link from another application (Closes: #486570)
+
+ -- Eddy Petrișor <[email protected]>  Tue, 13 Jan 2009 20:24:07 +0200
+
 iceweasel (3.0.5-1) unstable; urgency=low
 
   * New upstream release.
diff --git a/iceweasel-3.0.5/debian/iceweasel.js b/iceweasel-3.0.5/debian/iceweasel.js
index 8424c43..504ce1e 100644
--- a/iceweasel-3.0.5/debian/iceweasel.js
+++ b/iceweasel-3.0.5/debian/iceweasel.js
@@ -17,3 +17,7 @@ pref("intl.locale.matchOS", true);
 
 // Disable default browser checking.
 pref("browser.shell.checkDefaultBrowser", false);
+
+// Do not change desktop when opening a link from another app
+// see #486570
+pref("browser.tabs.loadDivertedInBackground", true);

Reply via email to