#566 closed enhancement (fixed)
[patch] Part widget suggestion
Reported by: | Morris Johns | Owned by: | morris |
---|---|---|---|
Priority: | high | Milestone: | |
Component: | Widgets | Version: | 0.3 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
Suggestion that we add a Part widget to the dojo widgets. See contributors mailing list for comments.
Attachments (4)
Change History (12)
Changed 15 years ago by
Attachment: | Part_alpha.zip added |
---|
comment:1 Changed 15 years ago by
Milestone: | → 0.3release |
---|
Changed 15 years ago by
Attachment: | test_Part.html added |
---|
The test_Part.html file in the zip file is just plain WRONG!
comment:2 Changed 15 years ago by
Owner: | changed from anonymous to morris |
---|
comment:3 Changed 15 years ago by
Milestone: | 0.3release → 0.4 |
---|
comment:4 Changed 15 years ago by
I posted a fix to the dojoAttachEvent to this message board url http://forums.opensymphony.com/thread.jspa?threadID=26595&tstart=0
I only been muckin around with dojo for a week, but thought this widget was very important to me.
So be kind with the quality of the code =P
BTW I could not get it to attach events to any nested turbo widgets (ie toolbar container with toolbar buttons) because the turbo widgets did not return any children even though they existed.
comment:5 Changed 15 years ago by
Part.2.js is the same as Part.js, but changed to use the defineWidget() call (compatible with changes for version 0.4). Jerimiah's code not included because I haven't tried it, and the first line he added didn't make sense to me. Put it in your dojo/src/widget directory and rename it to Part.js
Notes:
- I am using this widget extensively, including some complex templates containing many widgets and I have had no problems with the code - YMMV.
- I should update the example, but I had problems with the namespace changes so I haven't.
- At some point I need to merge this up into the DomWidget? code. I have not done so mostly because I am a scardy cat. (and also I need to change the code to use the identifiers 'dojoAttachPoint' and 'dojoAttachEvent').
comment:6 Changed 15 years ago by
Summary: | Part widget suggestion → [patch] Part widget suggestion |
---|
comment:7 Changed 15 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Hi everyone. Liucougar basically already checked this into DomWidget?.js, in checkin #5401.
Part code and test files