Closed Bug 1597077 Opened 5 years ago Closed 5 years ago

Fix padding for tip results

Categories

(Firefox :: Address Bar, defect, P1)

defect
Points:
2

Tracking

()

RESOLVED FIXED
Firefox 72
Iteration:
72.3 - Nov 18 - Dec 1
Tracking Status
firefox-esr68 --- unaffected
firefox70 --- unaffected
firefox71 --- unaffected
firefox72 --- fixed

People

(Reporter: adw, Assigned: adw)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files)

Tip results don't have their intended padding. Looks like bug 1593959 regressed this.

it would be nice to have a script to run in the Browser Console, or an add-on, registering a provider that returns one result per type, when testing theme/alignment changes. Getting Tip results is a problem because there's no integrated provider doing it, nor an add-on one can easily install for testing purposes.

Attached patch tip_test.diff (deleted) — Splinter Review

I've kept this patch floating around in my tree and rebase it on top of my work to test tip changes. I agree we should have something a bit easier to use though.

Is mozscreenshots still running? We could use that. Otherwise, in addition to the ideas mentioned here (an add-on, Harry's script), another manual testing possibility is to run one of the browser tests that creates a tip and stop it (e.g. add an await new Promise(r => {})) when it's shown.

Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 72

(In reply to Harry Twyford [:harry] from comment #3)

we should have something a bit easier to use

Here's an add-on! https://github.com/htwyford/urlbar-results-tester

Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: