Closed Bug 1673341 Opened 4 years ago Closed 4 years ago

Permanent menubar visibility toggling doesn't work (menu bar is hidden again)

Categories

(Firefox :: Toolbars and Customization, defect, P1)

defect

Tracking

()

RESOLVED FIXED
84 Branch
Tracking Status
firefox-esr78 --- unaffected
firefox81 --- unaffected
firefox82 --- unaffected
firefox83 --- unaffected
firefox84 --- fixed

People

(Reporter: Gijs, Assigned: Gijs)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

STR:

  1. open Firefox
  2. right click navbar
  3. click "menu bar" to unhide the menubar
  4. open a new window

ER:
menu bar is visible

AR:
menu bar is hidden

Root cause here is that we persist the attribute before we toggle it; so we end up persisting the "autohide" value "true" before toggling it to "false".

We should add tests for this...

Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Flags: in-testsuite?
Severity: -- → S2
Priority: -- → P1
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/3f3d77fb0f37
fix menu bar toggling to persist the correct state, r=jaws
Flags: in-testsuite? → in-testsuite+
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 84 Branch
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: