Tek-Tips is the largest IT community on the Internet today!

Members share and learn making Tek-Tips Forums the best source of peer-reviewed technical information on the Internet!

Register Log in
  • Congratulations Mike Lewis on being selected by the Tek-Tips community for having the most helpful posts in the forums last week. Way to Go!

Site Update - Review 5

Status
Not open for further replies.
Sort by date Sort by votes
Don't really have much time, so just my first impression--the text is too light to read easily and the orange stands out too much and shouldn't the light blue over there be more of the greyish color of the rest? Other than that, first impression is [2thumbsup].

Just make sure it validate, I'm assuming it does.

Rick

-----------------------------------------------------------
RISTMO Designs
Arab Church
Reference Guides
 
Upvote 0 Downvote
Hi Mate,

Checked the site out in Opera and IE 5.

Opera - The page was called over 4 minutes ago and it has not fully loaded yet, the problem being the weather box.

I've just reloaded after 4 minutes 30 seconds (Weather still didn't load) and now I get the weather box but it breaks out of the cell it should be in and over the news text.

The expandable navigation options don't work, the images change to show a - instead of a + but no new links appear.

On photo.php, the top banner looks totally screwed as the "Page 1" and "Total Photos" wrap and appear on the left of the banner.


IE 5 - The page renders fine apart from the navigation, where the text appears over the small images. (Which makes it totally unreadable) A reload of the page shows the same problem with the weather box, 3 minutes passed so far and it's still not loaded.


The site claims to be fully standards compliant yet validating the page I get the following error:

Line 16, column 30: there is no attribute "BORDER".

<frameset cols=&quot;115,*&quot; border=&quot;0&quot;>


Hope this helps


Wullie


The pessimist complains about the wind. The optimist expects it to change.
The leader adjusts the sails. - John Maxwell
 
Upvote 0 Downvote
Cheers Wullie

I suspected something was wrong with the weather page. It worked OK in the old site but since the PHP version was upgraded on the webserver something has gone amiss.

Opera Just out of interest what version did you use. I checked the site with Opera7 and the menu appeared fine. I was aware however that it didn't work to well with earlier versions though. I will look into the text wrapping problem on the Photo page.
IE5 Guess I need to install it I checked site with IE6 and menu appears OK

The site claims to be fully standards compliant yet validating the page I get the following error:

Line 16, column 30: there is no attribute &quot;BORDER&quot;.

<frameset cols=&quot;115,*&quot; border=&quot;0&quot;>

It was originally but the rest of the club members didn't like the white border around the frames. Forgot to re-write the text until I find a better way to resolve the problem.
Thanks for spending the time and passing comments
regards
Ian


Infinity exists! - I just haven't worked out a way to get there yet.

| |
 
Upvote 0 Downvote
ne4x4

Please refer to:
I don't appreciate you stealing the image of the browser images from my site. If you had asked me to use them I would have let you use them as I can't hold copyright over those icons anyway. However, that being said, the assembled work of the images is mine.

On a matter of principle for not asking me to use the image, I request you please remove it from your site immediately.


Nate

mainframe.gif

 
Upvote 0 Downvote
In terms of content, perhaps consider putting something more relevant on the home page. I think you're giving too much emphasis on technical site issues that would be of no interest to people looking for information about your club. I'm not sure if the general news headlines belong here either - it looks like your padding out content for the sake of it. Weather and racing news are probably more appropriate, and should be grouped together.

Overall, it's neat and looks well, but there are too many boxes for my liking. On the home page especially, it's difficult to know if any one piece of information is more important than the next. I would suggest grouping the login, weather, news vertically along the right, putting the important info in the same column as the what's new section (probably above it), and leave the center column for good relevant introduction text.

Other than that, my only problem would be with the side navigation. Once you've selected a page, it's good practice to make that link inactive, and preferably distinguish it from the others in some way. I appreciate that your use of frames might complicate this, but it can be done.

Hope that helps!
 
Upvote 0 Downvote
Thanks Blueark some interesting points worth considering. Normally the welcome page would have more 4x4 and club related info than it does currently but your point is a valid one. I will however have a look at redisigning the page to make it less confusing.

With regards to the menu I will make active /visible links more apparent than they are at present. The use of frames is a concession I made to maintain some consisteny with the old site it replaced which used frames and standard html (approx 1000 pages) and was used to make navigation simple. The move to php and more dynamic content has made the use of frames less necessary and they will probably dissapear completely in the next wre-write
Thanx for the comments
Regards
Ian

Infinity exists! - I just haven't worked out a way to get there yet.

| |
 
Upvote 0 Downvote
Well, it's a clean design that seems to work OK in IE5.5. You could try suppressing that border in a standards-compliant way like this:
[tt]
<frameset style=&quot;border:0&quot; cols=&quot;115,*&quot;>
[/tt]
though if your current page is working anyway it perhaps isn't worth the effort.

As well as making it more relavant to 4x4 users than browser geeks, perhaps you could use a photo on the front page to give it more visual impact.

Text colour looks OK, but it's pretty small. What's worse, you can't re-size it in IE. Consider using named font sizes or ems instead of pixels in your style sheet.

&quot;Site Info&quot; and &quot;Vote&quot; links along the bottom launched pop-ups (yuck) containing 404's (double-yuck).

-- Chris Hunt
 
Upvote 0 Downvote
hello Ne4x4!
What kind of critique are u looking for?
is it more towards the &quot;look and feel&quot; or validity of the code and cross-browser issues?
....so many aspect can be taken...soo which one would u like?( having in mind that peope can't write a 10 pg. review of the site :) )

All the best!


> need more info?
:: don't click HERE ::
 
Upvote 0 Downvote
Upvote 0 Downvote
my main pain was on the frames...but u answered your own question about that. :)
since u are using frames I would dump the JS popups...since its PHP (none here) I can't see the code behind that part...
I have seen this:
</script>
</head>
<body>
few times (diary.php- I think)...it seems that msot of the &quot;bodys&quot; are ok....
just a few quick notes :)
keep the good work!

All the best!

> need more info?
:: don't click HERE ::
 
Upvote 0 Downvote
Hi lebisol

Thanks for the comments. what you are probably referring to is the following

<script TYPE=&quot;text/javascript&quot;>
<!--
if (top==self)
self.location.href=&quot;index.php?page=/intro.php&quot;;
-->
</script>

This is locaited in the head of every page and is used by the php script in the index page to ensure that a particular page is loaded in a specific frame even if the frameset is not present (ie clicking on the result from a search engine)

Code: 
if (!$_GET['page']){
  $pg=&quot;intro.php&quot;;
  } else {
  $pg=$_GET['page'];
}
The location of the page in the main frame is also added by the php script as follows.

<frame name=&quot;main&quot; scrolling=&quot;auto&quot; noresize frameborder=&quot;0&quot; marginwidth=&quot;0&quot; marginheight=&quot;0&quot; src=&quot;<?php echo $pg; ?>&quot;>

Regards
Ian

Infinity exists! - I just haven't worked out a way to get there yet.

| |
 
Upvote 0 Downvote
:) Hi Ian!
Actually I was refering to the fact that u mix script into head tags ....now that I think about it...I am not sure if the script tags have to be after or before head but NOT in it....anyway
all the best!


> need more info?
:: don't click HERE ::
 
Upvote 0 Downvote
sometimes I forget where I live...thank good for IDs :)


> need more info?
:: don't click HERE ::
 
Upvote 0 Downvote
...spelling is the 2nd thing in line LOL

> need more info?
:: don't click HERE ::
 
Upvote 0 Downvote
Status
Not open for further replies.

Similar threads

tyutghf
  • Locked
  • Question
Linkedin updates on website
Replies
0
Views
169
tyutghf
tyutghf
glenmac
  • Locked
  • Question
site icon 1
Replies
4
Views
126
glenmac
glenmac
Pyrrhus
  • Locked
  • Question
Automatic subscription to multiple-site newsletters.
Replies
1
Views
141
kjv1611
kjv1611
phouv2000
  • Locked
  • Question
Update indicator
Replies
2
Views
45
phouv2000
phouv2000
nagornyi
  • Locked
  • Question
Site design tool advice needed 2
Replies
2
Views
84
kjv1611
kjv1611

Part and Inventory Search

Sponsor

Back
Top