Re: T5 Error in 5.0.5 palette.js

2007-10-04 Thread Joel Wiegman
>> }.bind(this)); >> if (candidate) before = candidate; >> } >> >> to.add(option, before); >> }, >> >> Is the T5 Palette working fine for anyone using IE? >> The IE Version I am testing is >> 6.0.2900.2180.xpsp_sp2_gdr.070

Re: T5 Error in 5.0.5 palette.js

2007-09-12 Thread Nick Westgate
There is a JIRA: https://issues.apache.org/jira/browse/TAPESTRY-1745 Cheers, Nick. Howard Lewis Ship wrote: Please add this as an issue in JIRA if you have not done so already. On 8/28/07, Daniel Jue <[EMAIL PROTECTED]> wrote: When using the Palette component, I am getting a Javascript error

Re: T5 Error in 5.0.5 palette.js

2007-09-12 Thread Daniel Jue
I have not added a JIRA, but here is the EnhancedPalette I whipped up using other people's code. =) http://wiki.apache.org/tapestry/Tapestry5EnhancedPalette?action=show Daniel Jue On 9/12/07, Howard Lewis Ship <[EMAIL PROTECTED]> wrote: > Please add this as an issue in JIRA if you have not done

Re: T5 Error in 5.0.5 palette.js

2007-09-12 Thread Howard Lewis Ship
Please add this as an issue in JIRA if you have not done so already. On 8/28/07, Daniel Jue <[EMAIL PROTECTED]> wrote: > > When using the Palette component, I am getting a Javascript error in IE, > > Line: 183 > Char: 1 > Error: Expected identifier, string or number > Code: 0 > > >

Re: T5 Error in 5.0.5 palette.js

2007-09-11 Thread Joel Wiegman
(candidate) before = candidate; > } > > to.add(option, before); > }, > > Is the T5 Palette working fine for anyone using IE? > The IE Version I am testing is > 6.0.2900.2180.xpsp_sp2_gdr.070227-2254 > > > Everything works fine in Firefox, of course. >

Re: T5 Error in 5.0.5 palette.js

2007-08-28 Thread Martin Reurings
Actually throwing an onclick event (which is what I assume is being used) on a disabled item isn't forbidden by specification. But, it shouldn't be the cause of Opera's unwillingness to function. In the updateButtons method the disabled state of the buttons should be update in accordance to th

Re: T5 Error in 5.0.5 palette.js

2007-08-28 Thread Thiago H. de Paula Figueiredo
Em Tue, 28 Aug 2007 20:51:30 -0300, Martin Reurings <[EMAIL PROTECTED]> escreveu: Probably the best way to get some clarity is by adding a few alerts in some key places and compare the results of those alert between FF and IE. Just because it works in FF doesn't mean it's correct BTW. T5'

Re: T5 Error in 5.0.5 palette.js

2007-08-28 Thread Martin Reurings
I suppose I'd have to make the time to test through the Palette component to figure this one out. The syntax appears to be correct (apart from the trailing comma you already identified) and the lack of comments obscures a lot of what's going on. There's some (to me) really obscure eventhandlin

Re: T5 Error in 5.0.5 palette.js

2007-08-28 Thread Daniel Jue
I am using JS Eclipse, however I am no javascript wizard. :-) JSEclipse didn't complaint about the extra comma! I just had a look at the T4 pallet js (from the apache trunk) and it does not have a comma after the end of the function listing. However, the T4 version is way different and uses some

Re: T5 Error in 5.0.5 palette.js

2007-08-28 Thread Martin Reurings
Without having a look at the rest of the code I can tell you without a doubt you are on the right track there. When using Object Notation Firefox has some weird parsing error which allows it to successfully parse an object that has the illegal syntax you just highlighted. Make no mistake though,

T5 Error in 5.0.5 palette.js

2007-08-28 Thread Daniel Jue
When using the Palette component, I am getting a Javascript error in IE, Line: 183 Char: 1 Error: Expected identifier, string or number Code: 0 reorderSelected : function(movers, before) { movers.each (function(option) { this.selected.add(option, before); }.bind(this)); t