Closed Bug 1932223 Opened 9 months ago Closed 9 months ago

Full keywords should not bypass length threshold for Sponsored Suggestion Higher Placement Phase 3

Categories

(Firefox :: Address Bar, defect, P1)

defect

Tracking

()

RESOLVED FIXED
134 Branch
Tracking Status
firefox132 --- wontfix
firefox133 --- wontfix
firefox134 --- fixed

People

(Reporter: adw, Assigned: adw)

References

Details

(Whiteboard: [sng])

Attachments

(1 file)

Bug 1913507 included a condition in Firefox’s “should show AMP as top pick?” logic that will show a top pick if the full keyword is typed, even if the full keyword length < the threshold length. That was an error I think, due in part to me misunderstanding what “full keyword” means. The full keywords stored in remote settings and returned by the Rust component are not what you would intuitively think of as "full keywords". They usually aren't full phrases and often aren't even full words. We should remove that condition.

Summary: Full keywords should not trigger top pick for Sponsored Suggestion Higher Placement Phase 3 → Full keywords should not bypass length threshold for Sponsored Suggestion Higher Placement Phase 3
Pushed by dwillcoxon@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f81aff297496 Full keywords should not bypass length threshold for Sponsored Suggestion Higher Placement Phase 3. r=daisuke
Status: ASSIGNED → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → 134 Branch
Blocks: 1961041
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: