[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-cygwin
Subject:    RE: New Layout/Style of kde-cygwin website
From:       "Ralf Habacker" <Ralf.Habacker () freenet ! de>
Date:       2002-06-27 22:46:46
[Download RAW message or body]

> > > I've checked in the sources under a new branch KDE-CYGWIN_WEBSITE_2_x
> > > website/htdocs. Fell free to check it out local and setup an apache
> > > server with
> > > the following configuration (please change the path):
> > >
> > Try http://cygwin.kde.org/new/  and send any comments to the list
> Using Mozilla, the News header text on the right of the overview page is
> quite small compared to the size of the blue bar. Also the text is quite
> hard to read on the blue background, because there is not much contrast
> between the blue and the black.

Okay I'll try to find another color scheme.

> Also, when I am navigating sometimes the page on the left I am on is
> highlighted and sometimes it is not.
> e.g. click Home and see that [Home] and [Overview] are highlighted.
> click the Press Info link and see that only [Press Info] is now highlighted.

It's a bug in root_header.inc:printcategories(), for which I haven't found a
solution yet.

function printcategories($cat)
{
	global $location, $cur_category, $root, $cat_nav;

	//$eol = '<td bgcolor="#808080"></td></tr>'."\n";
	$eol = '</tr>'."\n";

	for ($i = 0; $i < sizeof($cat); $i += 2) {
	  $link = $cat[$i];
	  $name = $cat[$i+1];
		print '<tr>';
		// category
	  if ($link == "") {
	  	if ($i > 0)
			print '<td colspan=2 class="navtitle"><b>' . $name . '<b></td>'.$eol;
		} else if (strstr($location,$root.'/'.$link) != "" ) {
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
This is the compare for detecting the current categorie and it fails
because the home categories lives in the root and all others lives in subdirs.
Perpahs you have an idea how to fix this :-)

	    print '<td width="10%"></td><td class="navselected">'.$name.'</td>'.$eol;
	    $cur_category = $name;
	  }
		else {
			if (strstr($link,"http"))
				$url = $link;
			else
				$url = $root."/".$link;
			print '<td width="10%"></td><td class="navunselected"><a
href="'.$url.'">'.$name.'</a></td>'.$eol;;
		}
	}
}

> I think the Press Info link should be renamed Press Articles. Press Info
> implies information about the project that the press could use when they
> wanted to write about us.
Okay, fixed.
>
> In line with what you said before about separating content and layout, it
> would be good to move the formatting information to stylesheets. Personally,
> I would prefer if stuff like valign, colspacing, etc. were moved as well.
> (Well, I would actually prefer if we were using the strict DTD, but I am
> aware not everyone sees the point in that.)

Me too, currently I'm patching some files to enable this, but it's hard work

I will check in my changes in a few minutes and go to bed.
Then you can check and patch if you like.

BTW:
1. I've rearranged some css definitions.

2. There is only one global css file global.css

3. pre.shell replaces of pre.sample for all shell relevant printings. I like to
   reduce the structure levels.
   Perhaps we can divide this later in more detailed classes.

> Also, picking a nit, it is now deprecated to use tables for content layout
> where they can be avoided. e.g. in the left hand nav bar, the content can be
> laid out perfectly well without using a table using appropriate div tags.

Perhaps you can give a try for an implementation ?

> Overall, I think you have done a good job on the design so far Ralph. It's
> clear you have thought carefully about navigation and content separation.

BTW: The logo is official granted by the kde team.

Thanks

_______________________________________________
kde-cygwin mailing list
kde-cygwin@mail.kde.org
http://mail.kde.org/mailman/listinfo/kde-cygwin
[prev in list] [next in list] [prev in thread] [next in thread]