 | |
Feedback on new index page appreciated
Websites/HTML pages critique & reviews Discuss and review existing WWW material (alt.html.critique)
 |
| | Thread Tools | Display Modes |
#1
 
| |||
| |||
|
Feedback on new index page appreciated -
08-07-2006
, 02:14 AM
#2
| |||
| |||
|
#3
| |||
| |||
|
|
Chris H. wrote: http://www.internetsecurity101.net As a rule of thumb, the total file size for a page should be no more than 30K. The image in the top left alone is 341K! (This is because its a pretty large image and then dramatically downscale it with the browser - resize the actual image). Your server fails to describe the character encoding you are using. Umm... <small><em></em></small> - why? p class="c1"> - Non-breaking spaces are for when you need a _space_ that isn't going to word wrap. Indentation is the job of the text-indent property. Blockquote elements mark up quoted material. If you want a margin use the margin-* properties. Underlining text on the WWW means "This is a link, click it". Don't underline text that isn't links (such as headings). li id="active"> </li> - a contentless list item? What is "DESiGN 2.0" and how do you expect the image in the top left to convey that information? Your stylesheet is invalid http://css-discuss.incutio.com/?page=UsingPoints -- David Dorward <http://blog.dorward.me.uk/> <http://dorward.me.uk/ Home is where the ~/.bashrc is |
index page. DESIGN 2.0 actually has nothing to do with my site. It is
the name of the template that I am using from http://www.oswd.org/. I
still got some more work to do on my index page
http:/www.internetsecurity101.net. Thanks for your critique. I really
appreciate it.
Chris H.
Internet Security 101 - http://www.internetsecurity101.net
#4
| |||
| |||
|
|
I would appreciate some feedback on my new index page http://www.internetsecurity101.net. I took into account many of the suggestions I was given in this group and used a template from http://www.oswd.org/. Wow! What an improvement! Very nice layout, clean, excellent |
Only a few minor bits:
- The background color for <body> is not specified.
- Some of the font specifications are little bizarre: "georgia, helvetica,
verdana, arial, sans-serif." Georgia is a serif font.
- Use of points (pt) for font sizes.
--
jmm (hyphen) list (at) sohnen-moe (dot) com
(Remove .AXSPAMGN for email)
#5
| |||
| |||
|
|
Hello, I would appreciate some feedback on my new index page http://www.internetsecurity101.net. I took into account many of the suggestions I was given in this group and used a template from http://www.oswd.org/. The exact template I used is http://www.oswd.org/design/preview/id/2834. Thanks again. Regards, Chris H. |
Nice, clean general appearance.
Using FireFox on a 600 X 800 display, I see a horizontal scroll bar; 25%
of the page is off-screen to the right. At 640 X 480, fully half the
page is off-screen. On pages other than the Home page, there was no
scroll bar at 600 X 800, though one did appear at slightly smaller
window widths.
Main content text above the white-on-green box overlaps the green
background slightly. Looks like the green box needs some top margin.
This was NOT a problem on pages other than the Home page.
As I increase font size, things start to overlap on the second increase,
which is a rather tight tolerance.
Links within the main text are hard to distinguish (I noticed some of
them only after enlarging the text). If you choose not to follow the
standard conventions for link appearance, you should at least make them
stand out more.
The image in the UL corner looks very grainy to me. I see that the
actual image is about 8 times the size, in both X and Y axes, as the
presented size, which means you are forcing the browser to resize it,
something browsers don't do well. (See
http://pages.prodigy.net/chris_beall...ge%20size.html and the
samples to which it refers.)
I felt that the "Please click on the links in the navigation bar on the
left to explore my site." was superfluous. It's unlikely that anyone
who couldn't deduce this would ever get to your site.
Your CSS uses numeric values without specifying the units, which won't
work (except for '0'). Amaya reports the following CSS errors:
*** Errors/warnings in http://www.internetsecurity101.net/style.css
line 52: Invalid margin-top value "4"
line 52: Invalid margin-right value "4"
line 52: Invalid margin-bottom value "4"
line 52: Invalid margin-left value "4"
line 53: Invalid white-space value "wrap"
line 54: Invalid padding-top value "4"
line 54: Invalid padding-right value "4"
line 54: Invalid padding-bottom value "4"
line 54: Invalid padding-left value "4"
line 62: Invalid margin-top value "4"
line 62: Invalid margin-right value "4"
line 62: Invalid margin-bottom value "4"
line 62: Invalid margin-left value "4"
line 63: Invalid white-space value "wrap"
line 78: Invalid value "green
font-weight: bold"
line 98: Invalid font value "sans-serif"
There seems to be an unnecessary amount of white space above the topmost
item on the screen, which is the top of the image. On my system the
space is about 2/3 the height of the image.
Chris Beall
|
« Previous Thread
|
Next Thread »
| Thread Tools | |
| Display Modes | |
Linear Mode
| |
Powered by vBulletin Version 3.5.4
Copyright ©2000 - 2008, Jelsoft Enterprises Ltd.
Copyright ©2000 - 2008, Jelsoft Enterprises Ltd.