1248427 - Notifications aren't shown in the right of the screen

Status: RESOLVED WORKSFORME

(Toolkit Graveyard :: Notifications and Alerts, defect)

Description

[Marco Castelluccio [:marco]]

Attachment: Screencast 2016-02-15 16:20:41.mp4

If I run the demo on MDN, the first notification is shown in the center of the screen, the following ones move to the left until they reach the left side. Then, all the following ones are shown on the left side.

Comment 1

[Jonathan Kew [:jfkthame]]

Ugh. :(  Which platform, and what's your display configuration like?

Comment 2

[Marco Castelluccio [:marco]]

Linux (Ubuntu), the window manager is the default one (Compiz).

In the settings, I've set scaling to 1,62.

`xrdb -query | grep dpi` tells me `Xft.dpi 156`

Comment 3

[Jonathan Kew [:jfkthame]]

What values of devicePixelRatio do we get for a window on each of your displays?

(One easy way to find out: load https://people.mozilla.org/~jkew/tests/devPixRatio.html, and drag the window to each display in turn.)

Comment 4

[Marco Castelluccio [:marco]]

I'm using a single display (I can reproduce the bug no matter how many displays I use).

Device Pixel Ratio: 2

Comment 5

[Liz Henry (:lizzard) (relman/hg->git project)]

Tracking for 47 since this sounds like a recent regression. Does it also affect aurora 46?

Comment 6

[Marco Castelluccio [:marco]]

I believe this is caused by bug 890156, which was backed out of Aurora. I'll test and update the flags.

Comment 7

[Jonathan Kew [:jfkthame]]

Huh, this is interesting.... for me (on Ubuntu 15.10) the notifications seem to be placed correctly if the display scaling is *below* 1.5 (tested at 1.25 and 1.38 as well as 1.0, and they worked OK in all cases). But at 1.5 or higher, they get misplaced.

Comment 8

[Jonathan Kew [:jfkthame]]

I believe the patch that just landed in bug 1240749 will fix this. We should re-test once that hits Nightly.

Comment 9

[Marco Castelluccio [:marco]]

Fixed!

Comment 10

[Liz Henry (:lizzard) (relman/hg->git project)]

Noting this was fixed in 47.