Closed
Bug 497123
Opened 15 years ago
Closed 15 years ago
Bring back gContextMenu.imageURL as per Firefox Bug 497098
Categories
(SeaMonkey :: UI Design, defect)
SeaMonkey
UI Design
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.0b1
People
(Reporter: philip.chee, Assigned: bugzilla)
References
Details
Attachments
(1 file)
|
(deleted),
patch
|
neil
:
review+
neil
:
superreview+
|
Details | Diff | Splinter Review |
Bug 497098 removed/replaced gContextMenu.imageURL with gContextMenu.mediaURL. Apparently there are a boatload of Firefox extensions that depend on imageURL so Firefox is bringing it back after a bit of angst.
Here is a patch that ports over the fix from Firefox.
Assignee: nobody → bugzilla
Status: NEW → ASSIGNED
Attachment #382849 -
Flags: superreview?(neil)
Attachment #382849 -
Flags: review?(neil)
|
||
Updated•15 years ago
|
Attachment #382849 -
Flags: superreview?(neil)
Attachment #382849 -
Flags: superreview+
Attachment #382849 -
Flags: review?(neil)
Attachment #382849 -
Flags: review+
|
|
||
Comment 2•15 years ago
|
||
Comment on attachment 382849 [details] [diff] [review]
Patch to add imageURL
[Checkin: Comment 4]
[Although I would probably have plumped for a ? : expression]
|
Reporter | |
Comment 3•15 years ago
|
||
Firefox/Toolkit reviewers seem to have this aversion to the ternary operator.
Keywords: checkin-needed
|
||
Comment 4•15 years ago
|
||
Comment on attachment 382849 [details] [diff] [review]
Patch to add imageURL
[Checkin: Comment 4]
http://hg.mozilla.org/comm-central/rev/c2280881464b
Attachment #382849 -
Attachment description: Patch to add imageURL → Patch to add imageURL
[Checkin: Comment 4]
|
|
||
Updated•15 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: seamonkey2.0b2 → seamonkey2.0b1
You need to log in
before you can comment on or make changes to this bug.
Description
•