Opened 14 years ago

Closed 13 years ago

Last modified 13 years ago

#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)

Part_alpha.zip (5.1 KB) - added by Morris Johns 14 years ago.
Part code and test files
test_Part.html (2.9 KB) - added by Morris Johns 14 years ago.
The test_Part.html file in the zip file is just plain WRONG!
Part.js (5.3 KB) - added by jeremiah.morrill@… 14 years ago.
With dojoAttachEvent working - not fully tested
Part.2.js (3.7 KB) - added by morris 13 years ago.
Using defineWidget()

Download all attachments as: .zip

Change History (12)

Changed 14 years ago by Morris Johns

Attachment: Part_alpha.zip added

Part code and test files

comment:1 Changed 14 years ago by alex

Milestone: 0.3release

Changed 14 years ago by Morris Johns

Attachment: test_Part.html added

The test_Part.html file in the zip file is just plain WRONG!

comment:2 Changed 14 years ago by anonymous

Owner: changed from anonymous to morris

comment:3 Changed 14 years ago by Dustin Machi

Milestone: 0.3release0.4

comment:4 Changed 14 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 14 years ago by jeremiah.morrill@…

Attachment: Part.js added

With dojoAttachEvent working - not fully tested

Changed 13 years ago by morris

Attachment: Part.2.js added

Using defineWidget()

comment:5 Changed 13 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 13 years ago by dylan

Summary: Part widget suggestion[patch] Part widget suggestion

comment:7 Changed 13 years ago by bill

Resolution: fixed
Status: newclosed

Hi everyone. Liucougar basically already checked this into DomWidget?.js, in checkin #5401.

comment:8 Changed 13 years ago by (none)

Milestone: 0.4

Milestone 0.4 deleted

Note: See TracTickets for help on using tickets.