Op deze website gebruiken we cookies om content en advertenties te personaliseren, om functies voor social media te bieden en om ons websiteverkeer te analyseren. Ook delen we informatie over uw gebruik van onze site met onze partners voor social media, adverteren en analyse. Deze partners kunnen deze gegevens combineren met andere informatie die u aan ze heeft verstrekt of die ze hebben verzameld op basis van uw gebruik van hun services. Meer informatie.

Akkoord

Vraag & Antwoord

Webdesign (HTML, CSS, Flash)

[Sitecheck] Metallica-Online.nl

None
18 antwoorden
  • Ik zou graag weten wat jullie van mijn site vinden, en wat er eventueel verbeterd aan kan worden. :D
    Het adres is: http://www.metallica-online.nl
  • Ik heb alleen even naar de tweede 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. W3.org geeft 135 errors, waaronder enkele ernstige, zoals fouten met body, head en een paar open tags.


    wimb
  • 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:
  • 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
  • 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
  • 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
  • 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.
  • :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?
  • 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…
  • En hoe betrouwbaar is die controle van Wc3?
    Want bij m'n standaard phpbb forum geeft hij ook bergen met fouten :-?
    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
  • 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: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
  • 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
    obert" 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
    ingtones" 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]
  • 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.
  • Oftewel ik moet de hele site opnieuw gaan opbouwen, en ik zet er voorlopig bij dat de site alleen is geschikt voor Internet Explorer.
  • 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.
  • OK, dankzij jullie zal ik m'n hele site overnieuw maken en wel geschikt maken voor alle webbrowsers.
    Bedankt voor de tips!
  • '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!

Beantwoord deze vraag

Dit is een gearchiveerde pagina. Antwoorden is niet meer mogelijk.