Topic: Website overhaul Pages that link to <a href="https://ozoneasylum.com/backlink?for=10442" title="Pages that link to Topic: Website overhaul" rel="nofollow" >Topic: Website overhaul\

 
Author Thread
Amerasu
Paranoid (IV) Inmate

From:
Insane since: Jun 2002

IP logged posted posted 09-16-2003 03:59 Edit Quote

I've just redone my website again (warning - I'm not exactly Joe Designer) and I think it's ok, viewable at least. Everything seems to work but I'm wondering if there are any glaring errors or issues I should address. I prefer to keep things simple (frames) because I don't have a lot of spare time to spend on this... or programming experience.

website

Thanks in advance


Amerasu

kromaZ
Bipolar (III) Inmate

From: Adanac
Insane since: Oct 2000

IP logged posted posted 09-16-2003 04:48 Edit Quote

Just love it....your site, the colors....especially your art!

Design, Coding Tech stuff?...don't know about that stuff...but do know talent when I see it!

cheers,
kromaz

ozphactor
Maniac (V) Inmate

From: California
Insane since: Jul 2003

IP logged posted posted 09-16-2003 05:19 Edit Quote

Doctype - Good to see that you have one, but there's one minor mistake. You're using the wrong one on the page that contains the frames. Any page with a <frameset> should technically be using a frameset doctype, like this one:

code:
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Frameset//EN"
"http://www.w3.org/TR/html4/frameset.dtd">



Encoding - Good to see that you have one of those too.

Frames - Okay, it's not that bad, but it's still something you can do without. I think things are simpler without frames, but I suppose that depends on how you look at it. Bottom line: Keep them if you must, but they're very 20th Century-ish...

CSS - I've spotted an accessibility issue with the way you're formatting your pages. The background color is specified in the <body> tag, but your text color is specified with CSS. If a person happened to have CSS turned off or a non-CSS browser, they would see their default text color (most likely black) on your HTML background color (black). I'm pretty sure that's not what you want My advice: specify the background color of the page using CSS as well, like so:

code:
body {
color: #ffffff;
background: #000000;
}


If we're going to get picky about it, the menu should be formatted as a <ul> as well, because that's essentially what it is: an unordered list. You can always remove the bullet with CSS, and replace it with a custom one, if you want some fancy formatting effect

Visual Design - In short... I like it Nice and simple. The text is just a tad too dark for me, but that's a minor issue.

Overall, nice work. And might I add that I'm a great fan of your artwork



[This message has been edited by ozphactor (edited 09-16-2003).]

Morph
Paranoid (IV) Inmate

From: The Soft Cell
Insane since: Nov 2001

IP logged posted posted 09-16-2003 10:10 Edit Quote

A nicely laid out site with good colour choice. Some very useful links on your links page although the one called 'posing guide' didn't work

well I hope you never get divorced - you're husband will have loads of ammo

some excellent paintings there




~We're not here for long, we're here for fun~

Amerasu
Paranoid (IV) Inmate

From:
Insane since: Jun 2002

IP logged posted posted 09-16-2003 17:07 Edit Quote

Thanks for the feedback and ideas, the comments are appreciated. I'll fix up those bits you mentioned, ozphactor - thanks for sifting through my crappy html

I do have a couple of questions though.. an older site of mine used to use CSS for positioning and I found that it wasn't very flexible for me. But I'm reading here and elsewhere that it's better for sites than the tables and frames I'm using now. Maybe I was doing it wrong. I set positions for my content boxes like this

code:
#PanelAContent { position : absolute; top : 149px; left : 240px; width : 760px; 
height : 300px; font-size : 11px; background-color : #A8A8A8; border : 1 solid #FFFFFF; }
#PanelAHeader { position : absolute; top : 100px; left : 240px; width : 760px;
height : 50px; font-size : 11px; background-color : #547FB3; border : 1 solid #FFFFFF; }
#PanelBContent { position : absolute; top : 509px; left : 240px; width : 760px;
height : 230px; font-size : 11px; background-color : #A8A8A8; border : 1 solid #FFFFFF; }
#PanelBHeader { position : absolute; top : 460px; left : 240px; width : 760px;
height : 50px; font-size : 11px; background-color : #547FB3; border : 1 solid #FFFFFF; }



Then I was stuck with an inflexible content box that didn't expand to the size of my display pic. I'm guessing that I was doing it wrong? How do I make my site with CSS and be flexible at the same time? I don't want to reduce all of my pics to one standard size in order to fit them nicely into a content box. I'd like to display them in whatever shape or size I originally made them. Does that make sense? How about iframes? Is that a viable option? Please help, I want to join the 21st century

Thanks again

Amerasu

Edit - to fix scrolling ugliness


[This message has been edited by Amerasu (edited 09-16-2003).]

Veneficuz
Paranoid (IV) Inmate

From: A graveyard of dreams
Insane since: Mar 2001

IP logged posted posted 09-16-2003 18:24 Edit Quote

The thing 'wrong' with the code above is that you've specified all sizes in px. To keep a liquid design you should avoid using pixels if possible. Instead use % and em to specify the size. You should also avoid using position: absolute and instead use the float property to posistion the elements... Blue Robot is a nice place to go for some tips on how it is done. They also have a couple of simple layouts there that you can read through the code for to see how they do it.

If you look through a couple of the other threads here you'll find a lot of links to places to learn CSS, they are all worth checking out.

The only thing I can think of design wise is that the hands in the top left corner stop too abrutly. Not sure how you could make them 'fade away' nicely, but with the sudden stop that you have now the whole thing seems too 'framey' (not a good word, but it'll have to do).

A little thing I noticed in your code. In your 'About' page you use the <br> tag to create a new paragraph. It works, but it is the wrong way of doing it. You should have each paragraph within the <p>...</p> tags. That way they be coded sematically correct and you'll get the same look as you have now

_________________________
"There are 10 kinds of people; those who know binary, those who don't and those who start counting at zero"
- the Golden Ratio -

[This message has been edited by Veneficuz (edited 09-16-2003).]

Amerasu
Paranoid (IV) Inmate

From:
Insane since: Jun 2002

IP logged posted posted 09-17-2003 03:21 Edit Quote

Thanks Veneficuz, I'm going to check out that link now. I appreciate your help. I'll fix that <br> thing too. I know I should use <p> but old habits die hard



Amerasu



Post Reply
 
Your User Name:
Your Password:
Login Options:
 
Your Text:
Loading...
Options:


« BackwardsOnwards »

Show Forum Drop Down Menu