The heartbeat notification is displayed at the bottom of the page instead of the top

VERIFIED FIXED in Firefox 65

Status

()

defect
VERIFIED FIXED
6 months ago
6 months ago

People

(Reporter: danibodea, Assigned: Paolo)

Tracking

({regression})

Trunk
Firefox 65
Desktop
All
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox-esr60 unaffected, firefox63 unaffected, firefox64 unaffected, firefox65 verified)

Details

Attachments

(2 attachments)

Reporter

Description

6 months ago
[Note]:
- The heartbeat notification bar is displayed at the bottom of the page instead of the top.

[Affected versions]:
- (affected) Nightly v65.0a1;
- (not affected) Beta v64.0b8;
- (not affected) Release v63.0.1.

[Affected platforms]:
- most likely all, tested on Windows 10

[Pre-conditions:]
user_pref("security.content.signature.root_hash", "DB:74:CE:58:E4:F9:D0:9E:E0:42:36:BE:6C:C5:C4:F6:6A:E7:74:7D:C0:21:42:7A:03:BC:2F:57:0C:8B:9B:90");
user_pref("app.normandy.api_url", "https://normandy.stage.mozaws.net/api/v1");
user_pref("app.normandy.dev_mode", true);
user_pref("app.normandy.logging.level", 0);

[Steps to reproduce]:
1. Create/Publish a show-heartbeat recipe.
2. Run a firefox client with the pre-conditions needed for the heartbeat notification to be displayed.

[Expected result]:
- The heartbeat notification bar is displayed correctly at the top of the page.

[Actual result]:
- The heartbeat notification bar gets displayed at the bottom of the page.

[Regression range]:
- The mozregression tool does not work with Normandy experiments; A regression could not be performed.

[Additional notes]:
- This issue does not occur on the Beta and Release versions of the browser.
Last good revision: 3dd2018ed59a855b06eb7e8404fccc8e499ea4b9
First bad revision: b6676334369f8034686e9c50ceb941a27bfa6674
Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=3dd2018ed59a855b06eb7e8404fccc8e499ea4b9&tochange=b6676334369f8034686e9c50ceb941a27bfa6674

Seems like the regression was introduced by bug 1471403 .
Blocks: 1471403
Assignee

Comment 2

6 months ago
Thanks for reporting, I made a typo when rebasing the patch in bug 1471403.

Comment 4

6 months ago
Pushed by paolo.mozmail@amadzone.org:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f1d1658e82dd
Fix positioning of the high priority notification box. r=dao

Comment 5

6 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/f1d1658e82dd
Status: NEW → RESOLVED
Last Resolved: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 65
Assignee: nobody → paolo.mozmail
Reporter

Comment 6

6 months ago
I have verified this issue in v65.0a1 from 2018-11-15. Thank you.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.