Summary: | Printable documents not easy to get or not really printable. | ||
---|---|---|---|
Product: | [OLD] Docs-user | Reporter: | Xavier Neys (RETIRED) <neysx> |
Component: | Other | Assignee: | Gentoo Infrastructure <infra-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | docs-team |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | Patch to several files to implement suggested improvement |
Description
Xavier Neys (RETIRED)
2004-09-14 14:08:36 UTC
Created attachment 39607 [details, diff]
Patch to several files to implement suggested improvement
Please review. Thanks.
Suggested change to the AxKit config. I am neither an apache nor an AxKit gutu, this is the one I used: <IfDefine PERL> LoadModule perl_module extramodules/libperl.so PerlModule AxKit SetHandler perl-script PerlHandler Apache::AxKit::StyleChooser::PathInfo AxKit AddHandler axkit .xml .xsp AxAddXSPTaglib AxKit::XSP::Util AxAddXSPTaglib AxKit::XSP::IfParam AxAddXSPTaglib AxKit::XSP::Param AxAddStyleMap application/x-xsp Apache::AxKit::Language::XSP AxAddStyleMap text/xsl Apache::AxKit::Language::LibXSLT <Files *.xml> AxAddProcessor text/xsl /xsl/guide.xsl </Files> </IfDefine> What's important: - not reference guide-print.xsl anymore so that it can be removed - use guide.xsl *** Bug 64051 has been marked as a duplicate of this bug. *** The changes have been committed. Is axkit still using guide-print.xsl? We'd like to get rid of this one. Kurt? I have no idea whether or not AxKit uses guide-print.xsl. Also, you should not depend on any AxKit-specific features as we will be replacing it at some point (hopefully sooner, rather than later) If we have to chose between losing features that we use (and even rely on) today vs. finding a suitable, if less feature-rich replacement for AxKit, we will definitely choose the later. --kurt AxKit doesn't "depend" on it, our configuration does (or at least did when I got the httpd.conf excerpt from you). Neysx' changes don't require the use of an additional XSL file for the printable version which is an improvement over the old style (since it makes the search for a new solution easier). But you do have to alter the AxKit configuration as stated in comment #2. What I did was comment out the printable stuff: """ <AxStyleName "#default"> AxAddProcessor text/xsl /xsl/guide.xsl </AxStyleName> <AxStyleName printable> AxAddProcessor text/xsl /xsl/guide-print.xsl </AxStyleName> """ became """ <AxStyleName "#default"> AxAddProcessor text/xsl /xsl/guide.xsl </AxStyleName> # <AxStyleName printable> # AxAddProcessor text/xsl /xsl/guide-print.xsl # </AxStyleName> """ Everything still works, so this particular AxKit configuration can probably be removed. I haven't checked everything (especially those pages working on the /dyn generated ones) since I don't have those scripts. As Sven explained, we do not rely on axkit's features, well, not anymore. My recent changes help make things simpler and the move to another engine has become simpler as well. FYI, AxKit looks at the requested URL and calls different stylesheets based on a style variable. The current setup is to default to guide.xsl and call guide-print.xsl whenever style=printable appears in the URL. This is not required anymore. Sven's sample config keeps the style functionality but uses guide.xsl only. My sample in #2 completely removes the functionality. It works for me on docs, GWN and project pages. BTW, what's the status on the new engine? Are you still looking for a solution? I thought you'd use tr4nsf0rm. If not, I could cook you a simple ruby cgi script that processes the .xml through xsltproc, even adding caching would be trivial. Just let me know if you're interested. Reassigning to klieber (you really need an infrastructure@gentoo.org alias for bugzilla :). Check out if parts of the site still rely on guide-print.xsl. If not, it can be removed. guide-print.xsl has been removed as current AxKit config does not use it. This can be closed IMHO No need to keep this one open any longer. |