Closed
Bug 971251
Opened 11 years ago
Closed 10 years ago
Bookmark(tab) should work
Categories
(Add-on SDK Graveyard :: General, defect, P2)
Add-on SDK Graveyard
General
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: evold, Unassigned, Mentored)
References
Details
I'm guessing this does work atm, but I see no tests for it.
Priority: -- → P2
|
||
Comment 1•11 years ago
|
||
Don't think this works currently, and shouldn't, but this would be a nice to have.
Whiteboard: [mentor=jsantell@mozilla.com]
|
Assignee | |
Updated•10 years ago
|
Mentor: jsantell
Whiteboard: [mentor=jsantell@mozilla.com]
|
Reporter | |
Updated•10 years ago
|
Assignee: nobody → evold
|
|
Reporter | |
Comment 2•10 years ago
|
||
I'm thinking about making a `bookmarkable = method("");` in say `sdk/places/bookmark/util.js`, does that sound alright to you Jordan?
Flags: needinfo?(jsantell)
|
|
||
Comment 3•10 years ago
|
||
I think that sounds good, maybe a different name to be consistent with our other method-ized functions (bookmarkFor?) depending on how it's used
Flags: needinfo?(jsantell)
|
|
Reporter | |
Comment 4•10 years ago
|
||
I can't get to this for awhile, I'm going to leave it open for now.
Assignee: evold → nobody
|
|
Reporter | |
Updated•10 years ago
|
Blocks: sdk/places
|
|
Reporter | |
Comment 5•10 years ago
|
||
At this point I think it would be best for a 3rd party module to do this.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•