Last Comment Bug 308591 - Button.dlgType sets an attribute with the wrong case
: Button.dlgType sets an attribute with the wrong case
: fixed1.8.1
Product: Toolkit
Classification: Components
Component: XUL Widgets (show other bugs)
: unspecified
: All All
-- normal (vote)
: mozilla1.8.1
Assigned To: Neil Deakin
: Neil Deakin
Depends on:
  Show dependency treegraph
Reported: 2005-09-14 19:16 PDT by Neil Deakin
Modified: 2006-02-01 11:06 PST (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

change case of dlgType (2.39 KB, patch)
2005-09-14 19:21 PDT, Neil Deakin
neil: first‑review+
mconnor: approval‑branch‑1.8.1+
Details | Diff | Splinter Review

Description User image Neil Deakin 2005-09-14 19:16:11 PDT
Button's dlgType property is setting the attribute with the uppercase T but the
dialog uses lowercase dlgtype.
Comment 1 User image Neil Deakin 2005-09-14 19:21:04 PDT
Created attachment 196110 [details] [diff] [review]
change case of dlgType
Comment 2 User image 2005-09-19 05:41:35 PDT
Comment on attachment 196110 [details] [diff] [review]
change case of dlgType

Comment 3 User image :Gavin Sharp [email:] 2005-11-25 13:11:35 PST
This was never checked in... Neil?
(Also, this means that the dlgType property never worked, right?)
Comment 4 User image Nickolay_Ponomarev 2006-01-15 14:17:57 PST
Enn said that "it can be checked in if the reviews are ok" on IRC.
Comment 5 User image Neil Deakin 2006-01-15 14:25:46 PST
I checked it in.
Comment 6 User image :Gavin Sharp [email:] 2006-02-01 11:06:46 PST
Checked in on the 1.8 branch.
mozilla/toolkit/content/widgets/button.xml; new revision:;
mozilla/xpfe/global/resources/content/bindings/button.xml; new revision:;

Note You need to log in before you can comment on or make changes to this bug.