Closed
Bug 748150
Opened 12 years ago
Closed 12 years ago
Document contentStyle and contentStyleFile in the tutorials
Categories
(Add-on SDK Graveyard :: Documentation, defect, P1)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: wbamberg, Assigned: wbamberg)
Details
Attachments
(1 file)
2.25 KB,
patch
|
irakli
:
review+
|
Details | Diff | Splinter Review |
Need to update the tutorials to mention contentStyle and contentStyleFile.
Assignee | ||
Comment 1•12 years ago
|
||
Added a couple of notes in the tutorials to introduce contentStyle and contentStyleFile. I thought people might expect that tab.attach() would support this option, so I explicitly called out that it doesn't. Maybe it should? Are these options experimental, do you know?
Attachment #617675 -
Flags: review?(rFobic)
Comment 2•12 years ago
|
||
Comment on attachment 617675 [details] [diff] [review] a patch I think it would be a good idea to allow this options in tab.attach, but I don't remember why it's not working in current implementation. There maybe some technical limitations or it's just we have decided to ship what we had and improve in next iterations. ZER0 should now better. I personally don't see any reason why would not we allow these options for tab.attach. Yes this is experimental for the time being.
Attachment #617675 -
Flags: review?(rFobic) → review+
Updated•12 years ago
|
Priority: -- → P1
Comment 3•12 years ago
|
||
Commit pushed to master at https://github.com/mozilla/addon-sdk https://github.com/mozilla/addon-sdk/commit/82e9ede44ff1be229061ed80dc88bef95a3c9664 Bug 748150 - Document contentStyle and contentStyleFile in the tutorials;r=gozala
Assignee | ||
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 4•12 years ago
|
||
Commit pushed to stabilization at https://github.com/mozilla/addon-sdk https://github.com/mozilla/addon-sdk/commit/3779dc9e8bdba44da62b0947d2d0ed9c36603dc3 Bug 748150 - Document contentStyle and contentStyleFile in the tutorials;r=gozala (cherry picked from commit 82e9ede44ff1be229061ed80dc88bef95a3c9664)
You need to log in
before you can comment on or make changes to this bug.
Description
•