Closed
Bug 253607
Opened 20 years ago
Closed 16 years ago
Warning: redeclaration of var column
Categories
(Core :: XUL, defect)
Core
XUL
Tracking
()
RESOLVED
FIXED
mozilla1.8alpha3
People
(Reporter: timeless, Assigned: sgautherie)
References
Details
Attachments
(3 obsolete files)
Warning: redeclaration of var column Source File: chrome://global/content/bindings/tree.xml Line: 766, Column: 18 Source Code: var column = tree.columns.getColumnFor(sib);
Comment 1•20 years ago
|
||
Updated•20 years ago
|
Assignee: varga → mcsmurf
OS: Windows XP → All
Hardware: PC → All
Attachment #154931 -
Flags: superreview?(neil.parkwaycc.co.uk)
Attachment #154931 -
Flags: review+
Comment 2•20 years ago
|
||
Comment on attachment 154931 [details] [diff] [review] Patch By my reckoning there are four declarations of column and they're all wrong.
Attachment #154931 -
Flags: superreview?(neil.parkwaycc.co.uk) → superreview-
Comment 3•20 years ago
|
||
Updated•20 years ago
|
Attachment #154931 -
Attachment is obsolete: true
Comment 4•20 years ago
|
||
Comment on attachment 155097 [details] [diff] [review] Patch 2 That's the idea, although you've chose an odd place to put your var column;
Comment 5•20 years ago
|
||
someone told me, it looks better this way :)
Attachment #155097 -
Attachment is obsolete: true
Updated•20 years ago
|
Attachment #155104 -
Flags: superreview?(neil.parkwaycc.co.uk)
Attachment #155104 -
Flags: review?(timeless)
Updated•20 years ago
|
Attachment #155104 -
Flags: superreview?(neil.parkwaycc.co.uk) → superreview+
Attachment #155104 -
Flags: review?(timeless) → review+
Comment 6•20 years ago
|
||
checked in Checking in tree.xml; /cvsroot/mozilla/xpfe/global/resources/content/bindings/tree.xml,v <-- tree.xml new revision: 1.41; previous revision: 1.40 done
Assignee | ||
Comment 8•20 years ago
|
||
What about <mozilla/toolkit/content/widgets/tree.xml> ?? It need the same fix !
Severity: normal → minor
Target Milestone: --- → mozilla1.8alpha3
Version: Other Branch → Trunk
Updated•20 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 9•20 years ago
|
||
It seems you just volunteered.
Assignee: mcsmurf → gautheri
Status: REOPENED → NEW
Assignee | ||
Updated•20 years ago
|
Attachment #155104 -
Attachment description: Last attempt ;-) → Last attempt ;-) (/Xpfe part)
[Checked in: Comment 6]
Attachment #155104 -
Attachment is obsolete: true
Assignee | ||
Comment 10•20 years ago
|
||
(In reply to comment #9) > It seems you just volunteered. Actually, it's already included in my '(Bv1a) <tree.xml> (/Toolkit part)' patch for bug 211046 ... which I'm only missing an sr+ for checkin... ;->
Depends on: 211046
Component: XP Toolkit/Widgets: Trees → XUL
QA Contact: xptoolkit.widgets
Comment 11•16 years ago
|
||
Fixed long ago
Status: NEW → RESOLVED
Closed: 20 years ago → 16 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•