Closed
Bug 709737
Opened 12 years ago
Closed 12 years ago
Test Pilot notifications don't work
Categories
(Thunderbird :: General, defect)
Thunderbird
General
Tracking
(thunderbird9+ fixed, thunderbird10 fixed)
RESOLVED
FIXED
Thunderbird 11.0
People
(Reporter: standard8, Assigned: standard8)
References
Details
Attachments
(1 file)
23.58 KB,
patch
|
bwinton
:
review+
bwinton
:
ui-review+
standard8
:
approval-comm-aurora+
|
Details | Diff | Splinter Review |
Having taken part in the test study on the development channel, the study is now complete, but I haven't been prompted to submit my data. The error console says: Error: popup is null Source File: resource://testpilot/modules/notifications.js Line: 103 Looks like as we removed the feedback button, we've not got a notification option. Is there somewhere else we could hang this from?
Comment 1•12 years ago
|
||
Mconley, do you mind taking a look at this? It should be fairly straightforward.
Assignee | ||
Comment 3•12 years ago
|
||
This should fix up test pilot for us. It fixes: - Notifications (they now get displayed, anchored to the toolbar) - Build test pilot on all channels (i.e. include release) - Looks the same on all channels - Links in the experiment page now open in the browser, and use the FF style one. - Removes obsolete interface.js.orig - Removes the preferences box that was meant for mobile and not the browser.
Attachment #581822 -
Flags: ui-review?(bwinton)
Attachment #581822 -
Flags: review?(bwinton)
Comment 4•12 years ago
|
||
Comment on attachment 581822 [details] [diff] [review] The fix Review of attachment 581822 [details] [diff] [review]: ----------------------------------------------------------------- So, I'm not so happy with the flash you can see in http://dl.dropbox.com/u/2301433/Screenshots/TestPilotFlash.png and the notification pops up in a bit of a weird place as seen in http://grab.by/bpeC , but I'm going to say ui-r=me for now, and we can fix these later. And I'm pretty happy with the code, too, so r=me, modulo the comments below. ::: mail/app/profile/extensions/tbtestpilot@labs.mozilla.com/install.rdf.in @@ -6,4 +6,4 @@ > > xmlns:em="http://www.mozilla.org/2004/em-rdf#"> > > <Description about="urn:mozilla:install-manifest"> > > <em:id>tbtestpilot@labs.mozilla.com</em:id> > > - <em:version>1.3.1</em:version> > > + <em:version>1.3.2</em:version> The version you sent me on IRC said "1.3.1" and "By: Mozilla Messaging". As long as those are "1.3.2" and "Mozilla Corporation" (or "Mozilla"), I'm happy with that. ::: mail/app/profile/extensions/tbtestpilot@labs.mozilla.com/modules/notifications.js @@ -186,4 +189,4 @@ > > // Show the popup: > > popup.hidden = false; > > popup.setAttribute("open", "true"); > > - popup.openPopup( anchor, "after_end"); > > + // XXX Thunderbird needs after_start for now. Why is this? If it's a reasonable reason, can we remove the "XXX"? If not, can we file a followup bug?
Attachment #581822 -
Flags: ui-review?(bwinton)
Attachment #581822 -
Flags: ui-review+
Attachment #581822 -
Flags: review?(bwinton)
Attachment #581822 -
Flags: review+
Assignee | ||
Updated•12 years ago
|
Attachment #581822 -
Flags: approval-comm-beta+
Attachment #581822 -
Flags: approval-comm-aurora+
Assignee | ||
Comment 5•12 years ago
|
||
Checked in: http://hg.mozilla.org/comm-central/rev/30263aa049c5 http://hg.mozilla.org/releases/comm-aurora/rev/a92b37de1cd9 http://hg.mozilla.org/releases/comm-beta/rev/63e206326ac3
Status: NEW → RESOLVED
Closed: 12 years ago
status-thunderbird10:
--- → fixed
status-thunderbird9:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 11.0
Assignee | ||
Comment 6•12 years ago
|
||
I had to do a follow-up to create the panel in the menu bar element rather than the toolbar, as this was messing up the migration assistant. The position is still the same though. This also fixed the unit tests on trunk: http://hg.mozilla.org/comm-central/rev/2368d15d14be http://hg.mozilla.org/releases/comm-aurora/rev/6a837d0eb99f http://hg.mozilla.org/releases/comm-beta/rev/8672f5e16ec0
Comment 7•12 years ago
|
||
> -<toolbar id="mail-bar3">
> +<toolbar id="mail-toolbox">
shouldn't this be <*toolbox* id="mail-toolbox">
Comment 8•12 years ago
|
||
Hm - yes, it should be.
Assignee | ||
Comment 9•12 years ago
|
||
Checked in an additional fix for the element tag, with r=mconley over irc: http://hg.mozilla.org/comm-central/rev/9a410ad48bb0 http://hg.mozilla.org/releases/comm-aurora/rev/fd114815d194 http://hg.mozilla.org/releases/comm-beta/rev/88ea5ee550f5
Comment 10•12 years ago
|
||
Checked in additional additional fix to comm-central as: http://hg.mozilla.org/comm-central/rev/b37a9a98d2ba
Assignee | ||
Comment 11•12 years ago
|
||
and to aurora as: http://hg.mozilla.org/releases/comm-aurora/rev/46cc23c3ac7d this also made it onto comm-release (comm-beta isn't required at this time, as that had already merged to release): http://hg.mozilla.org/releases/comm-release/rev/df55e2ad2983
You need to log in
before you can comment on or make changes to this bug.
Description
•