Closed Bug 1841409 Opened 1 year ago Closed 1 year ago

The Feedback menu button sometimes isn't shown

Categories

(Firefox :: Address Bar, defect, P1)

defect

Tracking

()

VERIFIED FIXED
117 Branch
Tracking Status
firefox116 --- verified
firefox117 --- verified

People

(Reporter: adw, Assigned: adw)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

When a row is reused, the new result has showFeedbackMenu, and the old result didn't, the row won't show the Feedback menu button. We need to create new row content in that case.

This hasn't been a problem before because the Feedback button was only shown in best matches, which practically are never reused. Pocket non-best-match rows will show the button, which is how I noticed this.

Pushed by dwillcoxon@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/5cf32b0ba18a Create new row content as necessary to make sure the Feedback button is shown. r=dao

STR for QA:

  1. Set browser.urlbar.pocket.featureGate = true to enable Pocket suggestions
  2. Type "breakfast" and verify a low confidence (non-top-pick) Pocket suggestion appears
  3. Verify the Feedback button appears in the suggestion
  4. Modify your search string several times so the suggestion goes away and reappears (e.g., backspace over the last few characters, type a few extra characters, etc.) and verify the Feedback button always appears in the suggestion
Flags: qe-verify+
Flags: in-testsuite-

Comment on attachment 9342040 [details]
Bug 1841409 - Create new row content as necessary to make sure the Feedback button is shown.

Beta/Release Uplift Approval Request

  • User impact if declined: This bug is necessary for the Pocket suggestions feature we intend to ship as an experiment in 116.
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: Yes
  • If yes, steps to reproduce: Please see STR in previous comment
  • List of other uplifts needed: The following bugs must be uplifted first: 1841553, Bug 1841408
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Very small change that only affects urlbar suggestions with the "Feedback" button in them, and those suggestions are disabled by default. Covered by tests.
  • String changes made/needed:
  • Is Android affected?: No
Attachment #9342040 - Flags: approval-mozilla-beta?

Backed out for causing mochitests failures.

  • Backout link
  • Push with failures
  • Failure Log
  • Failure line: PROCESS-CRASH | MOZ_ASSERT(mInitialZoomConstraints->mPresShellID == aPresShellId) [@ libxul.so + 0x00000000048f60f4] | toolkit/components/extensions/test/mochitest/mochitest.ini:toolkit/components/extensions/test/mochitest/mochitest-common.ini
Flags: needinfo?(adw)
Pushed by sstanca@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4c1ffa30508d Create new row content as necessary to make sure the Feedback button is shown. r=dao
Flags: needinfo?(adw)
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 117 Branch
QA Whiteboard: [qa-triaged]

Comment on attachment 9342040 [details]
Bug 1841409 - Create new row content as necessary to make sure the Feedback button is shown.

Approved for 116.0b2

Attachment #9342040 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

I have verified this issue on the latest Firefox Nightly 117.0a1 (Build ID: 20230706092222), on Windows 10 x64, macOS 13.4.1, and Ubuntu 20.04 x64.

  • The "Cat Cora" Pocket suggestion appears at the bottom of the Urlbar panel after typing the complete string (“breakfast”). The label displayed above the suggestion is "Firefox Suggest".
  • The “Feedback” button is displayed on the right side of the suggestion.
  • The “Feedback” button remains displayed when deleting one or more characters from the keyword.
  • The “Feedback” button remains displayed when adding one or more characters of a full keyword (“breakfast casserole recipes”)
  • The “Feedback” button is displayed again when typing a keyword after previously clicking the Pocket suggestion.
  • The “Feedback” button is displayed again if re-focusing the Urlbar when a keyword is already typed inside the Address Bar.

I have verified this issue on the latest Firefox Beta 116.0b2 (Build ID: 20230706193912), on Windows 10 x64, macOS 13.4.1, and Ubuntu 20.04 x64.

  • The "Cat Cora" Pocket suggestion appears at the bottom of the Urlbar panel after typing the complete string (“breakfast”). The label displayed above the suggestion is "Firefox Suggest".
  • The “Feedback” button is displayed on the right side of the suggestion.
  • The “Feedback” button remains displayed when deleting one or more characters from the keyword.
  • The “Feedback” button remains displayed when adding one or more characters of a full keyword (“breakfast casserole recipes”)
  • The “Feedback” button is displayed again when typing a keyword after previously clicking the Pocket suggestion.
  • The “Feedback” button is displayed again if re-focusing the Urlbar when a keyword is already typed inside the Address Bar.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: