Ticket #566 (closed enhancement: fixed)

Opened 3 years ago

Last modified 18 months ago

[patch] Part widget suggestion

Reported by: Morris Johns Owned by: morris
Priority: normal Milestone:
Component: Widgets Version: 0.3
Severity: normal Keywords:
Cc:

Description

Suggestion that we add a Part widget to the dojo widgets. See contributors mailing list for comments.

Attachments

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

Change History

Changed 3 years ago by Morris Johns

Part code and test files

Changed 3 years ago by alex

  • milestone set to 0.3release

Changed 3 years ago by Morris Johns

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

Changed 3 years ago by anonymous

  • owner changed from anonymous to morris

Changed 3 years ago by dmachi

  • milestone changed from 0.3release to 0.4

Changed 3 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 3 years ago by jeremiah.morrill@…

With dojoAttachEvent working - not fully tested

Changed 2 years ago by morris

Using defineWidget()

Changed 2 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').

Changed 2 years ago by dylan

  • summary changed from Part widget suggestion to [patch] Part widget suggestion

Changed 2 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.

Changed 18 months ago by anonymous

  • milestone deleted

Milestone 0.4 deleted

Note: See TracTickets for help on using tickets.