Closed
Bug 617570
Opened 14 years ago
Closed 14 years ago
editable menulist don't accept a placeholder attribute
Categories
(Toolkit :: UI Widgets, enhancement)
Toolkit
UI Widgets
Tracking
()
RESOLVED
FIXED
mozilla2.0b8
People
(Reporter: glazou, Assigned: glazou)
Details
Attachments
(1 file)
816 bytes,
patch
|
neil
:
review+
mconnor
:
superreview+
Gavin
:
approval2.0+
|
Details | Diff | Splinter Review |
Editable <menulist> elements should accept a placeholder attribute.
Assignee | ||
Comment 1•14 years ago
|
||
Updated•14 years ago
|
Attachment #496121 -
Flags: review+
Assignee | ||
Comment 2•14 years ago
|
||
Comment on attachment 496121 [details] [diff] [review]
fix proposal
asking mconnor for sr on this trivial patch
Attachment #496121 -
Flags: superreview?(mconnor)
Updated•14 years ago
|
Attachment #496121 -
Flags: superreview?(mconnor) → superreview+
Comment 4•14 years ago
|
||
While this wouldn't have needed super-review, it needs approval-2.0.
Assignee: nobody → daniel
Component: XP Toolkit/Widgets: Menus → XUL Widgets
Keywords: checkin-needed
Product: Core → Toolkit
QA Contact: xptoolkit.menus → xul.widgets
Updated•14 years ago
|
Attachment #496121 -
Flags: approval2.0+
Updated•14 years ago
|
Keywords: checkin-needed
Comment 5•14 years ago
|
||
Status: NEW → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
Assignee | ||
Comment 6•14 years ago
|
||
Gavin, thanks for a+!
Dão, thanks for checkin, that will help me a lot and _immediately_ for BlueGriffon!
You need to log in
before you can comment on or make changes to this bug.
Description
•