[Bug 310217] Re: Wrong search engine in right-click menu only in "wide screen" mode
** Changed in: firefox Status: Confirmed => Fix Released -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode To manage notifications about this bug go to: https://bugs.launchpad.net/firefox/+bug/310217/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
*** Bug 790956 has been marked as a duplicate of this bug. *** -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode To manage notifications about this bug go to: https://bugs.launchpad.net/firefox/+bug/310217/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
The code intentionally uses the default engine, it's not a matter of accidentally reading the wrong pref (the prefs are internal to the search service, the users can choose to use either .defaultEngine or .currentEngine, as shown in the patch). -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode To manage notifications about this bug go to: https://bugs.launchpad.net/firefox/+bug/310217/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
…There is actually a preference browser.search.selectedEngine — is it possible that the context menu and Ctrl+K are referring to browser.search.defaultenginename when they should be referring to this instead? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode To manage notifications about this bug go to: https://bugs.launchpad.net/firefox/+bug/310217/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
There's a workaround for this: in about:config, set browser.search.defaultenginename to the name of the selected search engine. This changes the search engine used by the context menu, and the destination of Ctrl+K. You'll have to do this each time you change your selected search engine. A fix for this would be for Firefox to change the value of this key each time a search engine is chosen in the search bar, and not to change this key when hiding the search bar. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode To manage notifications about this bug go to: https://bugs.launchpad.net/firefox/+bug/310217/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
There's two parts to this bug: 1. When in Full screen mode 2. When the Search bar is removed Issue 1 is fixed for me in Nightly (10.0a1). Issue 2 is not. Even though I use DuckDuckGo as my "default" search engine (it's on the top of the list, and it's the one I have selected when I remove the search bar) I am still sent to google.com when I press Ctrl+K. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode To manage notifications about this bug go to: https://bugs.launchpad.net/firefox/+bug/310217/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
This seems to have been fixed in Nightly (9.0a1). -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode To manage notifications about this bug go to: https://bugs.launchpad.net/firefox/+bug/310217/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
Comment on attachment 503735 Change loadSearch as well (and remove references to defaultEngine from comments) nsContextMenu.js drives the engine name that shows up in the context menu, so with the current patch, it'll use the right engine but show the wrong text. See isTextSelection in that file for a code path similar to the one you already changed. if (isElementVisible(BrowserSearch.searchBar)) -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
*** Bug 655106 has been marked as a duplicate of this bug. *** -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
As I alluded to in comment 1, there is a bit of history behind this bug. First, the use of the "default" engine is explained in bug 331105 comment 3 and bug 317107 comment 40. Next, when I confirmed this bug in 2008, the main problem was that in full-screen mode, the toolbar autohides, which the browser misinterprets as "the search box is unavailable", causing it to use the "default" engine. AFAICT, this particular problem with the autohide is no longer present in 4.0, so it was fixed at some point in the intervening years (see comment 7). So the problem that we have here is this: when the search box is not present (removed by user customization), the context menu search uses a "default" search engine that is configurable by the user only via about:config. The possible solutions to this are: 1) Do what this (now slightly morphed bug) is proposing, and that is to just not use the "default" engine and use whatever was the previously-selected engine. 2) Disable the context menu search if the search box is hidden (one of the options brought up in bug 331105 comment 4). 3) Fix bug 331105 and add a UI to set a "default" engine. 4) Fix bug 331105 by way of bug 429513 (the "lite" solution). The original intent of bug 429513 was to serve as a stopgap fix for bug 331105 (since at the time, it was too late in the release cycle to really consider bug 331105). Personally, I would prefer bug 331105, but bug 429513 is an option if a new UI is too much for an edge-caseish scenario like this. Personally, I don't like #2. As for #1 vs. #3 vs. #4, for people who remove the search box, what is the best way for them to communicate to the browser their preferred search engine choice? Explicitly via a dedicated button or option in the search manager UI? Implicitly via selecting their preferred engine prior to removing the search box? Or implicitly by moving their preferred engine to the top of the list in the search manager? Right now, there is no way for them to communicate that to the browser (except via about:config). -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
Comment on attachment 503735 Change loadSearch as well (and remove references to defaultEngine from comments) Pinging Beltzner for ui-r. This patch changes the behavior to use currentEngine instead of defaultEngine. Is this acceptable? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
Comment on attachment 503735 Change loadSearch as well (and remove references to defaultEngine from comments) Gavin says this needs ui-r. But who? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
*** Bug 635136 has been marked as a duplicate of this bug. *** -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
Created attachment 503735 Change loadSearch as well (and remove references to defaultEngine from comments) This patch also changes loadSearch to only use the currentEngine, and it changes the comments for the affected functions to no longer reference defaultEngine. (Are there any weird cases where currentEngine isn't set, but defaultEngine is?) -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
(In reply to comment #14) > Created attachment 503583 [details] > patch > Hrm, all the comments around that change explicitly mention taking the default rather than current if the search bar is removed. And it looks like I missed a few places for changing it to 'currentEngine' for loadSearch() right below the patch cuts off in the file. I can change loadSearch to only consider currentEngine in a revised patch. Why is the behavior changed based on the presence of the search bar in the first place, though? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
Comment on attachment 503583 patch You will have to ask for review/feedback. Gavin should be able to give you the answer. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
Created attachment 503583 patch Would something like this work? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
*** Bug 625466 has been marked as a duplicate of this bug. *** -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
Same also applies for the keyboard shortcut to focus the search bar. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
*** Bug 622790 has been marked as a duplicate of this bug. *** -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
This does not occur anymore with 4.0 beta 7 as packaged by the Debian Mozilla Team (http://mozilla.debian.net/packages/). -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/310217 Title: Wrong search engine in right-click menu only in "wide screen" mode -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217]
*** Bug 607597 has been marked as a duplicate of this bug. *** -- Wrong search engine in right-click menu only in "wide screen" mode https://bugs.launchpad.net/bugs/310217 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217] Re: Wrong search engine in right-click menu only in "wide screen" mode
** Changed in: firefox Importance: Unknown => Medium -- Wrong search engine in right-click menu only in "wide screen" mode https://bugs.launchpad.net/bugs/310217 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217] Re: Wrong search engine in right-click menu only in "wide screen" mode
** Changed in: firefox Status: Unknown => Confirmed -- Wrong search engine in right-click menu only in "wide screen" mode https://bugs.launchpad.net/bugs/310217 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217] Re: Wrong search engine in right-click menu only in "wide screen" mode
I've attached the bug report I found upstream. ** Bug watch added: Mozilla Bugzilla #436265 https://bugzilla.mozilla.org/show_bug.cgi?id=436265 ** Also affects: firefox via https://bugzilla.mozilla.org/show_bug.cgi?id=436265 Importance: Unknown Status: Unknown -- Wrong search engine in right-click menu only in "wide screen" mode https://bugs.launchpad.net/bugs/310217 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217] Re: Wrong search engine in right-click menu only in "wide screen" mode
I can also confirm this behavior, I think there is enough information here for a developer to begin work. Thank you for reporting this. con...@compy686:~$ lsb_release -rd Description:Ubuntu 8.04.1 Release:8.04 con...@compy686:~$ uname -a Linux compy686 2.6.24-20-generic #1 SMP Mon Jul 28 13:49:52 UTC 2008 i686 GNU/Linux con...@compy686:~$ apt-cache policy firefox firefox: Installed: 3.0.5+nobinonly-0ubuntu0.8.04.1 Candidate: 3.0.5+nobinonly-0ubuntu0.8.04.1 Version table: *** 3.0.5+nobinonly-0ubuntu0.8.04.1 0 500 http://ubuntu.media.mit.edu hardy-updates/main Packages 500 http://security.ubuntu.com hardy-security/main Packages 100 /var/lib/dpkg/status 3.0~b5+nobinonly-0ubuntu3 0 500 http://ubuntu.media.mit.edu hardy/main Packages ** Changed in: firefox-3.0 (Ubuntu) Importance: Undecided => Medium Status: Confirmed => Triaged -- Wrong search engine in right-click menu only in "wide screen" mode https://bugs.launchpad.net/bugs/310217 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217] Re: Wrong search engine in right-click menu only in "wide screen" mode
Thank you for taking the time to report this bug and helping to make Ubuntu better, I am seeing this same behavior running Ubuntu 8.10 with Firefox 3.0.5. In normal non-F11 mode (aka non full screen mode), if I select some text and right-click that text the search engine that is selected in the top right shows up in the menu. Google by default, but it changes if I switch that to Yahoo or Wikipedia for example. However, if I switch it Yahoo, and then hit F11 to go into full screen mode, select some text and right-click, it defaults back to Google. ** Changed in: firefox-3.0 (Ubuntu) Status: New => Confirmed -- Wrong search engine in right-click menu only in "wide screen" mode https://bugs.launchpad.net/bugs/310217 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217] Re: Wrong search engine in right-click menu only in "wide screen" mode
** Attachment added: "Dependencies.txt" http://launchpadlibrarian.net/20614782/Dependencies.txt ** Attachment added: "ExtensionSummary.txt" http://launchpadlibrarian.net/20614783/ExtensionSummary.txt ** Attachment added: "profiles.ini.txt" http://launchpadlibrarian.net/20614784/profiles.ini.txt -- Wrong search engine in right-click menu only in "wide screen" mode https://bugs.launchpad.net/bugs/310217 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 310217] [NEW] Wrong search engine in right-click menu only in "wide screen" mode
Public bug reported: Binary package hint: firefox-3.0 I use a french Firefox (3.0.5+nobinonly-0ubuntu0.8.04.1). When I select some text and right-click it, I expect the context menu search item to use the currently selected search engine. But this is what happens: when I select some text, do a right-click and Firefox is in "wide screen" mode (F11), the menu item for searching always uses Google instead of the currently selected engine. If I exit wide screen mode, the right engine is used without doing any further action. Note that if I am in wide screen mode, select some text, move the cursor to the top to make appear the search bar, click inside the search bar, then move the cursor back to the selected text (the search bar stays visible as it has focus) and right-click, the right engine menu is used. This means that the wrong engine is used only in wide screen mode and the search bar is not visible. Thanks in advance. ProblemType: Bug Architecture: i386 Date: Sun Dec 21 13:36:03 2008 DistroRelease: Ubuntu 8.04 NonfreeKernelModules: fglrx wl Package: firefox-3.0 3.0.5+nobinonly-0ubuntu0.8.04.1 PackageArchitecture: i386 ProcEnviron: PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games LANG=fr_FR.UTF-8 SHELL=/bin/bash SourcePackage: firefox-3.0 Uname: Linux 2.6.24-22-generic i686 ** Affects: firefox-3.0 (Ubuntu) Importance: Undecided Status: New ** Tags: apport-bug -- Wrong search engine in right-click menu only in "wide screen" mode https://bugs.launchpad.net/bugs/310217 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
