Page 1 of 2

RC6: Web template tweaks

Posted: Sun Jun 12, 2005 1:18 pm
by Olive
Don't get me wrong, I believe it's a nice template, layout- and color-wise, but it could use some tweaking.

- XHTML instead of HTML: The problem is that XnView use <br> as linebreaks between THUMB_TEXT items, whereas XHTML requires <br />. So it's basically impossible to generate a valid XHTML, even a transitional one.

<th class="header">#TITLE# - #HEADER#</th>: It's a bad idea to force the display of both title and header. If the user wants the title to be displayed there, he's free to include it in the header.

- align="center": the align attribute is deprecated

- <img... border="0" />: Deprecated as well

- <td class="thumbs" style="width:#THUMB_HEIGHT#px;height:#THUMB_HEIGHT#px;">: I could be missing something but height:#THUMB_HEIGHT#px looks useless to me. The cell's height will auto adjust to its content will it not? As for width:#THUMB_HEIGHT#px I suppose it was meant to be width:#THUMB_WIDTH#px ? It's indeed useful to specify a width in this case, otherwise long THUMB_TEXT lines wouldn't wrap. But what will happen when the thumb is small (say 50-75px), the text is long, and the font is big? I guess a massive number of lines of text under the thumbs. Another advantage of fixed cell width is it wouldn't need to be repeated in every cell.

- There's a display bug in IE: Thumbnails aren't properly centered

Here's a screenshot to show the last 2 points (plus curious vertical-alignment):
Image

Could whoever designed these templates [Simple and Simple (Deutsch)] tweak them a bit before 1.80 is final? I could look into it otherwise =)

XnView 1.80 <x>

Posted: Sun Jun 12, 2005 1:24 pm
by ckit
Could you do that screenshot again with Table borders turned ON. I think the problem is that the tables need fixed values and not percentages but I could be wrong.

Posted: Sun Jun 12, 2005 1:31 pm
by Olive
Excuse me, where do we turn this on? FYI I generated as follows:

Image

Posted: Sun Jun 12, 2005 1:34 pm
by ckit
Internet Explorer seems to be ignoring "text-align: center;" in the CSS file while Firefox handles it just fine.

Posted: Sun Jun 12, 2005 1:46 pm
by ckit
There should be a setting in XnView for it.

Posted: Sun Jun 12, 2005 2:21 pm
by Olive
I know IE is crap yet 85% out of 800 000 000 internet users use IE so the page must look decent in IE too :mrgreen:

Posted: Sun Jun 12, 2005 3:16 pm
by ckv
ckit wrote:There should be a setting in XnView for it.

People don't use IE it's crap and doesn't follow proper standards. I recommend using Firefox or Mozilla for creating Web Templates with XnView. I know I'll get flamed for this but it is the truth.
Yeah. Speaking the truth I noticed this IE bug when testing web templates, but I didn't fix it still. (IE's fault) :mrgreen:
Olive wrote:I know IE is crap yet 85% out of 800 000 000 internet users use IE so the page must look decent in IE too :mrgreen:
Actually that is something like 55% (at least in here Finland) on these days. And microsoft's plans to not relese IE 7.0 to Win2k and older versions isn't going to raise EI's market share. :|

But, hey. You got good point and if you want to make them to work in IE, that is great. :D

Of the topic:
Check this out. I am proud of this

Posted: Sun Jun 12, 2005 8:58 pm
by Olive
Heheh nice map. The Fins indeed are ahead of us :)

Posted: Sun Jun 12, 2005 10:08 pm
by scratch
it'll be nice when the rest of the world catches up. IE's support for CSS is just way too outdated and it doesn't look like this will likely get fixed until at least 8.0, if then.

Posted: Mon Jun 13, 2005 6:08 am
by Olive
I've fixed another IE-only layout glitch: The css specifies

Code: Select all

.header{
	...
	height: 50px;
	padding: 10px;
	...
}
That's a total height of 70 pixels (height+padding-top+padding-bottom). The background image's height being only 53 pixels, it was being vertically repeated to fit 70.

In IE (before):
Image

In IE (after):
Image

In firefox (before *and* after):
Image

Edit: Also, in nav/style.css, .header{height:50px; should be replaced with .header{height:53px; (look in IE before, there's a border below the header text, that is no longer displayed)
--> Now in IE:
Image

Posted: Mon Jun 13, 2005 1:01 pm
by xnview
Is it possible to send the fixed css/html files?

Posted: Mon Jun 13, 2005 6:22 pm
by Olive
Yes but in a few hours, I'm busy with work at the moment :(

Posted: Wed Jun 15, 2005 12:40 pm
by xnview
Olive wrote:Yes but in a few hours, I'm busy with work at the moment :(
Do you have some updated files??

Posted: Wed Jun 15, 2005 1:44 pm
by Olive
hmm sorry for being late on this. I've almost finished (they actually all need updating). Here are Simple and Simple[XnView], let's see if somebody has a bug or something to report about them, and I'll try to email you the whole template folder tonight.

Posted: Wed Jun 15, 2005 1:46 pm
by xnview
Olive wrote:hmm sorry for being late on this. I've almost finished (they actually all need updating). Here are Simple and Simple[XnView], let's see if somebody has something to report about them, and I'll try to email you the whole template folder tonight.
Ok, thanks