RC6: Web template tweaks

Bugs found in XnView Classic. Please report only one bug per topic!

Moderators: XnTriq, helmut, xnview

Olive
Posts: 215
Joined: Wed Oct 06, 2004 9:18 pm
Location: Marseille
Contact:

RC6: Web template tweaks

Post 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>
Last edited by Olive on Wed Jun 15, 2005 10:29 pm, edited 1 time in total.
ckit
XnThusiast
Posts: 2557
Joined: Tue Feb 17, 2004 1:11 am
Location: Cabarlah, Australia

Post 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.
AMD Ryzen 3 3300X 3.8Ghz, 16Gb DDR4, RX6600XT with Dell U2520D at 2560x1440@60Hz
Windows 11 Pro x64 23H2, PowerToys and Wintoys
Olive
Posts: 215
Joined: Wed Oct 06, 2004 9:18 pm
Location: Marseille
Contact:

Post by Olive »

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

Image
ckit
XnThusiast
Posts: 2557
Joined: Tue Feb 17, 2004 1:11 am
Location: Cabarlah, Australia

Post by ckit »

Internet Explorer seems to be ignoring "text-align: center;" in the CSS file while Firefox handles it just fine.
Last edited by ckit on Mon Jun 13, 2005 1:30 am, edited 2 times in total.
ckit
XnThusiast
Posts: 2557
Joined: Tue Feb 17, 2004 1:11 am
Location: Cabarlah, Australia

Post by ckit »

There should be a setting in XnView for it.
Last edited by ckit on Mon Jun 13, 2005 1:31 am, edited 1 time in total.
Olive
Posts: 215
Joined: Wed Oct 06, 2004 9:18 pm
Location: Marseille
Contact:

Post 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:
Last edited by Olive on Wed Jun 15, 2005 1:46 pm, edited 4 times in total.
User avatar
ckv
Posts: 786
Joined: Wed Feb 02, 2005 2:30 pm
Location: Glow
Contact:

Post 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
XnView Tweak UI - Tool to customize your XnView beyond the regular XnView options.
UI-less Settings - Documentation of all the hidden settings in XnView.
XFAM - Tool to create and customize XnView file associations.
Olive
Posts: 215
Joined: Wed Oct 06, 2004 9:18 pm
Location: Marseille
Contact:

Post by Olive »

Heheh nice map. The Fins indeed are ahead of us :)
scratch
Posts: 82
Joined: Sun May 22, 2005 11:23 pm
Location: USA

Post 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.
Olive
Posts: 215
Joined: Wed Oct 06, 2004 9:18 pm
Location: Marseille
Contact:

Post 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
User avatar
xnview
Author of XnView
Posts: 43328
Joined: Mon Oct 13, 2003 7:31 am
Location: France
Contact:

Post by xnview »

Is it possible to send the fixed css/html files?
Pierre.
Olive
Posts: 215
Joined: Wed Oct 06, 2004 9:18 pm
Location: Marseille
Contact:

Post by Olive »

Yes but in a few hours, I'm busy with work at the moment :(
User avatar
xnview
Author of XnView
Posts: 43328
Joined: Mon Oct 13, 2003 7:31 am
Location: France
Contact:

Post by xnview »

Olive wrote:Yes but in a few hours, I'm busy with work at the moment :(
Do you have some updated files??
Pierre.
Olive
Posts: 215
Joined: Wed Oct 06, 2004 9:18 pm
Location: Marseille
Contact:

Post 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.
Last edited by Olive on Wed Jun 15, 2005 1:47 pm, edited 1 time in total.
User avatar
xnview
Author of XnView
Posts: 43328
Joined: Mon Oct 13, 2003 7:31 am
Location: France
Contact:

Post 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
Pierre.
Post Reply