1277620 - Closing Outlook Web App new email notification instead opens email

Status: NEW

(Core :: DOM: Events, defect, P3)

Description

[Juliana Perry]

Attachment: owa_notification.jpg

User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:46.0) Gecko/20100101 Firefox/46.0
Build ID: 20160502172042

Steps to reproduce:

1. Receive new mail popup notification in Outlook Web App
2. Click X to close notification



Actual results:

New window opens with email


Expected results:

Notification should close

Comment 1

[Juliana Perry]

This sounds like it could be similar to https://bugzilla.mozilla.org/show_bug.cgi?id=785490

Comment 2

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

jaws, what do you think? Maybe this is a new issue in Windows 10.

Comment 3

[Jared Wein [:jaws] (please needinfo? me)]

It looks like this has some different styling than our notifications, maybe it is some part of the Outlook Web Access website? I don't think I have access to an Outlook Web Access account. 

Juliana, does this notification appear outside of the Firefox window? Does it appear if Firefox is minimized? If this is part of the webpage, there isn't much we can do here. Our notifications are external to the Firefox window.

Alternatively, do you have any themes installed that may have changed the look of our notifications? Can you reproduce this bug if you choose "Restart with Add-ons Disabled..." from the Firefox Help menu (Alt+H)?

Comment 4

[Juliana Perry]

Yes, these are pop-ups that come from the Outlook Web App webpages, rather than being Firefox notifications. Apologies. 
This issue occurs in the latest version of Firefox on Mac OSX and Windows 10 and 7 machines. It does not occur in current versions IE, Chrome, or Safari. We spoke with Microsoft Premier support about this issue and their position is that it's not an issue they're going to address since it occurs only in Firefox. However, I entirely understand if the Firefox development community sees this differently, or if there's little to be done on the Firefox side, particularly since it requires an Outlook account.

Comment 5

[Juliana Perry]

Also note: 
I did try restarting Firefox with add-ons disabled; this did not resolve the issue.

Comment 6

[Jared Wein [:jaws] (please needinfo? me)]

Thanks for the reply.

Comment 7

[Jared Wein [:jaws] (please needinfo? me)]

You can file this at https://webcompat.com/ so it can be investigated as to why it works differently for Firefox.

Comment 8

[Hallvord R. M. Steen [:hallvors]]

Hm.. We're looking at nested BUTTON elements!

<button type="button" class="ms-bgc-nl-h o365button o365cs-notifications-newMailLink"><div class="o365cs-notifications-newMailPopupButton ms-bcl-tp"> <div class="o365cs-notifications-newMailPopupButtonCell"> <div class="o365cs-notifications-newMailPopupButtonContent"> <span style="display: none;" autoid="__Microsoft_O365_ShellG2_Owa_templates_cs_Y" class="wf-size-x14 o365cs-notifications-text ms-fcl-nd">You have 1 new messages</span> <div><div style="display: none; background-color: rgb(0, 163, 0); outline-color: rgb(0, 163, 0);" class="o365cs-notifications-newMailPersonaImage">HR</div><img class="o365cs-notifications-newMailPersonaImageDontDisplay" src="service.svc/s/GetPersonaPhoto?email=[e-mail]&amp;UA=0&amp;size=HR64x64&amp;sc=1465392546749" style=""><span autoid="__Microsoft_O365_ShellG2_Owa_templates_cs_Z" class="wf-size-x14 o365cs-notifications-text o365cs-segoeRegular">Hallvord Reiar Michaelsen Steen</span><span autoid="__Microsoft_O365_ShellG2_Owa_templates_cs_01" class="wf-size-x12 o365cs-notifications-text o365cs-segoeSemiBold ms-fcl-tp">hallo, bli der da du boks</span><span autoid="__Microsoft_O365_ShellG2_Owa_templates_cs_11" class="wf-size-x12 o365cs-notifications-text o365cs-notifications-bodypreviewtext o365cs-segoeSemiLight ms-fcl-ns">
</span></div> <button aria-label="Close" type="button" class="o365cs-notifications-closeButton ms-fcl-np ms-bgc-w-h o365button"><span class="o365cs-notifications-closeIcon owaimg ms-Icon--x ms-icon-font-size-16"> </span></button> </div> </div> </div></button>

That's a weird way to do things.. :-o

Comment 9

[Karl Dubost💡 :karlcow]

not sure what it is supposed to do, but
Blink and Gecko seems consistent here

http://codepen.io/anon/pen/EyVJKq

They create two individual buttons.

Comment 10

[Hallvord R. M. Steen [:hallvors]]

Attachment: 1277620 test case

One needs to actually use JS to create such a DOM with truly nested buttons. The markup parser will not allow the construct.

The interesting thing here is that Gecko does not fire any click event on the inner BUTTON at all. From the script's point of view, clicking the inner BUTTON (the X) can not be distinguished from clicking the outer (which opens the E-mail).

Comment 11

[Hallvord R. M. Steen [:hallvors]]

This rings a bell, I think there is a master or related bug (perhaps to do with BUTTON inside A or vice versa). Boris, do you know?

Comment 12

[Boris Zbarsky [:bzbarsky]]

See discussion in bug 1089326.

Comment 13

[Andrew Overholt [:overholt]]

Boris, why does this depend on bug 1089326 instead of just being a dupe?

Comment 14

[Boris Zbarsky [:bzbarsky]]

Because it's entirely non-obvious to me what changes will get made for bug 1089326 in the end and whether they will fix the observable website problem here.