Closed Bug 1700137 Opened 4 years ago Closed 4 years ago

[Proton] "Add Search Engine" missing in address bar context menu after theme is changed.

Categories

(Firefox :: Address Bar, defect, P2)

Desktop
All
defect

Tracking

()

VERIFIED FIXED
89 Branch
Tracking Status
firefox86 --- unaffected
firefox87 --- unaffected
firefox88 --- wontfix
firefox89 --- verified

People

(Reporter: muirpablo, Assigned: mak)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [proton-address-bar])

Attachments

(2 files)

Attached image contextmenu.jpg (deleted) —

"Add Search Engine" missing in address bar context menu after theme is changed.

Tested on Windows10 64bit , MacOS 10.14, Ubuntu 20 64bit.
Firefox Nightly 88 Affected
Firefox beta 87 (option Not available)
Firefox Release 86.0.1 (option Not available)

Steps

  1. set user.js on a clean profile.
    user_pref("browser.proton.enabled", true);
    user_pref("browser.proton.toolbar.enabled", true);
    user_pref("browser.proton.urlbar.enabled", true);

  2. open a tab and visit youtube.com

  3. Right click on address bar

  4. notice "add youtube" option is available in context menu.( its the last option)

  5. Now Change the firefox theme to LIGHT theme.

  6. go again to youtube

  7. right click on address bar.

Actual result
add youtube is missing.

Expected result
Add youtube should still be visible

Has STR: --- → yes
Severity: -- → S3
Whiteboard: [proton-address-bar]
Summary: "Add Search Engine" missing in address bar context menu after theme is changed. → [Proton] "Add Search Engine" missing in address bar context menu after theme is changed.

@rtestard, could you please set a priority for this here?

Flags: needinfo?(rtestard)

The default access point will be the one in the shortcut buttons, that works as expected.
We also added the context menu access point to cover two cases:

  1. the user disabled top sites, so the urlbar doesn't open when clicking on it
  2. the page provides more than 3 engines

The latter is not so critical, since quite uncommon, but the former may be more common (I'm not sure whether we measure how many users disabled top sites though).

I actually thought to have addressed this issue, but I didn't consider that we destroy and recreate the widget in certain cases, Customize is one of them, so after customize this stops working because the event handler I added is pointing to a dead widget.
I think I have a good enough idea of the problem to look for a solution.

Assignee: nobody → mak
Status: NEW → ASSIGNED
Flags: needinfo?(rtestard)
Priority: -- → P2
Pushed by mak77@bonardo.net: https://hg.mozilla.org/integration/autoland/rev/ac1d613a06d2 Add Search Engine is missing in address bar context menu after customization. r=harry
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch

The patch landed in nightly and beta is affected.
:mak, is this bug important enough to require an uplift?
If not please set status_beta to wontfix.

For more information, please visit auto_nag documentation.

Flags: needinfo?(mak)

This only affects Proton that is not enabled in 88.

Flags: needinfo?(mak)

verified fix on Firefox nightly 89.0a1 using windows10 64bit , macos 10.15 and ubuntu 20 64bit

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: