[Kristoffer G]

ja alltså det va väl det du menade innan?

function getquerystring() {
var word = document.getElementById("feed").value
qstr = 'feed=' + escape(word);
return qstr;
}


<form name="feeds" action="index.html" method="POST">
<select id="feed">
<option value="">Välj</option>
<option value="http://www.geiser.se/feed" onClick='xmlhttpPost("show_feed.php")'>Geiser.se</option>
<option value="http://feeds.feedburner.com/tkjblogg?format=xml" onClick='xmlhttpPost("show_feed.php")'>TkJ</option>
<option value="http://www.dan.se/feed" onClick='xmlhttpPost("show_feed.php")'>Dan.se</option>
</select>

[tartareandesire]

Kan inte kolla så mycket mer noga nu, ett tips är att testa steg för steg och se var felet finns.

[Kristoffer G]

Känns som att ingenting skickas alls med IE eller Safari... alltså ingenting från formuläret behandlas.... det som är fel rättar firefox själv till..

[dAEk]

Hej och hallå,

• Ta bort onclick från dina options och ändra till onchange istället. Är osäker på hur olika webbläsare hanterar onclick på options men det vedertagna sättet att hantera ändring av selectlistor är onchange. Eller ännu bättre - lyssna inte på nåt event alls utan låt användaren själv bestämma när ändringen ska genomföras genom att stoppa in en submit. Använder man onchange blir det betydligt krångligare att hänga med på vad som händer när man navigerar m.h.a. tangentbordet och/eller skärmläsare.
  
• Värdet för select-listor hämtas genom att känna av vald rad och hämta dess värde. document.getElementById("feed").value är alltså fel eftersom select-elementet i sig inte har något värde.
  Ändra till nåt liknande:
  

  var form = document.getElementById("feeds");
  var feeds = form.elements["feed"];
  var selectedFeed = feeds.options[feeds.options.selectedIndex].value;

• Varför tittar du inte på de JS-libbar som finns? Du kommer spara in tid på det - garanterat. Är man inte så bra på Javascript blir koden lätt rörig och ostrukturerad. Plus att man slipper gå på samma nitar och buggar som andra redan gjort. Jag är inte så värst bra på Javascript själv men tycker att det blir betydligt bättre struktur om man använder t.ex. Prototype. I längden vill man ha ren, prydlig men främst av allt läsbar kod som är lätt att underhålla och då hjälper det ofta att använda sig av det gurusarna knåpat ihop.
  
  
• Sen tycker jag att du borde fundera två gånger till över hur du namnger dina id:n - "feed" är på tok generellt för min smak - men det kanske är subjektivt. I alla fall, det är ju en lista du har där så "availableFeeds" eller "feedList" vore mer beskrivande. Använder du upp generella termer kommer du sitta där en vacker dag och ångra dig när det verkar vettigare på en annan plats. Nåja, det kanske bara är jag som tänker för mycket igen.

HTH

[Kristoffer G]

Originally posted by dAEk@Jul 23 2008, 00:34
Hej och hallå,

• Ta bort onclick från dina options och ändra till onchange istället. Är osäker på hur olika webbläsare hanterar onclick på options men det vedertagna sättet att hantera ändring av selectlistor är onchange. Eller ännu bättre - lyssna inte på nåt event alls utan låt användaren själv bestämma när ändringen ska genomföras genom att stoppa in en submit. Använder man onchange blir det betydligt krångligare att hänga med på vad som händer när man navigerar m.h.a. tangentbordet och/eller skärmläsare.
• Värdet för select-listor hämtas genom att känna av vald rad och hämta dess värde. document.getElementById(feed).value är alltså fel eftersom select-elementet i sig inte har något värde.
  Ändra till nåt liknande:
  

  var form = document.getElementByIdfeeds;
  var feeds = form.elementsfeed;
  var selectedFeed = feeds.optionsfeeds.options.selectedIndex.value;

• Varför tittar du inte på de JS-libbar som finns? Du kommer spara in tid på det - garanterat. Är man inte så bra på Javascript blir koden lätt rörig och ostrukturerad. Plus att man slipper gå på samma nitar och buggar som andra redan gjort. Jag är inte så värst bra på Javascript själv men tycker att det blir betydligt bättre struktur om man använder t.ex. Prototype. I längden vill man ha ren, prydlig men främst av allt läsbar kod som är lätt att underhålla och då hjälper det ofta att använda sig av det gurusarna knåpat ihop.
  
• Sen tycker jag att du borde fundera två gånger till över hur du namnger dina id:n - feed är på tok generellt för min smak - men det kanske är subjektivt. I alla fall, det är ju en lista du har där så availableFeeds eller feedList vore mer beskrivande. Använder du upp generella termer kommer du sitta där en vacker dag och ångra dig när det verkar vettigare på en annan plats. Nåja, det kanske bara är jag som tänker för mycket igen.

HTH

- Snyggare utan submit-knapp
- Provade ändra allt det du sa, fungerar inte i någon webbläsare
- Vill inte använda ett färdigt paket... vill helst få detta att fungera



Edit:

Om någon har möjlighet till att testa allting så vore det bra... kanske är fel hos mig någonstans. MEN jag gissar på att ingenting kommer in i funktionerna från formuläret... får den inget kan den inget göra... och enda anledningen till att det fungerar i FF är att den rättar till problemet själv.

Har dessutom provat..
onclick
onchange
onfocus

alltid samma resultat

[dAEk]

Jag vet inte om det framgick men onchange ska alltså in på select - inte ersätta onclick på dina options. Använder du onchange behöver du ta bort onclick.

"Det fungerar inte", det säger oss ingenting. Vad får du för fel, vad händer/händer inte? IE är inte speciellt hjälpsam av sig men Opera och även Safari tror jag har en rätt hygglig felkonsol som kan berätta var det blir fel. Har du möjlighet att slänga upp en sida skulle det underlätta.

[Kristoffer G]

Nu börjar det likna någonting... jag får PHP error från alla webbläsare... detta error beror på att den inte får in någon url till klassen. url'en som ska in är den som finns i value="" i select.

Nu är vi nära

ser ut så här nu

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>Ajax</title>
<script type="text/javascript" src="js/fetch.js"></script>
</head>
<body>
<form id="feeds" action="index.html" method="post">
 <select id="feed" onchange='xmlhttpPost("show_feed.php")'> 
 <option value="">Välj</option>
 <option value="http://www.geiser.se/feed">Geiser.se</option>
 <option value="http://feeds.feedburner.com/tkjblogg?format=xml">TkJ</option>
 <option value="http://www.dan.se/feed">Dan.se</option>
 </select>
</form>
 <div id="result"></div>
</body>
</html>

function xmlhttpPost(strURL) {
 * *var xmlHttpReq = null;
try
 {
 xmlHttpReq = new XMLHttpRequest();
 }
catch (e)
 {
 try
 * {
 * xmlHttpReq = new ActiveXObject("Msxml2.XMLHTTP");

 * }
 catch (e)
 * {
 * xmlHttpReq = new ActiveXObject("Microsoft.XMLHTTP");

 * }
 }
	
 * *xmlHttpReq.open('POST', strURL, true);
 * *xmlHttpReq.setRequestHeader('Content-Type', 'application/x-www-form-urlencoded');
 * *xmlHttpReq.onreadystatechange = function() {
	
 * 
 * if (xmlHttpReq.readyState == 1) {
 * * * * * *updatepage('<br /><br /><img src="loader.gif" border="0" alt="" style="margin-left:15px;" />');
 * * * *}

 *
 * * * *if (xmlHttpReq.readyState == 4) {
 * * * * * *updatepage(xmlHttpReq.responseText);
 * * * *}
 * *}
 * *xmlHttpReq.send(getquerystring());
}

function getquerystring() {
var form = document.getElementById("feeds");
var feeds = form.elements["feed"];
var selectedFeed = feeds.options[feeds.options.selectedIndex].value;
return selectedFeed;
}


function updatepage(str){
 * *document.getElementById("result").innerHTML = str;
}

[dAEk]

Satte upp ett enkelt test och jag får ut vald url.

Fast det var med den gamla JS-koden ser jag nu.

[Kristoffer G]

YAY!! det fungerar nu.. va bara att lägga till

var selectedFeed = 'feed=' + feeds.options[feeds.options.selectedIndex].value;

alltså.. 'feed=' + ...

Tack ska ni ha alla som hjälpt mig... nu kan jag hålla på med det jag är bra på istället...PHP

[Kristoffer G]

hmm... en sak kvar. Måste luska ut hur man skippar <form> och istället anropar show_feed.php?feed=http://www.minfeed.se/feed direkt när sidan laddas.