You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@forrest.apache.org by br...@apache.org on 2004/01/09 15:33:32 UTC

cvs commit: xml-forrest/src/core/context/skins/krysalis-site/xslt/html site2xhtml.xsl

brondsem    2004/01/09 06:33:32

  Modified:    src/core/context/skins/krysalis-site/xslt/html
                        site2xhtml.xsl
  Log:
  lateral search box respects $config/disable-search
  
  Revision  Changes    Path
  1.9       +10 -6     xml-forrest/src/core/context/skins/krysalis-site/xslt/html/site2xhtml.xsl
  
  Index: site2xhtml.xsl
  ===================================================================
  RCS file: /home/cvs/xml-forrest/src/core/context/skins/krysalis-site/xslt/html/site2xhtml.xsl,v
  retrieving revision 1.8
  retrieving revision 1.9
  diff -u -r1.8 -r1.9
  --- site2xhtml.xsl	8 Jan 2004 16:18:43 -0000	1.8
  +++ site2xhtml.xsl	9 Jan 2004 14:33:32 -0000	1.9
  @@ -233,11 +233,14 @@
   
         <tr>
           <td height="10" colspan="2"></td>
  -      </tr>                
  -
  -      <tr>       	  
  +      </tr>
  +      
  +      <xsl:if test="not($config/disable-search) or
  +            $config/disable-search='false' and $config/searchsite-domain and
  +            $config/searchsite-name">
  +      <tr>
           <td ></td>
  -       	  <td class="search">
  +        <td class="search">
               <form method="get" action="http://www.google.com/search" target="_blank">
                <table class="dialog" cellspacing="0" cellpadding="0" border="0" width="100%">
                   <tr>
  @@ -275,7 +278,8 @@
                 </table>
                 </form>
                 </td>
  -          </tr>     
  +          </tr>
  +          </xsl:if>
    	  
             <xsl:if test="$filename = 'index.html' and $config/credits">
    	     <tr>
  
  
  

Re: lateral search box in krysalis-site

Posted by Dave Brondsema <da...@brondsema.net>.
On Fri, 9 Jan 2004, Nicola Ken Barozzi wrote:

> Dave Brondsema wrote:
>
> > On Fri, 9 Jan 2004 brondsem@apache.org wrote:
> >
> >
> >>brondsem    2004/01/09 06:33:32
> >>
> >>  Modified:    src/core/context/skins/krysalis-site/xslt/html
> >>                        site2xhtml.xsl
> >>  Log:
> >>  lateral search box respects $config/disable-search
> >>
> >>  Revision  Changes    Path
> >>  1.9       +10 -6     xml-forrest/src/core/context/skins/krysalis-site/xslt/html/site2xhtml.xsl
> >>
> >
> > If we have a search box on the left side, do we still want the one on the
> > top?  It seems kinda weird to have both.  I think either looks fine, but
> > not both.
>
> Yup, I still have to add an entry in skinconf to toggle it. Tigris-style
> has both too, but I remembered in that case to comment out one of them ;-)
>
> BTW, could you please move the 'nowrap' in the CSS in page.css?
> forrest.css should be used only for things that need information from
> skinconf.xml. TIA
>

Ah, thanks.  I didn't realize exactly what the difference was between
page.css and forrest.css.xslt

-- 
Dave Brondsema
dave@brondsema.net
http://www.brondsema.net - personal
http://www.splike.com - programming
http://csx.calvin.edu - Calvin club

Re: lateral search box in krysalis-site

Posted by Nicola Ken Barozzi <ni...@apache.org>.
Dave Brondsema wrote:

> On Fri, 9 Jan 2004 brondsem@apache.org wrote:
> 
> 
>>brondsem    2004/01/09 06:33:32
>>
>>  Modified:    src/core/context/skins/krysalis-site/xslt/html
>>                        site2xhtml.xsl
>>  Log:
>>  lateral search box respects $config/disable-search
>>
>>  Revision  Changes    Path
>>  1.9       +10 -6     xml-forrest/src/core/context/skins/krysalis-site/xslt/html/site2xhtml.xsl
>>
> 
> If we have a search box on the left side, do we still want the one on the
> top?  It seems kinda weird to have both.  I think either looks fine, but
> not both.

Yup, I still have to add an entry in skinconf to toggle it. Tigris-style 
has both too, but I remembered in that case to comment out one of them ;-)

BTW, could you please move the 'nowrap' in the CSS in page.css?
forrest.css should be used only for things that need information from 
skinconf.xml. TIA

-- 
Nicola Ken Barozzi                   nicolaken@apache.org
             - verba volant, scripta manent -
    (discussions get forgotten, just code remains)
---------------------------------------------------------------------


lateral search box in krysalis-site

Posted by Dave Brondsema <da...@brondsema.net>.
On Fri, 9 Jan 2004 brondsem@apache.org wrote:

> brondsem    2004/01/09 06:33:32
>
>   Modified:    src/core/context/skins/krysalis-site/xslt/html
>                         site2xhtml.xsl
>   Log:
>   lateral search box respects $config/disable-search
>
>   Revision  Changes    Path
>   1.9       +10 -6     xml-forrest/src/core/context/skins/krysalis-site/xslt/html/site2xhtml.xsl
>


If we have a search box on the left side, do we still want the one on the
top?  It seems kinda weird to have both.  I think either looks fine, but
not both.



-- 
Dave Brondsema
dave@brondsema.net
http://www.brondsema.net - personal
http://www.splike.com - programming
http://csx.calvin.edu - Calvin club