[PORTAL] Adamns inline questions

2006-12-27 Thread Scott O'Bryan
Hey Adam, In your last checkin to my branch you made some comments I would like to address. In the DispatchResponseConfiguratorImpl there is an isApplied function. You were asking why it was there. The reason for this is simple.. Backward compatibility. You mentioned to me in some

[Skinning Enhancement] -tr-icon-class or equivalent

2006-12-27 Thread Jeanne Waldman
Hi there, I see the need to render img class=someClass width=10 height=8/ when we render a skinning icon. However, there is currently not a way to specify a styleclass in the skin css file, although we do have a Java API to do this. We could do something like: af|foo::some-icon {

Re: [PORTAL] Adamns inline questions

2006-12-27 Thread Adam Winer
-- Adam On 12/27/06, Scott O'Bryan [EMAIL PROTECTED] wrote: Hey Adam, In your last checkin to my branch you made some comments I would like to address. In the DispatchResponseConfiguratorImpl there is an isApplied function. You were asking why it was there. The reason for this is simple..

Re: [Skinning Enhancement] -tr-icon-class or equivalent

2006-12-27 Thread Adam Winer
I think, for now, that I'm OK with just requiring inline specification of the styles - that feels a lot more natural in a CSS file. At least, as long as it works. :) -- Adam On 12/27/06, Jeanne Waldman [EMAIL PROTECTED] wrote: Hi there, I see the need to render img class=someClass width=10