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: [email protected]
> > > 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
> > > >
> > > >
> > >
> > >
> > >
> >
> >
>
>
