Closed
Bug 588552
Opened 14 years ago
Closed 14 years ago
notifications module should set textClickable argument if onClick handler is given
Categories
(Add-on SDK Graveyard :: General, defect)
Add-on SDK Graveyard
General
Tracking
(Not tracked)
RESOLVED
FIXED
0.7
People
(Reporter: adw, Assigned: adw)
Details
Attachments
(1 file)
974 bytes,
patch
|
myk
:
review+
|
Details | Diff | Splinter Review |
Someone tweeted [1] that his notification's onClick handler isn't getting called. He speculates it's because the implementation always sets the textClickable argument of nsIAlertsService.showAlertNotification() to false. The IDL says this about textClickable: If true, causes the alert text to look like a link and notifies the listener when user attempts to click the alert text. D'oh. textClickable's value appears to make no difference on OS X and Growl, but the screenshot provided with the tweet looks like Windows XP, so it might matter there and who knows where else. This patch sets textClickable to true if an onClick handler is given. [1] http://twitter.com/asukaze55/status/21211275998 [2] http://mxr.mozilla.org/mozilla-central/source/toolkit/components/alerts/public/nsIAlertsService.idl#52
Attachment #467151 -
Flags: review?(myk)
Comment 1•14 years ago
|
||
Comment on attachment 467151 [details] [diff] [review] patch r=myk and a0.7=myk for checkin during freeze!
Attachment #467151 -
Flags: review?(myk) → review+
Assignee | ||
Comment 2•14 years ago
|
||
Thanks for the quick review. http://hg.mozilla.org/labs/jetpack-sdk/rev/24e2df1a66eb
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: -- → 0.7
Comment 3•14 years ago
|
||
The Add-on SDK is no longer a Mozilla Labs experiment and has become a big enough project to warrant its own Bugzilla product, so the "Add-on SDK" product has been created for it, and I am moving its bugs to that product. To filter bugmail related to this change, filter on the word "looptid".
Component: Jetpack SDK → General
Product: Mozilla Labs → Add-on SDK
QA Contact: jetpack-sdk → general
Version: Trunk → unspecified
You need to log in
before you can comment on or make changes to this bug.
Description
•