Closed
Bug 1332393
Opened 8 years ago
Closed 1 year ago
web.telegram.org push notifications opened inconsistently
Categories
(Toolkit :: Alerts Service, defect)
Toolkit
Alerts Service
Tracking
()
RESOLVED
INACTIVE
People
(Reporter: marco, Unassigned)
References
(Blocks 1 open bug)
Details
Alerts opened from web.telegram.org push notifications are shown on the top left corner of the screen (other push notifications are shown on the top right).
Comment 1•7 years ago
|
||
Also I noticed that when you click on them it open a new tab rather than the current tab open with Telegram.
Reporter | ||
Updated•7 years ago
|
Component: Notifications and Alerts → DOM: Push Notifications
Product: Toolkit → Core
Comment 2•7 years ago
|
||
(In reply to Rémy Hubscher (:natim) from comment #1)
> Also I noticed that when you click on them it open a new tab rather than the
> current tab open with Telegram.
This can be due to the site's service worker script specifically choosing to do it.
It could also be a consequence of an issue with multi-e10s.
What config are you on?
Blocks: ServiceWorkers-compat
Comment 3•7 years ago
|
||
More specifically, we need to know:
* desktop or mobile
* FF version number
* e10s configuration
If you post about:support output I think all of this will be in there.
Comment 4•7 years ago
|
||
- desktop Firefox 55.0 (nightly build)
- e10s activated.
Nom: Firefox
Version: 55.0a1
Identifiant de compilation: 20170531100318
Canal de mise à jour: nightly
Agent utilisateur: Mozilla/5.0 (X11; Linux x86_64; rv:55.0) Gecko/20100101 Firefox/55.0
Système d’exploitation: Linux 4.10.0-21-generic
Multi-process staged rollout 1.50 e10srollout@mozilla.org
Sorry my about:support is in French :(
Comment 5•7 years ago
|
||
Also I have to say my telegram is in a pin tab, which could also be the source of the problem.
Comment 6•7 years ago
|
||
Ok, the issue with opening in a new window is likely due to multi-e10s and not having bug 1293277 yet. Once that is fixed the problem should be resolved. Most likely in FF56.
Lets keep this bug for the notification placement issue described in comment 0. That sounds different and unrelated.
Comment 7•7 years ago
|
||
Ok sorry about the noise thanks.
Reporter | ||
Comment 8•7 years ago
|
||
(In reply to Ben Kelly [reviewing, but slowly][:bkelly] from comment #2)
> This can be due to the site's service worker script specifically choosing to
> do it.
In this case not, as you can't have two Telegram tabs open (Telegram will complain), so pretty sure they don't want to open a new tab.
Comment 9•7 years ago
|
||
(In reply to Ben Kelly [reviewing, but slowly][:bkelly] from comment #6)
>
> Lets keep this bug for the notification placement issue described in comment
> 0. That sounds different and unrelated.
In that case we should probably move the component. William, have you seen this before?
Component: DOM: Push Notifications → Notifications and Alerts
Flags: needinfo?(wchen)
Product: Core → Toolkit
Updated•6 years ago
|
Flags: needinfo?(william)
Updated•2 years ago
|
Severity: normal → S3
Updated•1 year ago
|
Component: Notifications and Alerts → DOM: Notifications
Product: Toolkit → Core
Moving back to toolkit per comment #9, and closing as inactive at it is.
Status: NEW → RESOLVED
Closed: 1 year ago
Component: DOM: Notifications → Alerts Service
Product: Core → Toolkit
Resolution: --- → INACTIVE
You need to log in
before you can comment on or make changes to this bug.
Description
•