Missing media attribute on css imports

classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

Missing media attribute on css imports

Nathan Jones-2
After feeding Orbeon's xhtml output into itext pdf printer many hidden
elements appear on the pdf (repeat templates for example). This is due
to the css import not explicitly setting the media attribute which
should probably be set to something like media="screen, projection, print".

- Nathan


--
You receive this message as a subscriber of the [hidden email] mailing list.
To unsubscribe: mailto:[hidden email]
For general help: mailto:[hidden email]?subject=help
OW2 mailing lists service home page: http://www.ow2.org/wws
Reply | Threaded
Open this post in threaded view
|

Re: Missing media attribute on css imports

Erik Bruchez
Administrator
Nathan,

This is good information. Is there any way you can fix this and  
contribute the changes back?

-Erik

On Mar 9, 2008, at 10:19 PM, Nathan Jones wrote:

> After feeding Orbeon's xhtml output into itext pdf printer many  
> hidden elements appear on the pdf (repeat templates for example).  
> This is due to the css import not explicitly setting the media  
> attribute which should probably be set to something like  
> media="screen, projection, print".
>
> - Nathan
>
> --
> You receive this message as a subscriber of the [hidden email]  
> mailing list.
> To unsubscribe: mailto:[hidden email]
> For general help: mailto:[hidden email]?subject=help
> OW2 mailing lists service home page: http://www.ow2.org/wws
--
Orbeon Forms - Web Forms for the Enterprise Done the Right Way
http://www.orbeon.com/



--
You receive this message as a subscriber of the [hidden email] mailing list.
To unsubscribe: mailto:[hidden email]
For general help: mailto:[hidden email]?subject=help
OW2 mailing lists service home page: http://www.ow2.org/wws
Reply | Threaded
Open this post in threaded view
|

Re: Re: Missing media attribute on css imports

Nathan Jones-2
Erik Bruchez wrote:

> Nathan,
>
> This is good information. Is there any way you can fix this and
> contribute the changes back?
>
> -Erik
>
> On Mar 9, 2008, at 10:19 PM, Nathan Jones wrote:
>
>> After feeding Orbeon's xhtml output into itext pdf printer many
>> hidden elements appear on the pdf (repeat templates for example).
>> This is due to the css import not explicitly setting the media
>> attribute which should probably be set to something like
>> media="screen, projection, print".
>>
>> - Nathan
>>
>> --
>> You receive this message as a subscriber of the [hidden email]
>> mailing list.
>> To unsubscribe: mailto:[hidden email]
>> For general help: mailto:[hidden email]?subject=help
>> OW2 mailing lists service home page: http://www.ow2.org/wws
>
> --
> Orbeon Forms - Web Forms for the Enterprise Done the Right Way
> http://www.orbeon.com/
>
I don't have the latest code checked out so I can't make a patch. In the
version I have checked out (3.5.1 31 March 2007) the fix appears to be
to replace in XhtmlHeadHandler.java:

final String combinedResourceName = combinedResourcesPrefix + ".css";
            helper.element(prefix, XMLConstants.XHTML_NAMESPACE_URI,
"link", new String[] {
                        "rel", "stylesheet", "href",
combinedResourceName, "type", "text/css"});
with

final String combinedResourceName = combinedResourcesPrefix + ".css";
            helper.element(prefix, XMLConstants.XHTML_NAMESPACE_URI,
"link", new String[] {
                        "rel", "stylesheet", "href",
combinedResourceName, "type", "text/css", "media", "all"});

The possible media types are listed at
http://www.w3.org/TR/html401/types.html#type-media-descriptors. I
_think_ all the browsers supported by Orbeon understand media="all".

- Nathan

ps, the xhtml to pdf renderer I referred to is flying saucer not itext
(itext is the pdf library used by flying saucer).


--
You receive this message as a subscriber of the [hidden email] mailing list.
To unsubscribe: mailto:[hidden email]
For general help: mailto:[hidden email]?subject=help
OW2 mailing lists service home page: http://www.ow2.org/wws
Reply | Threaded
Open this post in threaded view
|

Re: Re: Re: Missing media attribute on css imports

Erik Bruchez
Administrator
>> I don't have the latest code checked out so I can't make a patch.  

>> In the version I have checked out (3.5.1 31 March 2007) the fix  
>> appears to be to replace in XhtmlHeadHandler.java:
>
> final String combinedResourceName = combinedResourcesPrefix + ".css";
>           helper.element(prefix, XMLConstants.XHTML_NAMESPACE_URI,  
> "link", new String[] {
>                       "rel", "stylesheet", "href",  
> combinedResourceName, "type", "text/css"});
> with
>
> final String combinedResourceName = combinedResourcesPrefix + ".css";
>           helper.element(prefix, XMLConstants.XHTML_NAMESPACE_URI,  
> "link", new String[] {
>                       "rel", "stylesheet", "href",  
> combinedResourceName, "type", "text/css", "media", "all"});
Sounds easy enough. Do you know if there are any drawback setting  
media="all"?

>
> ps, the xhtml to pdf renderer I referred to is flying saucer not  
> itext (itext is the pdf library used by flying saucer).


Looks like a great project which could help offer an alternative for  
PDF output in Orbeon Forms. Do you have a good experience with this so  
far?

-Erik

--
Orbeon Forms - Web Forms for the Enterprise Done the Right Way
http://www.orbeon.com/



--
You receive this message as a subscriber of the [hidden email] mailing list.
To unsubscribe: mailto:[hidden email]
For general help: mailto:[hidden email]?subject=help
OW2 mailing lists service home page: http://www.ow2.org/wws
Reply | Threaded
Open this post in threaded view
|

Re: Re: Re: Re: Missing media attribute on css imports

Nathan Jones-2
Erik Bruchez wrote:

>>> I don't have the latest code checked out so I can't make a patch. In
>>> the version I have checked out (3.5.1 31 March 2007) the fix appears
>>> to be to replace in XhtmlHeadHandler.java:
>>
>> final String combinedResourceName = combinedResourcesPrefix + ".css";
>>           helper.element(prefix, XMLConstants.XHTML_NAMESPACE_URI,
>> "link", new String[] {
>>                       "rel", "stylesheet", "href",
>> combinedResourceName, "type", "text/css"});
>> with
>>
>> final String combinedResourceName = combinedResourcesPrefix + ".css";
>>           helper.element(prefix, XMLConstants.XHTML_NAMESPACE_URI,
>> "link", new String[] {
>>                       "rel", "stylesheet", "href",
>> combinedResourceName, "type", "text/css", "media", "all"});
>
> Sounds easy enough. Do you know if there are any drawback setting
> media="all"?
>
>>
>> ps, the xhtml to pdf renderer I referred to is flying saucer not
>> itext (itext is the pdf library used by flying saucer).
>
>
> Looks like a great project which could help offer an alternative for
> PDF output in Orbeon Forms. Do you have a good experience with this so
> far?
>
> -Erik
>
> --
> Orbeon Forms - Web Forms for the Enterprise Done the Right Way
> http://www.orbeon.com/
>
I don't think there should be any drawbacks with using media="all" for
standard browsers. I guess it could mean wasted bandwidth usage for
handheld or screen reader browsers though. Also, Netscape 4.x ignores
css declared with media="all"
http://htmlhelp.com/reference/css/style-html.html.

We have limited experience with flying saucer but it seems to be pretty
good so far.

- Nathan


--
You receive this message as a subscriber of the [hidden email] mailing list.
To unsubscribe: mailto:[hidden email]
For general help: mailto:[hidden email]?subject=help
OW2 mailing lists service home page: http://www.ow2.org/wws
Reply | Threaded
Open this post in threaded view
|

Re: Missing media attribute on css imports

Erik Bruchez
Administrator
>> I don't think there should be any drawbacks with using media="all"  
>> for standard browsers. I guess it could mean wasted bandwidth usage  
>> for handheld or screen reader browsers though. Also, Netscape 4.x  
>> ignores css declared with media="all" http://htmlhelp.com/reference/css/style-html.html 
>> .
>
> We have limited experience with flying saucer but it seems to be  
> pretty good so far.

This is now fixed:

http://forge.objectweb.org/tracker/index.php?func=detail&aid=308825&group_id=168&atid=350207

We played a bit with Flying Saucer and may integrate it into Orbeon  
Forms. It is quite good!

-Erik

--
Orbeon Forms - Web Forms for the Enterprise Done the Right Way
http://www.orbeon.com/



--
You receive this message as a subscriber of the [hidden email] mailing list.
To unsubscribe: mailto:[hidden email]
For general help: mailto:[hidden email]?subject=help
OW2 mailing lists service home page: http://www.ow2.org/wws