AyrshireMinis.com - Suggestions/Comments

crmpicco · 11-17-2007, 11:21 PM

Hi All,

I have set my site - www.ayrshireminis.com - live for about a fortnight now. It is written in PHP with a MySQL Database.

I would really appreciate it if some of you guys could take a look at it and let me know what you think of it, if you find any errors then let me know. Or if you notice something that isn't there that should be let me know.

Would really appreciate it if some of you registered for the forum and posted up a quick message :-)

Cheers Again!
Picco

www.ayrshireminis.com

srule_ · 11-18-2007, 06:01 AM

-You use tables for layout - not good...
-The design is not very exciting, you need some more of that web2.0 feel
-shout boxes are so out...
-I have a 21inch monitor and your site does not look to great on it. For example your main text area stretches out into 1 very long line of text.

twodayslate · 11-18-2007, 08:25 PM

Quote:

Originally Posted by srule_

-You use tables for layout - not good...
-The design is not very exciting, you need some more of that web2.0 feel
-shout boxes are so out...
-I have a 21inch monitor and your site does not look to great on it. For example your main text area stretches out into 1 very long line of text.

same here

layout is very dull. I like how you have an option for languages, however ditch the drop down box.

Apostropartheid · 11-18-2007, 09:27 PM

The HTML 4.0 standard is deprecated. Try using HTML 4.01 or switching to XHTML.

crmpicco · 11-18-2007, 10:50 PM

Thanks, yes I am going to switch over to HTML 4.01.

Thanks for those comments, have been looking into moving to "Web 2.0" with this site. It is definately something I wish to do. Drop-down box not likely to be used do you think?

Picco

srule_ · 11-19-2007, 06:13 PM

well, you have the flags to switch languages and the drop down box. You do not need both, so drop one of them

Apostropartheid · 11-19-2007, 07:27 PM

Huh.

Your URL for German is http://www.ayrshireminis.com/mini/de/index.php?lang=DE.

But your one for English is http://www.ayrshireminis.com/mini/index.php?lang=en.

If you're generating the pages dynamically, what's the point in the /mini/de/ folder?

crmpicco · 12-02-2007, 07:07 PM

No, the pages aren't generated dynamically. After doing a bit of research it seems the best way to do this was to have two directories for the different languages:

/en for English and /de for German

Picco

Apostropartheid · 12-02-2007, 07:23 PM

That's my point.

Why put "/index?lang=DE" if you're not generating at all? What's wrong with "/"? Why is there no /en/ directory?

crmpicco · 12-08-2007, 03:15 PM

hi CyanLight, i've taken your recommendation and moved to HTML 4.01 and it all validates:

http://www.ayrshireminis.com/

Da, Picco

_Aerospace_Eng_ · 12-21-2007, 04:38 PM

In agreement with the others except why do you use javascript for the color change on your navigation? It can be done in CSS and links.

You should read the link in my sig titled "Why Tables for Layout is Stupid".

in10siTY · 12-28-2007, 05:36 AM

try checking the code again... link is not opening here at my place...

crmpicco · 03-15-2008, 09:03 PM

After a few changes over the past few months (when I had the chance and time) I've changed the layout and general look and feel of the site. I still have a few CSS issues to sort out, and change the German version of the site to the new design - but, I would appreciate any comments on the site as it stands now:

http://www.ayrshireminis.com/

Obrigado,
Picco

Fumigator · 03-16-2008, 08:40 PM

I can't see the text of the link when I mouseover it.. it's a grey color that is swallowed in the green color. (Firefox 2.0.0.6)

The thumbnail images in the gallery really should be separate thumbnail-sized images, rather than the normal-sized image forced into a thumbnail size by way of the width and height attributes of the <img> tag. They will look much better and the page will load quickly.

The newsletter signup thingie tells me I've entered an invalid email address even when I've entered a valid email address. Also, there should be some mention there that to sign up for the newsletter, enter your email address.

Other than that... attractive colors, pleasant to look at. You aught to get away from using tables for layout though.

gnomeontherun · 03-16-2008, 08:41 PM

Here are a few things I noticed while browsing:

Your link rollover color is hard to read.

The logo doesn't appear on the forums?

You should resize the images you use as the thumbnails in the gallery, that way it doesn't take forever to load the thumbs, which you are actually just using the fullsized image scaled down. So it still has to load .5 MB per image just to see the thumb! It wasn't all of them, but people need fast loading pages.

Es gibt eine Fehler mit den detuschen Seiten. (Ich muss meine Deutsch probieren!)

11-17-2007, 11:21 PM	PM User \| #1
crmpicco Regular Coder Join Date: Jan 2005 Location: Mauchline, Scotland Posts: 986 Thanks: 14 Thanked 1 Time in 1 Post	AyrshireMinis.com - Suggestions/Comments Hi All, I have set my site - www.ayrshireminis.com - live for about a fortnight now. It is written in PHP with a MySQL Database. I would really appreciate it if some of you guys could take a look at it and let me know what you think of it, if you find any errors then let me know. Or if you notice something that isn't there that should be let me know. Would really appreciate it if some of you registered for the forum and posted up a quick message :-) Cheers Again! Picco www.ayrshireminis.com __________________ Ayrshire Minis - a Mini E-Community \| Ballochmyle Web Solutions

11-18-2007, 09:27 PM	PM User \| #4
Apostropartheid Comrade Grammar Nazi Join Date: Oct 2007 Location: Do you know where Nazi Zombies come from? Posts: 2,216 Thanks: 13 Thanked 182 Times in 180 Posts	The HTML 4.0 standard is deprecated. Try using HTML 4.01 or switching to XHTML. __________________ Cyan Light Industries \| It's Firefox, just easier on the eyes Follow me on Twitter

11-18-2007, 10:50 PM	PM User \| #5
crmpicco Regular Coder Join Date: Jan 2005 Location: Mauchline, Scotland Posts: 986 Thanks: 14 Thanked 1 Time in 1 Post	Thanks, yes I am going to switch over to HTML 4.01. Thanks for those comments, have been looking into moving to "Web 2.0" with this site. It is definately something I wish to do. Drop-down box not likely to be used do you think? Picco __________________ Ayrshire Minis - a Mini E-Community \| Ballochmyle Web Solutions

11-19-2007, 07:27 PM	PM User \| #7
Apostropartheid Comrade Grammar Nazi Join Date: Oct 2007 Location: Do you know where Nazi Zombies come from? Posts: 2,216 Thanks: 13 Thanked 182 Times in 180 Posts	Huh. Your URL for German is http://www.ayrshireminis.com/mini/de/index.php?lang=DE. But your one for English is http://www.ayrshireminis.com/mini/index.php?lang=en. If you're generating the pages dynamically, what's the point in the /mini/de/ folder? __________________ Cyan Light Industries \| It's Firefox, just easier on the eyes Follow me on Twitter

12-02-2007, 07:07 PM	PM User \| #8
crmpicco Regular Coder Join Date: Jan 2005 Location: Mauchline, Scotland Posts: 986 Thanks: 14 Thanked 1 Time in 1 Post	No, the pages aren't generated dynamically. After doing a bit of research it seems the best way to do this was to have two directories for the different languages: /en for English and /de for German Picco __________________ Ayrshire Minis - a Mini E-Community \| Ballochmyle Web Solutions

11-18-2007, 06:01 AM	PM User \| #2
srule_ Regular Coder Join Date: Jul 2007 Posts: 568 Thanks: 25 Thanked 28 Times in 28 Posts	-You use tables for layout - not good... -The design is not very exciting, you need some more of that web2.0 feel -shout boxes are so out... -I have a 21inch monitor and your site does not look to great on it. For example your main text area stretches out into 1 very long line of text.

11-19-2007, 06:13 PM	PM User \| #6
srule_ Regular Coder Join Date: Jul 2007 Posts: 568 Thanks: 25 Thanked 28 Times in 28 Posts	well, you have the flags to switch languages and the drop down box. You do not need both, so drop one of them

12-02-2007, 07:23 PM	PM User \| #9
Apostropartheid Comrade Grammar Nazi Join Date: Oct 2007 Location: Do you know where Nazi Zombies come from? Posts: 2,216 Thanks: 13 Thanked 182 Times in 180 Posts	That's my point. Why put "/index?lang=DE" if you're not generating at all? What's wrong with "/"? Why is there no /en/ directory? __________________ Cyan Light Industries \| It's Firefox, just easier on the eyes Follow me on Twitter

12-08-2007, 03:15 PM	PM User \| #10
crmpicco Regular Coder Join Date: Jan 2005 Location: Mauchline, Scotland Posts: 986 Thanks: 14 Thanked 1 Time in 1 Post	hi CyanLight, i've taken your recommendation and moved to HTML 4.01 and it all validates: http://www.ayrshireminis.com/ Da, Picco __________________ Ayrshire Minis - a Mini E-Community \| Ballochmyle Web Solutions

12-21-2007, 04:38 PM	PM User \| #11
_Aerospace_Eng_ Supreme Master coder! Join Date: Dec 2004 Location: In a place far, far away... Posts: 17,190 Thanks: 2 Thanked 732 Times in 713 Posts	In agreement with the others except why do you use javascript for the color change on your navigation? It can be done in CSS and links. You should read the link in my sig titled "Why Tables for Layout is Stupid". __________________ \|\|\|\|If you are getting paid to do a job, don't ask for help on it!\|\|\|\| Get FIREFOX.::.Why Tables for Layout is Stupid?.::.PR Designz.::.OpenDNS

12-28-2007, 05:36 AM	PM User \| #12
in10siTY Banned Join Date: Dec 2007 Posts: 2 Thanks: 0 Thanked 0 Times in 0 Posts	try checking the code again... link is not opening here at my place...

03-15-2008, 09:03 PM	PM User \| #13
crmpicco Regular Coder Join Date: Jan 2005 Location: Mauchline, Scotland Posts: 986 Thanks: 14 Thanked 1 Time in 1 Post	After a few changes over the past few months (when I had the chance and time) I've changed the layout and general look and feel of the site. I still have a few CSS issues to sort out, and change the German version of the site to the new design - but, I would appreciate any comments on the site as it stands now: http://www.ayrshireminis.com/ Obrigado, Picco __________________ Ayrshire Minis - a Mini E-Community \| Ballochmyle Web Solutions

03-16-2008, 08:40 PM	PM User \| #14
Fumigator Master Coder Join Date: Dec 2005 Location: Utah, USA, Northwestern hemisphere, Earth, Solar System, Milky Way Galaxy, Alpha Quadrant Posts: 5,411 Thanks: 32 Thanked 373 Times in 364 Posts	I can't see the text of the link when I mouseover it.. it's a grey color that is swallowed in the green color. (Firefox 2.0.0.6) The thumbnail images in the gallery really should be separate thumbnail-sized images, rather than the normal-sized image forced into a thumbnail size by way of the width and height attributes of the <img> tag. They will look much better and the page will load quickly. The newsletter signup thingie tells me I've entered an invalid email address even when I've entered a valid email address. Also, there should be some mention there that to sign up for the newsletter, enter your email address. Other than that... attractive colors, pleasant to look at. You aught to get away from using tables for layout though. __________________ I heart Hostmonster

03-16-2008, 08:41 PM	PM User \| #15
gnomeontherun Senior Coder Join Date: Sep 2007 Location: Houston Posts: 2,797 Thanks: 10 Thanked 235 Times in 226 Posts	Here are a few things I noticed while browsing: Your link rollover color is hard to read. The logo doesn't appear on the forums? You should resize the images you use as the thumbnails in the gallery, that way it doesn't take forever to load the thumbs, which you are actually just using the fullsized image scaled down. So it still has to load .5 MB per image just to see the thumb! It wasn't all of them, but people need fast loading pages. Es gibt eine Fehler mit den detuschen Seiten. (Ich muss meine Deutsch probieren!) __________________ jeremy - gnomeontherun Educated questions often get educated answers, and simple questions often get simple answers. Last edited by gnomeontherun; 03-16-2008 at 08:45 PM..

Thread Tools
Show Printable Version Email this Page
Rate This Thread
Rate This Thread: