javascript uitklap menu

donderdag 11 maart 2010 09:20

Acties:

0 Henk 'm!

Verwijderd

Topicstarter

hallo tweakers,

ten eerste mijn excuses als dit nogal een domme vraag is, ik zit nu zo'n klein jaar in het programmeren, en ben nooit echt een held geweest in het maken van javascript code.
het probleem is dat ik een uitklap menu heb gemaakt die in een div moet openen.
om dat te bereiken heb ik in de css alles met de sub als id de waarde display:none; gegeven.

nu zit er rond klikbare text een span met onclick welke mijn javascript activeerd.
hieronder is de code:

HTML:

<html>
<head>
<script type="text/javascript">
function displayswitch(sub)
{
var object = document.getElementById(sub);
if (object.style.display == "none")
{object.style.display = "inline";}
else
{object.style.display = "none";}
if (object.style.display == "none")
{ echo ("fout"); }
}

</script>
<link rel="stylesheet" href="CSS/divs.css" type="text/css" >
<link rel="stylesheet" href="CSS/menu.css" type="text/css" >
</head>
<body>
<!-- begin top menu -->
<div class="top">
menu
</div>
<!-- einde top menu --> <br>
<table class="splitter">
<tr>
<td>
<img src="plaatjes/divmen.png"><br />
<!--begin menu-->
<div class="menu">
<span class="textlayer">
<img src="plaatjes/12point.png" /><span onclick="displayswitch(sub);" /><b>compleet systeem</b></span><br />
<div id="sub"> budget <br /> all-round <br /> gamer <br /> mediacentrum <br /></div>

dit is niet de volledige pagina, maar de rest is alleen maar herhaling (m.u.v de span, die is alleen bovenaan voor de test)
de css van het #sub stukje is

Cascading Stylesheet:

#sub {
display: none;
position: relative;
margin-left: 15px;}

hopelijk hebben jullie meer geluk met dit geval als ik, want ik zit er al een dagje of drie aan vast

[ Voor 0% gewijzigd door RobIII op 11-03-2010 09:39 . Reden: Code tags toegevoegd. ]

donderdag 11 maart 2010 09:26

Acties:

0 Henk 'm!

DanielG

i = 0x5f3759df - (i>>1); ☠₧ℳ🀪❣

HTML:

1	<span onclick="displayswitch(sub);" />

Je moet even doorhebben waarom dit niet werkt, specifiek wat je nou precies probeert door te geven aan de functie displayswitch.

http://xyproblem.info/

donderdag 11 maart 2010 09:27

Acties:

0 Henk 'm!

macciez

Verwijderd schreef op donderdag 11 maart 2010 @ 09:20:

ten eerste mijn excuses als dit nogal een domme vraag is,
[...]
hopelijk hebben jullie meer geluk met dit geval als ik, want ik zit er al een dagje of drie aan vast

edit: zie dat je al hulp hebt gevonden

[ Voor 23% gewijzigd door macciez op 11-03-2010 09:28 ]

Do what you love, do it often

donderdag 11 maart 2010 09:38

Acties:

0 Henk 'm!

RobIII

Admin Devschuur®

^ Romeinse Ⅲ ja!

Css
Javascript

Als je code post, gebruik dan code tags a.u.b.

There are only two hard problems in distributed systems: 2. Exactly-once delivery 1. Guaranteed order of messages 2. Exactly-once delivery.

Je eigen tweaker.me redirect

Over mij

donderdag 11 maart 2010 09:45

Acties:

0 Henk 'm!

DanielG

i = 0x5f3759df - (i>>1); ☠₧ℳ🀪❣

JavaScript:

1
2
3

if (object.style.display == "none")
{ echo ("fout"); }
}

echo is geen standaard javascript functie (denk dat je door de war bent met PHP).
Plus kijk dan ook meteen even waarom dit altijd aangeroepen gaat worden als je je menu dichtklapt.

http://xyproblem.info/

donderdag 11 maart 2010 10:14

Acties:

0 Henk 'm!

MueR

Admin Tweakers Discord

is niet lief

Je wil misschien nog eens kijken naar je HTML. Die tabellen zijn niet nodig, en je menu opbouwen uit spans is heel onverstandig. Google wil nog steeds geen javascript lezen, dus dat menu gaat totaal niks voor je doen.

Je hebt overigens vast ook een foutmelding van je javascript console of firebug?

Anyone who gets in between me and my morning coffee should be insecure.

donderdag 11 maart 2010 11:37

Acties:

0 Henk 'm!

Verwijderd

Topicstarter

ok, alvast bedankt zo ver

dat echo is mijn fout wel, een restant van proberen te ontdekken waar de fout zit.
ik heb nu al lessen javascript gevolgd, maar direct daar na kwam php, dus de verwarring wil wel is toe slaan.

de span haal ik wel weg, daar kan ik een klein plusje (als img) voor gebruiken.
ik heb de fout nu gevonden, het sub moet ook tussen '' staan

donderdag 11 maart 2010 21:33

Acties:

0 Henk 'm!

pieturp

gaffa!

Overigens is 't raadzaam om een goede indent style te leren gebruiken. Welke stijl je dan kiest maakt niet zoveel uit, maar niet indenten is echt niet handig. Het scheelt enorm in leesbaarheid van je code, en voorkomt daardoor fouten. Zeker als je scripts groter worden echt onmisbaar.

Meest voorkomend is 1TBS "The One True Brace Style". Het is misschien eerst even wennen, maar uiteindelijk wil je niets anders meer

... en etcetera en zo

vrijdag 12 maart 2010 22:30

Acties:

0 Henk 'm!

Verwijderd

Topicstarter

ik heb ondertussen ook een indent style gemaakt van deze pagina, hij is nog vele stukken langer dan dat hier staat, en ik gebruik deze al langer voor PHP.
maar toch bedankt ;P

Onderwerpen