You are viewing a plain text version of this content. The canonical link for it is here.
Posted to adffaces-dev@incubator.apache.org by Simon Lessard <si...@gmail.com> on 2006/09/23 20:25:47 UTC
Process train and IE
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
Re: Process train and IE
Posted by Simon Lessard <si...@gmail.com>.
IE 6, IE 7 is *supposed* to have fixed it.
On 9/24/06, Arjuna Wijeyekoon <ar...@gmail.com> wrote:
>
> is this IE 6 or IE 7?
>
> On 9/23/06, Simon Lessard <si...@gmail.com> 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
> >
> >
>
>
Re: Process train and IE
Posted by Jeanne Waldman <je...@oracle.com>.
Thanks Simon!!!! I really appreciate all your help with the skinning stuff.
Jeanne
Simon Lessard wrote:
> Alright,
>
> Thank Jeanne. I don't think a JIRA issue was opened yet, I'll do that
> tonight and change the reference in me FIXME to that issue.
>
>
> Regards,
>
> ~ Simon
>
> On 9/27/06, Jeanne Waldman <je...@oracle.com> wrote:
>
>>
>> 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 <da...@gmail.com> 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 <si...@gmail.com> 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 <ar...@gmail.com> wrote:
>> >> >> >
>> >> >> > thanks Pavitra,
>> >> >> > Simon, can you post an example of what your "patch" markup looks
>> >> like?
>> >> >> >
>> >> >> > On 9/26/06, Pavitra Subramaniam <pa...@oracle.com>
>> >> >> 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:arjuna@gmail.com]
>> >> >> > > > 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 <si...@gmail.com> 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
>> >> >> > > > >
>> >> >> > > > >
>> >> >> > > >
>> >> >> > > >
>> >> >> > > >
>> >> >> > >
>> >> >> > >
>> >> >> >
>> >> >> >
>> >> >>
>> >> >>
>> >> >
>> >>
>> >>
>> >
>>
>>
>
Re: Process train and IE
Posted by Simon Lessard <si...@gmail.com>.
Alright,
Thank Jeanne. I don't think a JIRA issue was opened yet, I'll do that
tonight and change the reference in me FIXME to that issue.
Regards,
~ Simon
On 9/27/06, Jeanne Waldman <je...@oracle.com> wrote:
>
> 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 <da...@gmail.com> 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 <si...@gmail.com> 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 <ar...@gmail.com> wrote:
> >> >> >
> >> >> > thanks Pavitra,
> >> >> > Simon, can you post an example of what your "patch" markup looks
> >> like?
> >> >> >
> >> >> > On 9/26/06, Pavitra Subramaniam <pa...@oracle.com>
> >> >> 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:arjuna@gmail.com]
> >> >> > > > 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 <si...@gmail.com> 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
> >> >> > > > >
> >> >> > > > >
> >> >> > > >
> >> >> > > >
> >> >> > > >
> >> >> > >
> >> >> > >
> >> >> >
> >> >> >
> >> >>
> >> >>
> >> >
> >>
> >>
> >
>
>
Re: Process train and IE
Posted by Jeanne Waldman <je...@oracle.com>.
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 <da...@gmail.com> 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 <si...@gmail.com> 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 <ar...@gmail.com> wrote:
>> >> >
>> >> > thanks Pavitra,
>> >> > Simon, can you post an example of what your "patch" markup looks
>> like?
>> >> >
>> >> > On 9/26/06, Pavitra Subramaniam <pa...@oracle.com>
>> >> 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:arjuna@gmail.com]
>> >> > > > 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 <si...@gmail.com> 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
>> >> > > > >
>> >> > > > >
>> >> > > >
>> >> > > >
>> >> > > >
>> >> > >
>> >> > >
>> >> >
>> >> >
>> >>
>> >>
>> >
>>
>>
>
Re: Process train and IE
Posted by Simon Lessard <si...@gmail.com>.
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 <da...@gmail.com> 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 <si...@gmail.com> 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 <ar...@gmail.com> wrote:
> >> >
> >> > thanks Pavitra,
> >> > Simon, can you post an example of what your "patch" markup looks
> like?
> >> >
> >> > On 9/26/06, Pavitra Subramaniam <pa...@oracle.com>
> >> 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:arjuna@gmail.com]
> >> > > > 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 <si...@gmail.com> 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
> >> > > > >
> >> > > > >
> >> > > >
> >> > > >
> >> > > >
> >> > >
> >> > >
> >> >
> >> >
> >>
> >>
> >
>
>
Re: Process train and IE
Posted by Scott O'Bryan <da...@gmail.com>.
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 <si...@gmail.com> 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 <ar...@gmail.com> wrote:
>> >
>> > thanks Pavitra,
>> > Simon, can you post an example of what your "patch" markup looks like?
>> >
>> > On 9/26/06, Pavitra Subramaniam <pa...@oracle.com>
>> 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:arjuna@gmail.com]
>> > > > 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 <si...@gmail.com> 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
>> > > > >
>> > > > >
>> > > >
>> > > >
>> > > >
>> > >
>> > >
>> >
>> >
>>
>>
>
Re: Process train and IE
Posted by Arjuna Wijeyekoon <ar...@gmail.com>.
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 <si...@gmail.com> 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 <ar...@gmail.com> wrote:
> >
> > thanks Pavitra,
> > Simon, can you post an example of what your "patch" markup looks like?
> >
> > On 9/26/06, Pavitra Subramaniam <pa...@oracle.com> 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:arjuna@gmail.com]
> > > > 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 <si...@gmail.com> 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
> > > > >
> > > > >
> > > >
> > > >
> > > >
> > >
> > >
> >
> >
>
>
Re: Process train and IE
Posted by Simon Lessard <si...@gmail.com>.
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 <ar...@gmail.com> wrote:
>
> thanks Pavitra,
> Simon, can you post an example of what your "patch" markup looks like?
>
> On 9/26/06, Pavitra Subramaniam <pa...@oracle.com> 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:arjuna@gmail.com]
> > > 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 <si...@gmail.com> 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
> > > >
> > > >
> > >
> > >
> > >
> >
> >
>
>
Re: Process train and IE
Posted by Arjuna Wijeyekoon <ar...@gmail.com>.
thanks Pavitra,
Simon, can you post an example of what your "patch" markup looks like?
On 9/26/06, Pavitra Subramaniam <pa...@oracle.com> 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:arjuna@gmail.com]
> > 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 <si...@gmail.com> 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
> > >
> > >
> >
> >
> >
>
>
RE: Process train and IE
Posted by Pavitra Subramaniam <pa...@oracle.com>.
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:arjuna@gmail.com]
> 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 <si...@gmail.com> 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
> >
> >
>
>
>
Re: Process train and IE
Posted by Arjuna Wijeyekoon <ar...@gmail.com>.
is this IE 6 or IE 7?
On 9/23/06, Simon Lessard <si...@gmail.com> 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
>
>