anoniem Geplaatst: 19 december 2003 Delen Geplaatst: 19 december 2003 Ik zou graag weten wat jullie van mijn site vinden, en wat er eventueel verbeterd aan kan worden. :D Het adres is: [url]http://www.metallica-online.nl[/url] Quote Link naar reactie
anoniem Geplaatst: 19 december 2003 Auteur Delen Geplaatst: 19 december 2003 Ik heb alleen even naar de [url=http://www.metallica-online.nl/index2.php]tweede[/url] pagina gekeken. Het uiterlijk ziet er wel aardig uit, maar aan de html valt nog wel wat te verbeteren. met Mozilla komt er maar een halve pagin in beeld. [url=http://validator.w3.org/check?verbose=1&uri=http%3A//www.metallica-online.nl/index2.php]W3.org[/url] geeft 135 errors, waaronder enkele ernstige, zoals fouten met body, head en een paar open tags. wimb Quote Link naar reactie
anoniem Geplaatst: 19 december 2003 Auteur Delen Geplaatst: 19 december 2003 Als ik zo ff naar de opstelling etc. kijk, dan vind ik het er verdder wel goed uitzien. Alleen wat mij een beetje tegenvalt is, dat je bij www.metallica-online.nl , en dan de hoofdpagina, dat zo'n plaatje een beetje een moeilijk zichtbare link is. Waarom plaats je ook niet een gewone link eronder, zodat je op het plaatje en op de link kunt klikken. P.S. Verder vind ik het wel een mooie site!! Je hebt goed werk verricht :lol: :lol: Quote Link naar reactie
anoniem Geplaatst: 19 december 2003 Auteur Delen Geplaatst: 19 december 2003 Nette site, heel erg netjes. Mooi design. Alleen jammer dat de poll niet laad in Opera. Verder, zoals WimB als zei, aardig wat foutjes. Ook hele erge. Maar als je die oplost, zal het zeker een verademing zijn om je site te bezoeken :). Jochem Quote Link naar reactie
anoniem Geplaatst: 19 december 2003 Auteur Delen Geplaatst: 19 december 2003 Ok ik zie dat er veel fouten zijn in andere browsers, niet dat ik lui ben ofso, maar waarom zou ik het gaan verbeteren als eigelijk iedereen Internet Explorer gebruikt? En trouwens ook bedankt voor de positiefe reacties voor de ley-out, heeft me veel tijd gekost :D Quote Link naar reactie
anoniem Geplaatst: 19 december 2003 Auteur Delen Geplaatst: 19 december 2003 Niet iedereen gebruikt IE. Uit statistieken van mijn eigen site (maar dan praat je over een webdesign volkje, en dat wijkt een beetje af qua browser gebruik) gebruikt iets meer 64% IE. De rest dus mozilla/opera/safari/iets anders dus. Maar wat ik wil zeggen, je kan je site wel leuk compatible voor IE alleen maken, maar daarbij schrik je dus minimaal 10% van je potentiele bezoekers af. Dit kan je op lossen door gewoon simpel weg de fouter eruit te halen (en zeg nou zelf, een fout loze pagina geeft toch veel meer bevrediging dan een pagina die alleen werkt met IE6?). Jochem Quote Link naar reactie
anoniem Geplaatst: 19 december 2003 Auteur Delen Geplaatst: 19 december 2003 Dat is waar, maar ik weet niet hoe ik de fouten eruit moet halen ps. ik ben geen beginner met html hoor, alleen met php kan ik alleen nog maar basis dingen. Quote Link naar reactie
anoniem Geplaatst: 20 december 2003 Auteur Delen Geplaatst: 20 december 2003 :roll: [code:1:737f4884d6]<DIV id="pagina" > <table width="100%" border="0" cellpadding="00"> <tr> <td> <link rel="stylesheet" content="text/css" href="style.css"><head> </head>[/code:1:737f4884d6]Weet je dat wel zeker dat je geen beginnner bent? Quote Link naar reactie
anoniem Geplaatst: 20 december 2003 Auteur Delen Geplaatst: 20 december 2003 Dat weet ik wel zeker ja, hoe wou ik anders deze site en lay-out maken? Maar ik zal dus de hele broncode wel ff controleren. EDIT: Op welke pagina zie jij die fouten dan van DIV in de HEAD? Ik kan het nergens vinden hoor... Quote Link naar reactie
anoniem Geplaatst: 20 december 2003 Auteur Delen Geplaatst: 20 december 2003 En hoe betrouwbaar is die controle van Wc3? Want bij m'n standaard phpbb forum geeft hij ook bergen met fouten :-? [url]http://validator.w3.org/check?uri=http%3A%2F%2Fwww.metallica-online.nl%2Fforum%2F&charset=%28detect+automatically%29&doctype=%28detect+automatically%29&verbose=1[/url] Quote Link naar reactie
anoniem Geplaatst: 20 december 2003 Auteur Delen Geplaatst: 20 december 2003 W3c is de organisatie die de webstandaarden bepaald, en dus altijd correct. Verder zit het forum ook vol met fouten, maar vaak zijn die niet zo erg dat een pagina niet meer correct gedisplayed kan worden. Jochem Quote Link naar reactie
anoniem Geplaatst: 20 december 2003 Auteur Delen Geplaatst: 20 december 2003 [quote:e05d58d4d5="Hari-Bo"]Dat weet ik wel zeker ja, hoe wou ik anders deze site en lay-out maken?[/quote:e05d58d4d5]Dreamweaver? Frontpage? Er klopt namelijk helemaal niks van je gehele broncode dus dat dacht ik in eerste instantie. Fouten waren hier gevonden: http://www.metallica-online.nl/index2.php Quote Link naar reactie
anoniem Geplaatst: 20 december 2003 Auteur Delen Geplaatst: 20 december 2003 Ik heb de site gemaakt met DreamWeaver MX2004, maar ik vind het vaag want die fouten die jullie aangeven zie ik er niet staan :-? Dit is de broncode van index2.php: [code:1:0e4ac02dea]<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd"> <html> <head> <title>Metallica-Online.nl ,De Nederlandse #1 Metallica site!</title> <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1"> <link rel="stylesheet" href="metallica-online.css" type="text/css"> <style type="text/css"> <!-- body { background-color: #000000; } --> </style></head> <body class="metallica-online"> <DIV id="menu" ><IMG SRC="menu.jpg" usemap="#menu" BORDER="0"> <MAP NAME="menu"> <AREA HREF="index2.php?pagina=biografie/james" ALT="Lees hier de complete biografie van James Hetfield" SHAPE="rect" COORDS="381,9,454,114"> <AREA HREF="index2.php?pagina=biografie/kirk" ALT="Lees hier de complete biografie van Kirk Hammet" SHAPE="rect" COORDS="475,9,548,114"> <AREA HREF="index2.php?pagina=biografie/lars" ALT="Lees hier de complete biografie van Lars Ulrich" SHAPE="rect" COORDS="569,8,642,113"> <AREA HREF="index2.php?pagina=biografie/robert" ALT="Lees hier de complete biografie van Robert Truijlo" SHAPE="rect" COORDS="662,9,735,114"> <AREA HREF="index2.php?pagina=nieuws/nieuwssysteem" ALT="Lees hier het laatste nieuws over Metallica!" SHAPE="rect" COORDS="27,163,92,183"> <AREA HREF="http://www.metallica-online.nl/forum" TARGET="blank" ALT="Discusieer hier over Metallica en dergelijke" SHAPE="rect" COORDS="110,163,171,183"> <AREA HREF="index2.php?pagina=albums/albums" ALT="Een lijst van alle officiële albums" SHAPE="rect" COORDS="189,163,257,183"> <AREA HREF="index2.php?pagina=lyrics/lyrics" ALT="Alle muziekteksten van Metallica" SHAPE="rect" COORDS="272,163,332,183"> <AREA HREF="index2.php?pagina=tabs/tabs" ALT="Tabs om Metallica nummers te kunnen spelen" SHAPE="rect" COORDS="346,163,396,183"> <AREA HREF="index2.php?pagina=tour/tour" ALT="Aankomende tour datums, oude tour datums" SHAPE="rect" COORDS="411,163,463,183"> <AREA HREF="index2.php?pagina=ringtones/ringtones" ALT="Metallica op je mobiel!" SHAPE="rect" COORDS="480,163,564,183"> <AREA HREF="index2.php?pagina=ecards/ecard" ALT="Metallica ecards versturen" SHAPE="rect" COORDS="578,163,642,183"> <AREA HREF="index2.php?pagina=contact" ALT="Opmerkingen of vragen?" SHAPE="rect" COORDS="657,163,729,183"> </MAP></DIV> <DIV id="pagina" > <table width="100%" border="0" cellpadding="00"> <tr> <td> <?php // ----------------------------------------- // Verander de variabelen hieronder $StandaardMap = "/home/virtual/site107/fst/var/www/html/"; // De standaardmap waar de pagina's in staan. $StandaardExtensie = ".php"; // De standaardextensie voor de pagina's. // Als je hier dus .php invult wordt als je // ?pagina=pagina dus pagina.php geïncludeerd // Vul je /?pagina=pagina.html in dan wordt // pagina.html gewoon geïncluded. $StandaardBestand = "nieuws/nieuwssysteem.php"; // Het bestand dat standaard moet worden geïncludeerd $FoutBestand = "fout.php"; // Het bestand voor als er een fout optreedt // Na hier hoef je niets te veranderen // ----------------------------------------- // Als $pagina geen waarde heeft... if(!$pagina) { include("$StandaardMap$StandaardBestand"); } // Beveiliging... elseif(strstr($pagina,"admin") || strstr($pagina,"index") || strstr($pagina,"..")) { include("$StandaardMap/$FoutBestand"); } // Als $pagina wel een waarde heeft en het bestand met de extensie $StandaardExtensie erachter bestaat... elseif(file_exists("$StandaardMap$pagina$StandaardExtensie")) { include("$StandaardMap$pagina$StandaardExtensie"); } // Als $pagina wel een waarde heeft en het bestand zonder de extensie $StandaardExtensie erachter bestaat... elseif(file_exists("$StandaardMap$pagina")) { include("$StandaardMap$pagina"); } // Als $pagina wel een waarde heeft en het bestand niet bestaat... else { include("$StandaardMap/$FoutBestand"); } ?></td> </tr> </table> </DIV> <DIV id="forum"> <table width="210" height="275" border="0" cellpadding="00" background="forum.jpg"> <tr> <td> <?php /////////////////////////////////////////////////////////////////////////////// // ACTIVE_TOPICS.PHP /////////////////////////////////////////////////////////////////////////////// // Copyright: (C) 2002 Matthijs van de Water <matthijs@beryllium.net> // Version: 1.1 // Date: 03/02/2002 /////////////////////////////////////////////////////////////////////////////// // Show phpBB 2.0 Active Topics List // Output format can be any HTML or XML // This script must be able to access vital phpBB 2.0 configuration scripts /////////////////////////////////////////////////////////////////////////////// /////////////////////////////////////////////////////////////////////////////// // CUSTOM SETTINGS /////////////////////////////////////////////////////////////////////////////// // Amount of active topics to show define("TOPIC_COUNT", 10); // Path to the phpBB 2.0 root directory define("PHPBB_PATH", "/home/virtual/site107/fst/var/www/html/forum/"); // URL to the phpBB 2.0 installation define("PHPBB_LOCATION", "http://www.metallica-online.nl/forum/"); // Time format to output the date/time (for format see PHP manual) define("TIME_FORMAT", "H:i"); /////////////////////////////////////////////////////////////////////////////// // Includes of phpBB scripts $phpbb_root_path = PHPBB_PATH; if ( !defined('IN_PHPBB') ) { define('IN_PHPBB', true); include(PHPBB_PATH . 'extension.inc'); include(PHPBB_PATH . 'config.'.$phpEx); include(PHPBB_PATH . 'includes/constants.'.$phpEx); include(PHPBB_PATH . 'includes/db.'.$phpEx); } /////////////////////////////////////////////////////////////////////////////// // HTML header start /////////////////////////////////////////////////////////////////////////////// ?> <link rel="stylesheet" href="metallica-online.css" type="text/css"> <table valign="top" align="center" border="0" cellpadding="0" cellspacing="3" class="metallica-online" height="240" width="200"> <?php /////////////////////////////////////////////////////////////////////////////// // HTML header end /////////////////////////////////////////////////////////////////////////////// // sql statement to fetch active topics of public forums $sql = "SELECT DISTINCT t.topic_title, t.topic_last_post_id, p.post_time, f.forum_name FROM " . TOPICS_TABLE . " AS t, " . POSTS_TABLE . " AS p, " . FORUMS_TABLE . " AS f WHERE t.forum_id = f.forum_id AND f.auth_view = " . AUTH_ALL . " AND p.topic_id = t.topic_id AND p.post_id = t.topic_last_post_id ORDER BY p.post_time DESC LIMIT " . TOPIC_COUNT; $nt_result = $db->sql_query($sql); if(!$nt_result) { die("Failed obtaining list of active topics".mysql_error()); } else { $nt_data = $db->sql_fetchrowset($af_result); } if ( count($nt_data) == 0 ) { die("No topics found"); } else { // $nt_data contains all interesting data for ($i = 0; $i < count($nt_data); $i++) { $title = $nt_data[$i]['topic_title']; $url = PHPBB_LOCATION . 'viewtopic.' . $phpEx . "?" . POST_POST_URL . "=" . $nt_data[$i]['topic_last_post_id'] . "#" . $nt_data[$i]['topic_last_post_id']; $on_forum = 'On the ' . $nt_data[$i]['forum_name'] . ' forum'; $post_time = date(TIME_FORMAT, $nt_data[$i]['post_time']); // As of now you can actually do anything with the data // I chose to output in XML /////////////////////////////////////////////////////////////////////////////// // Item HTML start /////////////////////////////////////////////////////////////////////////////// ?> <tr> <td width="25%"><?php echo $post_time; ?> <div align="right"></div></td> <td width="75%"><b><a href="<?php echo $url; ?>" target="blank" title="<?php echo $on_forum; ?>"><?php echo $title; ?></a></b></td> </tr> <?php /////////////////////////////////////////////////////////////////////////////// // Item HTML end /////////////////////////////////////////////////////////////////////////////// } } /////////////////////////////////////////////////////////////////////////////// // Footer HTML start /////////////////////////////////////////////////////////////////////////////// ?> </table> <?php /////////////////////////////////////////////////////////////////////////////// // Footer HTML end /////////////////////////////////////////////////////////////////////////////// // EOF ?></td> </tr> </table> </DIV> <DIV id="poll" > <table width="210" height="200" border="0" cellpadding="00" background="poll.jpg"> <tr> <td><table width="80%" align="center" border="0" cellpadding="00"> <tr> <td> <?php require '/home/virtual/site107/fst/var/www/html/sympoll/booth.php'; display_booth(4); ?></td> </tr> </table> </td> </tr> </table> </DIV> <DIV id="reclame" ><IFRAME SRC="http://www2.kliks.nl/dvdbox.php?63hbos" frameborder="no" border="0" MARGINWIDTH="0" MARGINHEIGHT="0" SCROLLING="no"WIDTH=150 HEIGHT=70></IFRAME><br> <a href="http://www.kliks.nl/stats2/statischklik.php?63hbos&gsm&button" target=new><img src="http://www.kliks.nl/stats2/statisch.php?63hbos&gsm&button" width=120 height=60 border=0 alt="Adverteren via Kliks.nl"></A><br> <a href="http://www.metallica-online.nl/index2.php?pagina=adverteren">adverteren?</a><br> </DIV> </body> </html>[/code:1:0e4ac02dea] Quote Link naar reactie
anoniem Geplaatst: 20 december 2003 Auteur Delen Geplaatst: 20 december 2003 Euh, je werkt met PHP, dus include je uiteraard wat extra templates etc. Wellicht tijd om de dit is door te spitten:[code:1:a36ef83dd2]if(!$pagina) { include("$StandaardMap$StandaardBestand"); } // Beveiliging... elseif(strstr($pagina,"admin") || strstr($pagina,"index") || strstr($pagina,"..")) { include("$StandaardMap/$FoutBestand"); } // Als $pagina wel een waarde heeft en het bestand met de extensie $StandaardExtensie erachter bestaat... elseif(file_exists("$StandaardMap$pagina$StandaardExtensie")) { include("$StandaardMap$pagina$StandaardExtensie"); } // Als $pagina wel een waarde heeft en het bestand zonder de extensie $StandaardExtensie erachter bestaat... elseif(file_exists("$StandaardMap$pagina")) { include("$StandaardMap$pagina"); } // Als $pagina wel een waarde heeft en het bestand niet bestaat... else { include("$StandaardMap/$FoutBestand"); } [/code:1:a36ef83dd2]Daar staan dus een aantal $variabelen die verwijzen naar mappen die in het stukje code beschreven staan, daarin zitten dus de vele fouten. Hoewel de code die je hier plakt ook niet al te best is. Quote Link naar reactie
anoniem Geplaatst: 20 december 2003 Auteur Delen Geplaatst: 20 december 2003 Oftewel ik moet de hele site opnieuw gaan opbouwen, en ik zet er voorlopig bij dat de site alleen is geschikt voor Internet Explorer. Quote Link naar reactie
anoniem Geplaatst: 20 december 2003 Auteur Delen Geplaatst: 20 december 2003 Zou best kunnen, ik heb geen idee van je achterliggende code etc. Maar dit zijn dingen die je voordat je aan de site begint duidelijk moet hebben. Wat wordt de structuur, hoe komt de markup er dus uit te zien. Wat ga ik waarvoor gebruiken etc. En dan correct opbouwen ipv, elementje hier even kijken in IE, ziet het er goed uit? Nee, nog even een spatie erbij en dan lijkt het iets beter, etc. Dat is dus _niet_ de manier om een site op te zetten (alhoewel het wel vaak wordt gedaan). Als het ontwerp bekend is, kijk ik meestal eerst hoe ik het precies wil gaan aanpakken, op welke ondersteuning van browsers ik kan rekenen. Dan zet ik globaal de structuur op. Dan pak ik CSS erbij, zet alles op de goede plek, positioneer de plaatjes, etc. Eventueel krijgen sommigen elementen een class, indien dat nodig is (bijna nooit). Uiteraard zijn er meerdere wegen die naar Rome leiden, maar alleen het visuele resultaat bekijken geeft zeker geen goede site. Quote Link naar reactie
anoniem Geplaatst: 21 december 2003 Auteur Delen Geplaatst: 21 december 2003 OK, dankzij jullie zal ik m'n hele site overnieuw maken en wel geschikt maken voor alle webbrowsers. Bedankt voor de tips! Quote Link naar reactie
anoniem Geplaatst: 3 januari 2004 Auteur Delen Geplaatst: 3 januari 2004 'De' website ziet er erg mooi uit, maar bij de 'splash=page' gaat de kwaliteit erg omlaag door die lelijke GSMPlaza.nl banner. Weghalen dus! Quote Link naar reactie
Aanbevolen berichten
Om een reactie te plaatsen, moet je eerst inloggen