Ga naar inhoud

[Sitecheck] Metallica-Online.nl


anoniem

Aanbevolen berichten

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
Link naar reactie
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:
Link naar reactie
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
Link naar reactie
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]
Link naar reactie
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.
Link naar reactie
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.
Link naar reactie
  • 2 weken later...

Om een reactie te plaatsen, moet je eerst inloggen

Gast
Reageer op dit topic

×   Geplakt als verrijkte tekst.   Herstel opmaak

  Er zijn maximaal 75 emoji toegestaan.

×   Je link werd automatisch ingevoegd.   Tonen als normale link

×   Je vorige inhoud werd hersteld.   Leeg de tekstverwerker

×   Je kunt afbeeldingen niet direct plakken. Upload of voeg afbeeldingen vanaf een URL in

  • Populaire leden

    Er is nog niemand die deze week reputatie heeft ontvangen.

  • Leden

    Geen leden om te tonen

×
×
  • Nieuwe aanmaken...