Ticket #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
Change History
Changed 6 years ago by Morris Johns
-
attachment
Part_alpha.zip
added
Changed 6 years ago by Morris Johns
-
attachment
test_Part.html
added
The test_Part.html file in the zip file is just plain WRONG!
comment:4 Changed 6 years ago by jeremiah.morrill@…
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.
Changed 6 years ago by jeremiah.morrill@…
-
attachment
Part.js
added
With dojoAttachEvent working - not fully tested
comment:5 Changed 6 years ago by morris the cat
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 6 years ago by dylan
- Summary changed from Part widget suggestion to [patch] Part widget suggestion
comment:7 Changed 6 years ago by bill
- Status changed from new to closed
- Resolution set to fixed
Hi everyone. Liucougar basically already checked this into DomWidget?.js, in checkin #5401.
Part code and test files