Re: Process train and IE

Wed, 27 Sep 2006 13:13:11 -0700 

Simon,
Your workaround is fine. Pavitra submitted a bug internally for this, but there should be an JIRA issue created if it hasn't been already, then such hacks can refer to the issue number. I'm planning to start on the discussion/resolution of this bug very soon, cuz we are using composite selectors all over the place for our internal renderers, so we'll be hearing about it if we don't fix this soon. 

Thanks,
Jeanne

Simon Lessard wrote:


A browser sepcific key would not help in this case as the problem come 
from

the evaluation of a state (autogenerated from :<state> structure) style
class.


Regards,

~ Simon

On 9/27/06, Scott O'Bryan <[EMAIL PROTECTED]> wrote:



Can't we add a browser specific skinning key for this?

Scott

Arjuna Wijeyekoon wrote:
> I think this is fine.
> +1
>
> one thing is to check to see if there already is a utility method to
> figure
> out if the agent is IE.
> --arjuna
>
> On 9/27/06, Simon Lessard <[EMAIL PROTECTED]> wrote:
>>
>> My patch is extremely specific to process train, so you won't be able
to

>> apply it directly in any other problem. The main idea behind it is 
that

>> the
>> composite selectors cause problems for joins on the content row
>> mainly and

>> that joins are background images 99% of the times. So, for IE I 
add the

>> following to the content row:
>>
>>     /* -= Simon =-
>>      * FIXME HACK for MSIE CSS bug involving composite style classes.

>>      *       Since the bug is most obvious with join background 
images,

>>      *       I hard code them to none
>>      */

>>     
if(arc.getAgent().getAgentName().equalsIgnoreCase(Agent.AGENT_IE))

>>     {
>>       writer.writeAttribute(XhtmlConstants.STYLE_ATTRIBUTE,
>>                             "background-image:none;",
>>                             null);
>>     }
>>

>> Note that I didn't check IE version, I just assume that IE never 
works.

>>
>>
>> Regards,
>>
>> ~Simon
>>
>> On 9/26/06, Arjuna Wijeyekoon <[EMAIL PROTECTED]> wrote:
>> >
>> > thanks Pavitra,
>> > Simon, can you post an example of what your "patch" markup looks
like?
>> >
>> > On 9/26/06, Pavitra Subramaniam <[EMAIL PROTECTED]>
>> wrote:
>> > >
>> > > This is certainly on IE6. But I haven't tried IE7. Here is an
>> example
>> > >
>> > >   <html>
>> > >   <head>
>> > >     <title>train Demo</title>
>> > >     <style type="text/css">
>> > >       .af_train_stop-link.p_AFVisited{text-decoration:underline
>> > > ;color:red;background-color:transparent;}
>> > >       .af_train_stop-spacer.p_AFVisited{background-color:red}
>> > >     </style>
>> > >   </head>
>> > >
>> > >   <body class="">
>> > >     <table cellpadding="0" cellspacing="0" border="0" summary=""
>> > > class="af_train">
>> > >       <tbody>
>> > >         <tr>
>> > >           <td><div class="af_train_stop-spacer
>> p_AFVisited"></div></td>
>> > >           <td></td>
>> > >         </tr>
>> > >         <tr>
>> > >           <td><div class="af_train_stop-content ">
>> > >             <a class="af_train_stop-link p_AFVisited"
>> onclick="return
>> > > false;">First Step</a></div></td>
>> > >           <td></td>
>> > >         </tr>
>> > >       </tbody>
>> > >     </table>
>> > >    </body>
>> > > </html>
>> > >
>> > > - Pavitra
>> > >
>> > >
>> > > > -----Original Message-----
>> > > > From: Arjuna Wijeyekoon [mailto:[EMAIL PROTECTED]
>> > > > Sent: Sunday, September 24, 2006 1:30 PM
>> > > > To: adffaces-dev@incubator.apache.org
>> > > > Subject: Re: Process train and IE
>> > > >
>> > > > is this IE 6 or IE 7?
>> > > >
>> > > > On 9/23/06, Simon Lessard <[EMAIL PROTECTED]> wrote:
>> > > > >
>> > > > > Hello all,
>> > > > >
>> > > > > As you might know, IE has some problems with composite
selectors
>> > > > > (.something.somethingElse for example, but more commonly
>> > > > > .af_train_stop.p_AFVisited). With process train the most
>> important
>> > > > > issue it yield is showing the stop join on the content row
>> > > > because of
>> > > > > how IE evaluate those selectors. I was able to "patch" it
>> > > > writing an
>> > > > > hard coded style on the content row, is that something
>> acceptable?.
>> > > > >
>> > > > >
>> > > > > Regards,
>> > > > >
>> > > > > ~ Simon
>> > > > >
>> > > > >
>> > > >
>> > > >
>> > > >
>> > >
>> > >
>> >
>> >
>>
>>
>




























					Reply via email to