Closed Bug 1841887 Opened 2 years ago Closed 2 years ago

When restarting Firefox a translation is proposed for a website I added in the deny list

Categories

(Firefox :: Translations, defect)

Firefox 117
defect

Tracking

()

RESOLVED FIXED
Tracking Status
firefox117 --- affected

People

(Reporter: calixte, Unassigned)

References

Details

I disabled translations for gmail which is in a pinned tab.
And when I restarted Firefox, a translation is proposed but when I click on the parameter wheel I can see that Never translate this site is already checked.

Might be the same root cause as bug 1841045.

See Also: → 1841045

I think I've a reliable way to reproduce it (note that I've around 40 pinned tabs).
After I restarted Firefox, if I click on the gmail pinned tab when it's loading then I get the popup but if I click once it's loaded then no popup.

This is similar to what I'm doing to reproduce in bug 1841045.
I can also reproduce randomly while navigating, but it always seems related to some page being in the loading phase.

I found it, it's a race condition.

    browser.dispatchEvent(
      new CustomEvent("TranslationsParent:OfferTranslation", {
        bubbles: true,
      })
    );

This event needs to double check that it's still on the current tab.

This should be fixed by Bug 1841045.

Depends on: 1841045

Calixte, could you verify this is fixed in latest Nightly?

Flags: needinfo?(cdenizet)
Severity: -- → S3

Yep it works fine.
But (sorry there's a but), now the icon in the toolbar is no more here which means that if I want to be able to translate it (because I changed my mind or just punctually) I don't know where I can do that.

Flags: needinfo?(cdenizet)

That doesn't sound right. Adding a NI to myself to investigate.

Flags: needinfo?(gtatum)

Calixte: Can you provide a steps to reproduce and verify the icon is still hidden in your case? I suspect you are reporting an older behavior. I changed the behavior when we added automatic popup offerings so that when you chose "never translate" it only doesn't popup the menu, but it will still show the icon. In addition, "Translate page..." is available from the hamburger menu when you still need to manually invoke it.

Flags: needinfo?(gtatum) → needinfo?(cdenizet)

I removed French from Webpage Language Settings, and everything is ok.

Status: NEW → RESOLVED
Closed: 2 years ago
Flags: needinfo?(cdenizet)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.