Onclick

Det här forumet visas bäst i en modern webbläsare med JavaScript påslaget.

Evl

Hejsan.

Håller på med lite javascript och ajax nu och det flyter på bra men när jag gör en button med onclick fungerar det bara i IE inte i Safari eller Firefox.

Någon som har ett svar på vad man ska göra? Inte första gången det inte fungerar.

Arood

Jag tror vi behöver lite mer kod att titta på. Finns en hel del som kan vara fel.

Evl

<textarea name="frm_chatEntry" style="width:90%; height:65px;"></textarea><br />
 <input type="button" value="Skicka" onclick="javascript:makeRequest('addEntry.asp?chatText='+ document.getElementById('frm_chatEntry').value +'&rnd='+Math.random(),'overflow');" />

Det är koden för själva knappen och formuläret, och ajaxen fungerar som den ska för den används till andra funktioner.

Arood

Ta bort

javascript:

. Den användes för åratal sen när man använde javascript i href-attributet.

Nu för tiden använder man knappt onClick, man föredrar att använda JavaScript helt och hållet i externa filer, men det är möjligtvis överkurs än så länge.

Kanin

Överkurs eller inte, att ha all Javascript i en extern fil är något man verkligen borde satsa på från start. Och anledningen är precis densamma som varför man inte borde köra med style-taggar till CSS:en; man vill ha utseende, innehåll och uppförande separerat så gott det går. Vi lever inte på 90-talet längre!

I så fall blir koden du skrev där uppe såhär:

javascript.js

var inputButton = 
{
   clickHandler: function() {
      makeRequest('addEntry.asp?chatText='+ document.getElementById('frm_chatEntry').value +'&rnd='+Math.random(),'overflow');
   }
};

var button = document.getElementById("sendButton");
button.onclick = inputButton.clickHandler;

index.html

...

<textarea name="frm_chatEntry" style="width:90%; height:65px;">
 <input id="sendButton" type="button" value="Skicka" />

...

Det blir genast mer överskådligt enligt mig.

(Har inte orkat testa koden själv, så var inte förvånad om den inte fungerar... Var ändå mest för att visa hur det hela fungerar)

Evl

Tackar ska kolla om det fungerar!

Johan

Kanin skrev:
var inputButton = 
{
   clickHandler: function() {
      makeRequest('addEntry.asp?chatText='+ document.getElementById('frm_chatEntry').value +'&rnd='+Math.random(),'overflow');
   }
};

Det är väl ganska onödigt med ett JSON bara för den lilla? Självklart beror det på resten, men om det är hela skriptet kan man lika gärna sätta clickhandlern direkt. Förresten är det antagligen bättre att köra addEventListener och attachEvent för mer struktur - jag har i alla fall aldrig gillat att lägga till eventlyssnare på det sättet.

Evl tänk på att sätta onclick:en efter att DOM:et har laddat klart, javascript.js ska alltså innehålla en typ av ondomready eller onload (helst ondomready).

ice

Använd JQuery

$(document).ready(function(){
      
    $('#button').click(function() {
        alert('Hey World!');
    });      

});

Johan

Det här inlägget blev en egen tråd och hamnade här: http://w3sidan.net/forum/visa/1018 (på monkeycids begäran).

monkeycid

"Jag ursäktar för vad jag antar var lite off topic, men jag tyckte en egen tråd för det bara var onödigt, sorry för att jag skräpar ner din tråd Evl."

Knappast! Skapa gärna en egen tråd, så kan vi diskutera - jag håller nämligen inte med alls.