fix calendar-ui-utils' addMenuItem

VERIFIED FIXED in 0.9

Status

VERIFIED FIXED
11 years ago
10 years ago

People

(Reporter: dbo, Assigned: dbo)

Tracking

unspecified

Details

Attachments

(1 attachment)

(Assignee)

Description

11 years ago
The passed over value is not set.
(Assignee)

Comment 1

11 years ago
Created attachment 321468 [details] [diff] [review]
fix
Assignee: nobody → daniel.boelzle
Status: NEW → ASSIGNED
Attachment #321468 - Flags: review?(Berend.Cornelius)
Comment on attachment 321468 [details] [diff] [review]
fix

Interesting, http://developer.mozilla.org/en/docs/XUL:menuitem tells us that the value property is valid. While this workaround is valid, checking if this works on trunk and filing a core bug otherwise might be an option for the future, since setting the value property is a bit more compact.

r=philipp
Attachment #321468 - Flags: review?(Berend.Cornelius) → review+
(Assignee)

Comment 3

11 years ago
Checked in on HEAD and MOZILLA_1_8_BRANCH => FIXED.
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 0.9

Comment 4

10 years ago
Checked via mxr.mozilla.org -> VERIFIED.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.