Closed Bug 1585847 Opened 5 years ago Closed 5 years ago

Megabar results have transparent background after pressing Esc

Categories

(Firefox :: Address Bar, defect, P1)

defect
Points:
3

Tracking

()

RESOLVED FIXED
Firefox 71
Iteration:
71.3 - Sept 30 - Oct 13
Tracking Status
firefox-esr60 --- unaffected
firefox-esr68 --- unaffected
firefox67 --- unaffected
firefox68 --- unaffected
firefox69 --- unaffected
firefox70 --- unaffected
firefox71 --- fixed

People

(Reporter: emilio, Assigned: mak)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

STR (with megabar enabled):

  • Focus the megabar in any way (like Ctrl + L on Linux).
  • Pres the Esc key. This selects the URL.
  • Start typing to navigate somewhere else.

Expected:

  • The results popup appears.

Actual:

  • You can type on the URL bar, but there's no autocompletion or history of any sort.
Points: --- → 3
Priority: -- → P1

Harry or Dão, could this be a regression caused specifically by bug 1583855?

Flags: needinfo?(htwyford)
Flags: needinfo?(dao+bmo)

This regression appears to be related to autofill behaviour. If the character typed after pressing ESC does not have an autofill result associated with it (e.g. typing a number or a special character) the view opens. Furthermore, this behaviour seems to have been introduced since bug 1583855 landed since this behaviour doesn't exist after reverting to the bug 1583855 patch. Can I please get a regressionwindow on this?

Assignee: nobody → htwyford
Status: NEW → ASSIGNED
Iteration: --- → 71.3 - Sept 30 - Oct 13
Flags: needinfo?(htwyford)

(In reply to Mike de Boer [:mikedeboer] from comment #1)

Harry or Dão, could this be a regression caused specifically by bug 1583855?

I would expect so. I suspect we're getting the results without displaying them because the height of the textbox is fixed.

Flags: needinfo?(dao+bmo)
Regressed by: 1583855

Bug 1586052 might also be related. I think the root cause of both bugs is that there's an issue with the Focused & Collapsed state and the view opening. I'll investigate.

(In reply to Dão Gottwald [::dao] from comment #4)

(In reply to Mike de Boer [:mikedeboer] from comment #1)

Harry or Dão, could this be a regression caused specifically by bug 1583855?

I would expect so. I suspect we're getting the results without displaying them because the height of the textbox is fixed.

Bug 1586232 being fixed should make this more visible as we're not hiding content overflowing the textbox anymore.

Summary: Megabar stops auto-completing when you press escape. → Megabar results have transparent background after pressing Esc

STR:

  1. Focus the urlbar and press "m" key (autocompletes to mozilla.org).
  2. Press Esc key twice (removes results and shrinks megabar).
  3. Press "m" key.

Results fail to appear, megabar does not expand.

Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=5d83a502d367de1332ceb234275416f315328340&tochange=a633169505d41ea1377124758adc13eac7ec0ff0

Regressed by Bug 1584270.

Bug 1586232 improved the situation, now the results appear but still without megabar expansion.

Has Regression Range: --- → yes
Has STR: --- → yes
Regressed by: 1584270
No longer regressed by: 1583855
Assignee: htwyford → mak77
Pushed by mak77@bonardo.net: https://hg.mozilla.org/integration/autoland/rev/9323e7d49d49 Megabar doesn't expand properly when autofill is involved. r=harry
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 71
Flags: qe-verify+
Flags: qe-verify+ → qe-verify-
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: