RC6: Web template tweaks

Olive · Post by **Olive** » Sun Jun 12, 2005 1:18 pm

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):

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>

ckit · Post by **ckit** » Sun Jun 12, 2005 1:24 pm

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.

Olive · Post by **Olive** » Sun Jun 12, 2005 1:31 pm

Excuse me, where do we turn this on? FYI I generated as follows:

ckit · Post by **ckit** » Sun Jun 12, 2005 1:34 pm

Internet Explorer seems to be ignoring "text-align: center;" in the CSS file while Firefox handles it just fine.

ckit · Post by **ckit** » Sun Jun 12, 2005 1:46 pm

There should be a setting in XnView for it.

Olive · Post by **Olive** » Sun Jun 12, 2005 2:21 pm

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

ckv · Post by **ckv** » Sun Jun 12, 2005 3:16 pm

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)

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

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.

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

Olive · Post by **Olive** » Sun Jun 12, 2005 8:58 pm

Heheh nice map. The Fins indeed are ahead of us

scratch · Post by **scratch** » Sun Jun 12, 2005 10:08 pm

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.

Olive · Post by **Olive** » Mon Jun 13, 2005 6:08 am

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):

In IE (after):

In firefox (before *and* after):

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:

Post by **xnview** » Mon Jun 13, 2005 1:01 pm

Is it possible to send the fixed css/html files?

Olive · Post by **Olive** » Mon Jun 13, 2005 6:22 pm

Yes but in a few hours, I'm busy with work at the moment

Post by **xnview** » Wed Jun 15, 2005 12:40 pm

Olive wrote:Yes but in a few hours, I'm busy with work at the moment

Do you have some updated files??

Olive · Post by **Olive** » Wed Jun 15, 2005 1:44 pm

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.

Post by **xnview** » Wed Jun 15, 2005 1:46 pm

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