From: Michael P. <mp...@ph...> - 2001-02-21 11:21:08
Attachments:
events.zip
|
I've recently finished splitting the events code into separate files. It removes the mouse and drag events code from the dynapi.api section. I've placed the drag / dragdrop / mouse / key code all in their own dynapi.events section. I found that there was a slight modification to the dynlayer required. It was posted a few days ago. If anyone finds any problems with it, please post your comments. I've been working with the new setup for the last week but may have missed a few bugs. I've also included a tweaked version of Henrik Våglin's keyevents code. Thanks for that goes to him. -- Michael Pemberton mp...@ph... ICQ: 12107010 |
From: Jordi - I. - M. <jmi...@or...> - 2001-02-21 11:35:03
|
This is one thing to be taken into account now that we're planning a code clean-up Michael Pemberton wrote: > I've recently finished splitting the events code into separate files. > > It removes the mouse and drag events code from the dynapi.api section. > > I've placed the drag / dragdrop / mouse / key code all in their own > dynapi.events section. > > I found that there was a slight modification to the dynlayer required. > It was posted a few days ago. > > If anyone finds any problems with it, please post your comments. I've > been working with the new setup for the last week but may have missed a > few bugs. > > I've also included a tweaked version of Henrik Våglin's keyevents code. > Thanks for that goes to him. > -- > Michael Pemberton > mp...@ph... > ICQ: 12107010 > > ------------------------------------------------------------------------ > Name: events.zip > events.zip Type: Zip Compressed Data (application/x-zip-compressed) > Encoding: base64 |
From: Michael P. <mp...@ph...> - 2001-02-21 11:48:43
|
it has been mentioned before. I'm just surprised no one did it earlier. pascal must be getting behind in his work : ) Jordi - IlMaestro - Ministral wrote: > This is one thing to be taken into account now that we're planning a code > clean-up > > Michael Pemberton wrote: > > > I've recently finished splitting the events code into separate files. > > > > It removes the mouse and drag events code from the dynapi.api section. > > > > I've placed the drag / dragdrop / mouse / key code all in their own > > dynapi.events section. > > > > I found that there was a slight modification to the dynlayer required. > > It was posted a few days ago. > > > > If anyone finds any problems with it, please post your comments. I've > > been working with the new setup for the last week but may have missed a > > few bugs. > > > > I've also included a tweaked version of Henrik Våglin's keyevents code. > > Thanks for that goes to him. > > -- > > Michael Pemberton > > mp...@ph... > > ICQ: 12107010 > > > > ------------------------------------------------------------------------ > > Name: events.zip > > events.zip Type: Zip Compressed Data (application/x-zip-compressed) > > Encoding: base64 > > _______________________________________________ > Dynapi-Dev mailing list > Dyn...@li... > http://lists.sourceforge.net/lists/listinfo/dynapi-dev -- Michael Pemberton mp...@ph... ICQ: 12107010 |
From: <ma...@ab...> - 2001-02-21 12:48:10
Attachments:
require.zip
|
i've made a quick and dirty .require() method written to use in widgets to make sure the right files are included. it was first meant to call DynAPI.include() for each file missed but this didn't really work so i decided just to show an alert telling which files. take a look in button.js for example how to use the require() method. i've made some minor changes in include() and added a small _include() method to get everything to work i've also added a DynAPI.alert() method which by default just shows an alert but can be overwritten for debugging-issues. this is better than common alert()s popuping up here and where in DynAPI-code. I know this maybe isn't exacly what we need talking optimizing, speed and size but i thought it could be usefull, please take a look. (as far as i know, a require() method was discussed a couple of months ago?) as i said, the code i quick written, just wanted to show my ideas (maybe this code could be an extension instead of core api, and now i noticed it has some bugs which order the files are included in netscape. well..) cheers /martin |
From: Michael P. <mp...@ph...> - 2001-02-21 13:18:19
|
there is a DynAPI.errorHandler method that is used for debugging purposes. I suggest that you use this method for all API related alerts. This allows for the error to be hidden from the user and logged for later processing. martin ström wrote: > i've made a quick and dirty .require() method written > to use in widgets to make sure the right files are included. > it was first meant to call DynAPI.include() for each file > missed but this didn't really work so i decided just to show > an alert telling which files. > > take a look in button.js for example how to use the > require() method. > > i've made some minor changes in include() and added > a small _include() method to get everything to work > > i've also added a DynAPI.alert() method which by > default just shows an alert but can be overwritten > for debugging-issues. > this is better than common alert()s popuping up here and > where in DynAPI-code. > > I know this maybe isn't exacly what we need talking > optimizing, speed and size but i thought it could be usefull, please > take a look. (as far as i know, a require() method was discussed > a couple of months ago?) > > as i said, the code i quick written, just wanted to show my ideas > > (maybe this code could be an extension instead of core api, > and now i noticed it has some bugs which order the files are > included in netscape. well..) > > cheers > /martin > > ------------------------------------------------------------------------ > Name: require.zip > require.zip Type: Zip Compressed Data (application/x-zip-compressed) > Encoding: base64 -- Michael Pemberton mp...@ph... ICQ: 12107010 |
From: <ma...@ab...> - 2001-02-21 13:29:14
|
yes, i know, i thought the DynAPI.alert could be used instead of alerts in for example addLibrary() and when debugging, forward this alert to a console instead. instead of getting a alert("Incorrect DynAPI.addLibrary usage:\n\nExample: DynAPI.addLibrary('dynapi.ext',['inline.js'])"); this string could be shown i a debugging-console just to get everything at one place. /martin > -----Original Message----- > From: dyn...@li... > [mailto:dyn...@li...]On Behalf Of Michael > Pemberton > Sent: den 21 februari 2001 14:09 > To: dyn...@li... > Subject: Re: [Dynapi-Dev] DynAPI.require() method > > > there is a DynAPI.errorHandler method that is used for debugging > purposes. I > suggest that you use this method for all API related alerts. > This allows for > the error to be hidden from the user and logged for later processing. > > martin ström wrote: > > > i've made a quick and dirty .require() method written > > to use in widgets to make sure the right files are included. > > it was first meant to call DynAPI.include() for each file > > missed but this didn't really work so i decided just to show > > an alert telling which files. > > > > take a look in button.js for example how to use the > > require() method. > > > > i've made some minor changes in include() and added > > a small _include() method to get everything to work > > > > i've also added a DynAPI.alert() method which by > > default just shows an alert but can be overwritten > > for debugging-issues. > > this is better than common alert()s popuping up here and > > where in DynAPI-code. > > > > I know this maybe isn't exacly what we need talking > > optimizing, speed and size but i thought it could be usefull, please > > take a look. (as far as i know, a require() method was discussed > > a couple of months ago?) > > > > as i said, the code i quick written, just wanted to show my ideas > > > > (maybe this code could be an extension instead of core api, > > and now i noticed it has some bugs which order the files are > > included in netscape. well..) > > > > cheers > > /martin > > > > > ------------------------------------------------------------------------ > > Name: require.zip > > require.zip Type: Zip Compressed Data > (application/x-zip-compressed) > > Encoding: base64 > > -- > Michael Pemberton > mp...@ph... > ICQ: 12107010 > > > > > _______________________________________________ > Dynapi-Dev mailing list > Dyn...@li... > http://lists.sourceforge.net/lists/listinfo/dynapi-dev |
From: Michael P. <mp...@ph...> - 2001-02-21 13:50:52
|
check for my earlier post reagrding this. I included code that would collect the errorHandler errors and display them in a separate window when the page has finsihed loading. martin ström wrote: > yes, i know, i thought the DynAPI.alert could be used > instead of alerts in for example addLibrary() and > when debugging, forward this alert to a console instead. > > instead of getting a > alert("Incorrect DynAPI.addLibrary usage:\n\nExample: > DynAPI.addLibrary('dynapi.ext',['inline.js'])"); > > this string could be shown i a debugging-console > just to get everything at one place. > > /martin > > > -----Original Message----- > > From: dyn...@li... > > [mailto:dyn...@li...]On Behalf Of Michael > > Pemberton > > Sent: den 21 februari 2001 14:09 > > To: dyn...@li... > > Subject: Re: [Dynapi-Dev] DynAPI.require() method > > > > > > there is a DynAPI.errorHandler method that is used for debugging > > purposes. I > > suggest that you use this method for all API related alerts. > > This allows for > > the error to be hidden from the user and logged for later processing. > > > > martin ström wrote: > > > > > i've made a quick and dirty .require() method written > > > to use in widgets to make sure the right files are included. > > > it was first meant to call DynAPI.include() for each file > > > missed but this didn't really work so i decided just to show > > > an alert telling which files. > > > > > > take a look in button.js for example how to use the > > > require() method. > > > > > > i've made some minor changes in include() and added > > > a small _include() method to get everything to work > > > > > > i've also added a DynAPI.alert() method which by > > > default just shows an alert but can be overwritten > > > for debugging-issues. > > > this is better than common alert()s popuping up here and > > > where in DynAPI-code. > > > > > > I know this maybe isn't exacly what we need talking > > > optimizing, speed and size but i thought it could be usefull, please > > > take a look. (as far as i know, a require() method was discussed > > > a couple of months ago?) > > > > > > as i said, the code i quick written, just wanted to show my ideas > > > > > > (maybe this code could be an extension instead of core api, > > > and now i noticed it has some bugs which order the files are > > > included in netscape. well..) > > > > > > cheers > > > /martin > > > > > > > > ------------------------------------------------------------------------ > > > Name: require.zip > > > require.zip Type: Zip Compressed Data > > (application/x-zip-compressed) > > > Encoding: base64 > > > > -- > > Michael Pemberton > > mp...@ph... > > ICQ: 12107010 > > > > > > > > > > _______________________________________________ > > Dynapi-Dev mailing list > > Dyn...@li... > > http://lists.sourceforge.net/lists/listinfo/dynapi-dev > > _______________________________________________ > Dynapi-Dev mailing list > Dyn...@li... > http://lists.sourceforge.net/lists/listinfo/dynapi-dev -- Michael Pemberton mp...@ph... ICQ: 12107010 |
From: Doug M. <do...@cr...> - 2001-02-21 21:09:11
|
Hw about adding a switch? I had to comment out the errorhandler in my copy as it did not work in IE. It would be nice to be able to turn this on and off. Rather than haveing to comment the damned thing out every time I update my copy if the API ----- Original Message ----- From: "Michael Pemberton" <mp...@ph...> To: <dyn...@li...> Sent: Wednesday, February 21, 2001 5:09 AM Subject: Re: [Dynapi-Dev] DynAPI.require() method > there is a DynAPI.errorHandler method that is used for debugging purposes. I > suggest that you use this method for all API related alerts. This allows for > the error to be hidden from the user and logged for later processing. > > martin ström wrote: > > > i've made a quick and dirty .require() method written > > to use in widgets to make sure the right files are included. > > it was first meant to call DynAPI.include() for each file > > missed but this didn't really work so i decided just to show > > an alert telling which files. > > > > take a look in button.js for example how to use the > > require() method. > > > > i've made some minor changes in include() and added > > a small _include() method to get everything to work > > > > i've also added a DynAPI.alert() method which by > > default just shows an alert but can be overwritten > > for debugging-issues. > > this is better than common alert()s popuping up here and > > where in DynAPI-code. > > > > I know this maybe isn't exacly what we need talking > > optimizing, speed and size but i thought it could be usefull, please > > take a look. (as far as i know, a require() method was discussed > > a couple of months ago?) > > > > as i said, the code i quick written, just wanted to show my ideas > > > > (maybe this code could be an extension instead of core api, > > and now i noticed it has some bugs which order the files are > > included in netscape. well..) > > > > cheers > > /martin > > > ------------------------------------------------------------------------ > > Name: require.zip > > require.zip Type: Zip Compressed Data (application/x-zip-compressed) > > Encoding: base64 > > -- > Michael Pemberton > mp...@ph... > ICQ: 12107010 > > > > > _______________________________________________ > Dynapi-Dev mailing list > Dyn...@li... > http://lists.sourceforge.net/lists/listinfo/dynapi-dev --- Outgoing mail is certified Virus Free by AVG Free Edition Download at: http://www.grisoft.com/html/us_index.cfm Checked by AVG anti-virus system (http://www.grisoft.com). Version: 6.0.231 / Virus Database: 112 - Release Date: 2/12/01 |
From: Pascal B. <pa...@dy...> - 2001-02-21 21:24:28
|
think I already added the code from michael to CVS (unless the update didn't work :) Pascal Bestebroer pa...@dy... http://www.dynamic-core.net > -----Oorspronkelijk bericht----- > Van: dyn...@li... > [mailto:dyn...@li...]Namens Doug Melvin > Verzonden: donderdag 22 februari 2001 1:08 > Aan: dyn...@li... > Onderwerp: Re: [Dynapi-Dev] DynAPI.require() method > > > Hw about adding a switch? > I had to comment out the errorhandler in my copy as it did not work in IE. > It would be nice to be able to turn this on and off. > Rather than haveing to comment the damned thing out every time I update my > copy if the API > ----- Original Message ----- > From: "Michael Pemberton" <mp...@ph...> > To: <dyn...@li...> > Sent: Wednesday, February 21, 2001 5:09 AM > Subject: Re: [Dynapi-Dev] DynAPI.require() method > > > > there is a DynAPI.errorHandler method that is used for > debugging purposes. > I > > suggest that you use this method for all API related alerts. > This allows > for > > the error to be hidden from the user and logged for later processing. > > > > martin ström wrote: > > > > > i've made a quick and dirty .require() method written > > > to use in widgets to make sure the right files are included. > > > it was first meant to call DynAPI.include() for each file > > > missed but this didn't really work so i decided just to show > > > an alert telling which files. > > > > > > take a look in button.js for example how to use the > > > require() method. > > > > > > i've made some minor changes in include() and added > > > a small _include() method to get everything to work > > > > > > i've also added a DynAPI.alert() method which by > > > default just shows an alert but can be overwritten > > > for debugging-issues. > > > this is better than common alert()s popuping up here and > > > where in DynAPI-code. > > > > > > I know this maybe isn't exacly what we need talking > > > optimizing, speed and size but i thought it could be usefull, please > > > take a look. (as far as i know, a require() method was discussed > > > a couple of months ago?) > > > > > > as i said, the code i quick written, just wanted to show my ideas > > > > > > (maybe this code could be an extension instead of core api, > > > and now i noticed it has some bugs which order the files are > > > included in netscape. well..) > > > > > > cheers > > > /martin > > > > > > ------------------------------------------------------------------------ > > > Name: require.zip > > > require.zip Type: Zip Compressed Data > (application/x-zip-compressed) > > > Encoding: base64 > > > > -- > > Michael Pemberton > > mp...@ph... > > ICQ: 12107010 > > > > > > > > > > _______________________________________________ > > Dynapi-Dev mailing list > > Dyn...@li... > > http://lists.sourceforge.net/lists/listinfo/dynapi-dev > > > --- > Outgoing mail is certified Virus Free by AVG Free Edition > Download at: http://www.grisoft.com/html/us_index.cfm > Checked by AVG anti-virus system (http://www.grisoft.com). > Version: 6.0.231 / Virus Database: 112 - Release Date: 2/12/01 > > > _______________________________________________ > Dynapi-Dev mailing list > Dyn...@li... > http://lists.sourceforge.net/lists/listinfo/dynapi-dev > |
From: Doug M. <do...@cr...> - 2001-02-21 21:05:51
|
when can we see this and all of the other enhancements in the CVS snapshot? I am getting sick of patching my copy of DynAPI three times a week. We REALLY need to develop some sort of benchmarking standard so we can determine if a change is beneficial, and worth adding to the production copy of DynAPI. ----- Original Message ----- From: "Michael Pemberton" <mp...@ph...> To: <dyn...@li...> Sent: Wednesday, February 21, 2001 3:18 AM Subject: [Dynapi-Dev] Event Code Separation > I've recently finished splitting the events code into separate files. > > It removes the mouse and drag events code from the dynapi.api section. > > I've placed the drag / dragdrop / mouse / key code all in their own > dynapi.events section. > > I found that there was a slight modification to the dynlayer required. > It was posted a few days ago. > > If anyone finds any problems with it, please post your comments. I've > been working with the new setup for the last week but may have missed a > few bugs. > > I've also included a tweaked version of Henrik Våglin's keyevents code. > Thanks for that goes to him. > -- > Michael Pemberton > mp...@ph... > ICQ: 12107010 > --- Outgoing mail is certified Virus Free by AVG Free Edition Download at: http://www.grisoft.com/html/us_index.cfm Checked by AVG anti-virus system (http://www.grisoft.com). Version: 6.0.231 / Virus Database: 112 - Release Date: 2/12/01 |
From: Michael P. <mp...@ph...> - 2001-02-22 03:12:49
|
I'm not sure about CVS but the code I attached is THE SAME as we already have. It is simply split into separate files. there should be NO speed difference other than the browser having to download more small files intead of less big ones. Doug Melvin wrote: > when can we see this and all of the other enhancements in the CVS snapshot? > I am getting sick of patching my copy of DynAPI three times a week. > > We REALLY need to develop some sort of benchmarking standard so we can > determine if a change is beneficial, and worth adding to the production copy > of DynAPI. > ----- Original Message ----- > From: "Michael Pemberton" <mp...@ph...> > To: <dyn...@li...> > Sent: Wednesday, February 21, 2001 3:18 AM > Subject: [Dynapi-Dev] Event Code Separation > > > I've recently finished splitting the events code into separate files. > > > > It removes the mouse and drag events code from the dynapi.api section. > > > > I've placed the drag / dragdrop / mouse / key code all in their own > > dynapi.events section. > > > > I found that there was a slight modification to the dynlayer required. > > It was posted a few days ago. > > > > If anyone finds any problems with it, please post your comments. I've > > been working with the new setup for the last week but may have missed a > > few bugs. > > > > I've also included a tweaked version of Henrik Våglin's keyevents code. > > Thanks for that goes to him. > > -- > > Michael Pemberton > > mp...@ph... > > ICQ: 12107010 > > > > --- > Outgoing mail is certified Virus Free by AVG Free Edition > Download at: http://www.grisoft.com/html/us_index.cfm > Checked by AVG anti-virus system (http://www.grisoft.com). > Version: 6.0.231 / Virus Database: 112 - Release Date: 2/12/01 > > _______________________________________________ > Dynapi-Dev mailing list > Dyn...@li... > http://lists.sourceforge.net/lists/listinfo/dynapi-dev -- Michael Pemberton mp...@ph... ICQ: 12107010 |
From: Eytan H. <ey...@tr...> - 2001-02-25 08:42:32
|
Which makes it slightly slower (but I'm not compaining) 8an |