Ronald McDonald Huis dagboek site - Forum

Thijs X

08-09-2005 18:11 gewijzigd op 08-09-2005 18:15

Mooi gedaan en vrolijke kleuren. Wat wel toepasselijk is voor dit onderwerp.

Alleen zou ik dit nog even fixen: http://www.gastenboekrmhrotterdam.nl/faq/
iig ik neem aan dat hij in het midden moest zoals de rest van je site

En als je op die (?) plaatjes klikt krijg een popup waar een foutje in zit

H

Han eev

08-09-2005 18:44

Mooie kleuren idd! en verder leuke opmaak niks op aan t e merken

Jelmer -

08-09-2005 18:53

Nog wat schoonheidsfoutjes:

[item]Foute gebruikersnaam en wachtwoord staat nog in het engels[/item][item]E-cards kan ik 'succesvol versturen' waneer ik niets invul[/item][item]de link /search/, kan dat niet gewoon beter /zoeken/?[/item][item]Links de links naar 'veelgestelde vragen' en 'voorbeelden', ik dacht in eerste instantie dat dat 1 link was. Misschien wat ruimte ertussen[/item]

En er zit inderdaad nog een foutje in het help-popup pagina-dinges.

E

Eris -

08-09-2005 19:06

Wat het probleem van het laaste is weet ik ook niet.

Het wordt gewoon als "html"/plain weer gegeven ipv als php behandeld :|
<?php

$help = array();
$help[1] = 'Naam van de persoon waarover het gaat';
$help[2] = 'Achternaam van de persoon waar het over gaat';
$help[3] = 'Je eigen e-mailadres';
$help[4] = 'Je wachtwoord. Hoewel het is toegestaan dat je wachtwoord uit 1 of 2 tekens bestaat adviseren wij minimaal 6 tekens. Het gebruik van hoofdletters, kleine letters, cijfers en of leestekens door elkaar.';
$help[5] = 'Kies de toegangslevel. Wil je de de dagboek als een ouder gaan beheren kies dan "Ouder" en voor kind kies je "Kind".';
$help[6] = 'Geef het e-mailadres op dat je hebt gebruikt bij de aanmelding.';
$help[7] = 'Je eigen naam';
$help[8] = 'Je eigen e-mailadres. Je e-mailadres is alleen zichtbaar voor de mensen van wie het gastenboek is.';
$help[9] = 'Je wachtwoord. Dit is de code die je hebt opgegeven bij de aanmelding of die is genereert bij het opnieuw aanvragen van het wachtwoord.';
$help[10] = 'Plaats hier uw eigen bericht';
$help[11] = 'Vink deze checkbox aan om de foto te verwijderen';
$help[12] = 'Selecteer een foto';
$help[13] = 'U dient de eerste keer hier aan te melden voor het openen van een dagboek. Vervolgens kunt u zich aanmelden via “emailadres” en “wachtwoord”';

echo '<html>
<head>
<title>Help pagina</title>
<head>
<body onload="window.focus();">
'.$help[$_GET['id']].'</a>
<a href="" onclick="window.close()">Sluit venster</a>
</body>
</html>';
?>

S

Steff an

08-09-2005 21:24 gewijzigd op 08-09-2005 22:43

Ziet er goed uit, alleen zou ik een border aan de rechter en linkerkant doen. en de pagina uitlaten rekken naar beneden.

<head>
<title>Help pagina</title>
<head>
<body onload="window.focus();">
'.$help[$_GET['id']].'</a>
<a href="" onclick="window.close()">Sluit venster</a>
</body>
</html>';
?>

Dat zou ik niet zo doen, er wordt niet gecontroleerd of $_GET['id'] wel een nummer is en/of het wel in de array zit.

Alfred -

08-09-2005 22:13

Mooie site, fris ontwerp, alleen de form elementen zou ik nog opmaken... :) Login velden & knop dus...

Dat zou ik niet zo doen, er wordt nog gecontroleerd of $_GET['id'] wel een nummer is en/of het wel in de array zit.

Dit lijkt me totaal ongevaarlijk, dus kan op zich weinig kwaad denk ik.... :)

Jan Koehoorn

08-09-2005 22:28

Ik vind het ook een mooie frisse lay-out, kloppend bij de doelgroep.

Klein puntje: bij de e-cards krijg je als hover-tekst:
"Klik op de plaatje voor een groter foto"

Maak daar van:
"Klik op het plaatje voor een grotere foto"

E

Eris -

08-09-2005 23:58

Steffan schreef op 08.09.2005 21:24
Ziet er goed uit, alleen zou ik een border aan de rechter en linkerkant doen. en de pagina uitlaten rekken naar beneden.

<head>
<title>Help pagina</title>
<head>
<body onload="window.focus();">
'.$help[$_GET['id']].'</a>
<a href="" onclick="window.close()">Sluit venster</a>
</body>
</html>';
?>

Dat zou ik niet zo doen, er wordt niet gecontroleerd of $_GET['id'] wel een nummer is en/of het wel in de array zit.

Wat kan hier mis gaan.
Alles staat al gedefineerd in een array. Er is geen include of mysql injection toe pasbaar. Dus opzich is het geen potentieel gevaar. (Bij includes is het een ander verhaal. (Daar controleer ik ook het gevraagde bestand wel bestaat enz...))

Dus het is geen gevaar :)

B

Barman V

09-09-2005 00:09

Ik zie daar inderdaad helemaal geen gevaar in

Elwin - Fratsloos

09-09-2005 11:28

Jan Koehoorn schreef op 08.09.2005 22:28
Ik vind het ook een mooie frisse lay-out, kloppend bij de doelgroep.

Klein puntje: bij de e-cards krijg je als hover-tekst:
"Klik op de plaatje voor een groter foto"

Maak daar van:
"Klik op het plaatje voor een grotere foto"

Ja.. maar wat is het nou? Een foto of een plaatje? :) "Klik op het plaatje voor een vergroting." "Klik op de foto voor een vergroting."

Elwin