``` men det verkar inte som det funkar , det går inte anropa andra css filer (tex pink.css) sidan använder fortfarande index.css , vet du vart felet är ? tackar !! :) Permalänk: https://www.webforum.nu/p/2283480 ## #9 — voigtann1, 2011-09-26T16:49Z din HTML markup för link-elementet är fel Sen förstå jag inte alls switchStyleOfUser() varför du vill köra det på det sättet i huvudtaget, du kör en document.write i metoden och sen kör du alltid en document.write på din: ``` document.write(switchStyleOfUser()); ``` Du skulle mycket hellre göra en metod anrop som gör allt åt dig, istället för att köra samma kod om och om igen, och hellre köra en array och ännu bättre, inte använda så många globala variabler. Har du rätt URL på din variabel \`urlToCSSDirectory\`? När jag försöker köra din kod: > urlToCSSTB is not defined Ändra din metod så kommer det fungera. Permalänk: https://www.webforum.nu/p/2283482 ## #10 — vagann, 2011-09-26T17:02Z > **[voigtann1 skrev:](https://www.webforum.nu/p/1540460)** > > din HTML markup för link-elementet är fel > > Sen förstå jag inte alls switchStyleOfUser() varför du vill köra det på det sättet i huvudtaget, du kör en document.write i metoden och sen kör du alltid en document.write på din: > > > > ``` > document.write(switchStyleOfUser()); > ``` > > > > Du skulle mycket hellre göra en metod anrop som gör allt åt dig, istället för att köra samma kod om och om igen, och hellre köra en array och ännu bättre, inte använda så många globala variabler. > > Har du rätt URL på din variabel \`urlToCSSDirectory\`? > > När jag försöker köra din kod: > > > Ändra din metod så kommer det fungera. ja, har rätt url på urlToCSSDirectory och sedan urlToCSSTB behöver jag inte den så kommer att ta bort den och som sagt , jag kan nästan ingenting om JS , så kan du inte bara modifera den och sedan ge mig den rätta koden , enligt dig så är koden inte alls rätt/ bra , kanske du kunde göra det på ditt sätt ? tack för du hjälper mig , väldigt snällt av dig :) Permalänk: https://www.webforum.nu/p/2283484 ## #11 — voigtann1, 2011-09-26T17:20Z > **[vagann skrev:](https://www.webforum.nu/p/1540462)** > > ja, har rätt url på urlToCSSDirectory och sedan urlToCSSTB behöver jag inte den så kommer att ta bort den och som sagt , jag kan nästan ingenting om JS , så kan du inte bara modifera den och sedan ge mig den rätta koden , enligt dig så är koden inte alls rätt/ bra , kanske du kunde göra det på ditt sätt ? > tack för du hjälper mig , väldigt snällt av dig :) Klart man kan göra den bättre på många sätt, men att bara ge dig en ändrad kod kommer ju inte hjälpa dig att lära dig javascript alls. Om du inte få igång det efter du har plockat bort urlToCSSTB så få du gärna postar din nya kod och berätta vad som inte fungera. Fungerar för mig på: Permalänk: https://www.webforum.nu/p/2283486 ## #12 — vagann, 2011-09-26T18:21Z ohh shiit va nice , efter att jag plockade bort urlToCSSTB blev koden så här : ``` // These are the variables; you can change these if you want var expDays = 9999; // How many days to remember the setting var standardStyle = '5'; // This is the number of your standard style sheet; this will be used when the user did not do anything. var nameOfCookie = 'switchstyle'; // This is the name of the cookie that is used. var urlToCSSDirectory = 'http://www.test.com/v2/css/'; // This is the URL to your directory where your .css files are placed on your site. // These are the names of your different .css files; use the name exactly as on your Web site var ScreenCSS_1 = 'index.css'; var ScreenCSS_2 = 'pink.css'; var ScreenCSS_3 = 'green.css'; var ScreenCSS_4 = 'red.css'; var ScreenCSS_5 = 'blue.css'; var ScreenCSS_6 = 'talkbox.css'; // If you use different .css files for printing a document, you can set these. If you don't even know what this is, name then everything exactly the same as above. // So: make then PrintCSS_1 the same as ScreenCSS_1, for example "screen_1.css". //var PrintCSS_1 = 'index.css'; //var PrintCSS_2 = 'test.css'; //var PrintCSS_3 = 'print_3.css'; //var PrintCSS_4 = 'print_4.css'; //var PrintCSS_5 = 'print_5.css'; /*********************************************************************************************** DO NOT CHANGE ANYTHING UNDER THIS LINE, UNLESS YOU KNOW WHAT YOU ARE DOING ***********************************************************************************************/ // This is the main function that does all the work function switchStyleOfUser(){ var fontSize = GetCookie(nameOfCookie); if (fontSize == null) { fontSize = standardStyle; } if (fontSize == "1") { document.write(''); } if (fontSize == "2") { document.write(''); } if (fontSize == "3") { document.write(''); } if (fontSize == "4") { document.write(''); } if (fontSize == "5") { document.write(''); } //if (fontSize == "1") { document.write(''); } //if (fontSize == "2") { document.write(''); } //if (fontSize == "3") { document.write(''); } //if (fontSize == "4") { document.write(''); } //if (fontSize == "5") { document.write(''); } var fontSize = ""; return fontSize; } var exp = new Date(); exp.setTime(exp.getTime() + (expDays*24*60*60*1000)); // Function to get the settings of the user function getCookieVal (offset) { var endstr = document.cookie.indexOf (";", offset); if (endstr == -1) endstr = document.cookie.length; return unescape(document.cookie.substring(offset, endstr)); } // Function to get the settings of the user function GetCookie (name) { var arg = name + "="; var alen = arg.length; var clen = document.cookie.length; var i = 0; while (i < clen) { var j = i + alen; if (document.cookie.substring(i, j) == arg) return getCookieVal (j); i = document.cookie.indexOf(" ", i) + 1; if (i == 0) break; } return null; } // Function to remember the settings function SetCookie (name, value) { var argv = SetCookie.arguments; var argc = SetCookie.arguments.length; var expires = (argc > 2) ? argv[2] : null; var path = (argc > 3) ? argv[3] : null; var domain = (argc > 4) ? argv[4] : null; var secure = (argc > 5) ? argv[5] : false; document.cookie = name + "=" + escape (value) + ((expires == null) ? "" : ("; expires=" + expires.toGMTString())) + ((path == null) ? "" : ("; path=" + path)) + ((domain == null) ? "" : ("; domain=" + domain)) + ((secure == true) ? "; secure" : ""); } // Function to remove the settings function DeleteCookie (name) { var exp = new Date(); exp.setTime (exp.getTime() - 1); var cval = GetCookie (name); document.cookie = name + "=" + cval + "; expires=" + exp.toGMTString(); } // This function is used when the user gives his selection function doRefresh(){ location.reload(); } // This will call the main function. Do not remove this, because otherwise this script will do nothing... document.write(switchStyleOfUser()); ``` Det som inte vekar funka är att background färgen ändras inte men där emot när jag klickar på bilderna så uppdateras sidan ! :) Permalänk: https://www.webforum.nu/p/2283490 ## #13 — voigtann1, 2011-09-26T18:47Z Sidan laddas ju om med: ``` function doRefresh(){ location.reload(); } ``` Så det är ju helt OK, du har ju eventet click så det händer inget förrän du trycker på länkarna. Permalänk: https://www.webforum.nu/p/2283491 ## #14 — vagann, 2011-09-26T18:52Z ja det sant :) , hur får vi det att ändra css filen då? Permalänk: https://www.webforum.nu/p/2283492 ## #15 — voigtann1, 2011-09-26T19:04Z Det vardet jag frågade, skulle den skriva över alla css filer du ha? Eller den skall bara läggas in i din tillsammans med din huvud CSS? Du få berätta nog exakt vad du förväntas att göra med din kod.... Permalänk: https://www.webforum.nu/p/2283493 ## #16 — vagann, 2011-09-26T19:30Z jaha alright :) jag vill att koden ska använda index.css i början men senare när användaren byter till "blå" så ska sidan använda blue.css istället. ber om ursäkt , jag e lite trög ;P Permalänk: https://www.webforum.nu/p/2283496 ## #17 — voigtann1, 2011-09-27T04:52Z Ändra: var standardStyle = '5' till det nummer du kör som default css så är du hemma. Fast om jag få tipsar om något så hade jag gjort: Haft en orginal CSS alltid, så även om de som kör utan Javascript/innan skriptet har hunnit köras) så ser inte helt hemskt ut. Och haft alla extra CSS som extendar orginal CSS:en Permalänk: https://www.webforum.nu/p/2283499 ## #18 — vagann, 2011-09-27T18:29Z kan du inte bara ge mig den rätta/färdiga koden , vill fixa det för mitt forum . kan lära mig JS senare just nu måste jag plugga inför fysik provet ! tackar! Permalänk: https://www.webforum.nu/p/2283517 ## #19 — voigtann1, 2011-09-27T18:33Z > **[vagann skrev:](https://www.webforum.nu/p/1540504)** > > kan du inte bara ge mig den rätta/färdiga koden , vill fixa det för mitt forum . kan lära mig JS senare just nu måste jag plugga inför fysik provet ! > tackar! Koden finns på länken jag har gett dig. Men även forumet kan ju vänta, studier gå före ;) Permalänk: https://www.webforum.nu/p/2283518 ## #20 — vagann, 2011-09-28T17:20Z haha visst speciellt man går i NV :P , ja men hur som helst , jag kopierade dina koder men ändå funkade den inte , vet inte varför ... håller på att bli dum \*!! jag laddade upp min zip som innehåller css och html koder om du vill kontrollera koderna Permalänk: https://www.webforum.nu/p/2283535 --- Tråden på webben: https://www.webforum.nu/amne/javascript/187990-byta-css-filer-genom-klickning Nästa sida: https://www.webforum.nu/amne/javascript/187990-byta-css-filer-genom-klickning/page2.md