1355000 - Permission panel opened in fullscreen is attached to current tab after exiting fullscreen

Status: NEW

(Firefox :: Site Permissions, defect, P3)

Description

[684sigma]

+++ This bug was initially created as a clone of Bug #1353980 +++

Nightly 55, somebody should check what's going on in other version and before bug 1353980

1. Open two tabs with https://www.youtube.com/watch?v=ZGqhhx8cxMg
2. Execute in each tab "navigator.geolocation.getCurrentPosition(function(){})"
3. Click on fullscreen button in the video in the first tab
4. Press Ctrl+Tab to switch to the second tab
5. Scroll the tab strip to the left

Result: when switching to the tab that isn't in fullscreen, permission panel is attached to current tab; after scrolling tab strip permission panel goes off-screen
Expected: should be attached to its button; shouldn't go off-sceen

Comment 1

[684sigma]

(Nightly 2017-04-09)

Comment 2

[Johann Hofmann [:johannh]]

Yeah the panel seems to get attached to the tab in that case, presumably because the anchor isn't there yet. That's after bug 1353980 which definitely made the situation better already. I think this is enough of an edge case to not affect any specific release.

I don't see the rapid blinking anymore though, can you clarify?

Comment 3

[Hani Yacoub]

Attachment: Recording #5.mp4

The permission panel is attached to the tab instead of the notification center after exiting full screen.
(Please see the screen cast attached)
[Steps to reproduce]:
1. Go to https://permission.site
2. Click on Fullscreen
3. Click on Notifications
4. Exit full screen (Esc)

Comment 4

[Neil Deakin]

I intentionaly didn't add support for when the anchor gets scrolled out of view in bug 1109868 as I didn't know what behaviour would be desirable: do nothing, hide the popup, move the popup to the scrolled edge.

Comment 5

[684sigma]

(In reply to Johann Hofmann [:johannh] from comment #2)
> I think this is enough of an edge case to not affect any specific release.

I'm pretty sure this bug affects one specific release - release 53.
The new notifications are extremely broken in general, and it also affects release 53.

> I don't see the rapid blinking anymore though, can you clarify?

There's no blinking, yes. The description doesn't mention it; the summary and priority was copied from Bug 1353980 automatically.

Comment 6

[Johann Hofmann [:johannh]]

(In reply to 684sigma from comment #5)
> (In reply to Johann Hofmann [:johannh] from comment #2)
> > I think this is enough of an edge case to not affect any specific release.
> 
> I'm pretty sure this bug affects one specific release - release 53.

What I mean is that we can ship 53 or any other release without fixing this.

The STR that Hani added suggest that we should get to this sooner rather than later though.

Comment 7

[Liz Henry (:lizzard Please n-i to RyanVM, jcristau, or pascal)]

Too late for 53, not a release blocker.