Printable View
I made the chages you showed me but there seems to be somthing wrong.
The error I get is this:
Webpage Script Errors
User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 5.1; Trident/4.0; .NET CLR 1.1.4322)
Timestamp: Thu, 5 Mar 2009 03:25:11 UTC
Message: 'href' is null or not an object
Line: 31
Char: 7
Code: 0
URI: http://www.marie-joe.110mb.com/footer.js
I don't think that it's an issue that you have a similar (if not the same) aesthetic for your personal site. If that's your aesthetic, so be it.
With that said, I do find the aesthetic to be amateur, pedestrian and somewhat trite. This doesn't do anything to showcase the offerings of Marie-Joe Restaurant. If you're charging this woman for this website, you're doing her a grave disservice.
No am not charging her. I am doing it for free. The reason for that is because I live in a small community. One way for me to get my foot in the door for computer services is to have a link on a site. thus why I did it in the first place.
Fair enough. I suspected that might be the case. That does change my criticism a tiny bit.Quote:
Originally Posted by robin9000
But, my basic point is that after viewing the site, I don't really want to eat at that restaurant more. It's not an effective marketing/advertisement site. It looks generic in all the wrong ways.
Try this script:Code:// JavaScript Document
/* Original Script ©2009 John Davenport Scheuer
as first seen in http://www.dynamicdrive.com/forums/
username: jscheuer1 - This Notice Must Remain for Legal Use
Modified By username:Master_script_maker of http://www.dynamicdrive.com/forums/
*/
(function(t) {
document.write('<div id="js_footer"><\/div>');
var links = [
[ // declare href: , text: , title: for each centered link -
{href: 'privacy.html', text: 'Privacy Policy', title: ''},
{href: 'terms.html', text: 'Terms of use', title: ''},
{href: 'disclaimer.html', text: 'Disclaimer', title: ''},
{href: 'http://validator.w3.org/check/referer', text: 'XHTML', title: 'This page validates as XHTML 1.0 Transitional'},
{href: 'http://jigsaw.w3.org/css-validator/check/referer', text: 'CSS', title: 'This page validates as CSS'}
],
[
{href: 'http://www.robinsden.110mb.com', text: 'Designed by Robin\'s Den', title: 'This site was designed by Robin\'s Den'}
]
];
var a = [],
y = new Date().getFullYear(),
p = [document.createElement("span")];
p[0].appendChild(document.createTextNode(t.replace(/_y_/, y)));
for(var i=0;i<links.length;++i) {
a[i] = [];
for(var n=0;n<links[i].length;++n) {
a[i][n]=document.createElement("a");
a[i][n].href = links[i][n].href;
a[i][n].appendChild(document.createTextNode(links[i][n].text));
a[i][n].title = links[i][n].title;
p[i]=p[i]||p[i-1].cloneNode(0);
p[i].id = "js_footer_s"+(i+1);
p[i].appendChild(a[i][n]);
if(n<links[i].length-1) {
p[i].appendChild(document.createTextNode("|"));
}
}
document.getElementById("js_footer").appendChild(p[i]);
}
})("Copyright \xa9 _y_, Marie-Joe Restaurant - ");
What is so terrible about the design?Quote:
Originally Posted by Medyman
Thanks Master Scripter.
That works, one last thing though is there someway to my my link only a little bigger then the rest of it.
all you have to do is edit the css:Maybe:Code:#js_footer_s2 {
text-align: right;
float: right;
}
Code:#js_footer_s2 {
text-align: right;
float: right;
font-size: 10pt;
}
Thanks Script Master, by the way the other guy dose not seem to think to highly of my design, I'd like to get your oppinon though, what do you think?
Although design is not my specialty, I'd say it is a good layout, but not really appropriate for the nature of the website.
Ok, it's an ok site. I'd suggest a few things though:
Maybe on the home page, there could be a "Welcome to Mari-Joe's Restaurant's Website! [some text about the quality of our food and such]."
Also, instead of having a link "Specials," you could put a little side bar in that has the specials(of course, no specials for today. :()
Also, your text is over flowing your menu.
Try different colors too, for example the font colors. Maybe even change the font to arial, verdana, or something else?