<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Hei,<div><br></div><div>näyttää varsin lupaavalta! Tässä muutamia varsin sekalaisia huomioita:</div><div><br></div><div>* Mielenkiintoista on, miten tämä saadaan jatkossa integroitumaan esim. Ckeditoriin.</div><div><br></div><div>* Kieliopin tarkistuksen toteutus on kaiketi vielä aika pienimuotoinen. (Esim. "Minä osaat" ei ole virhe.) Tämä ei liittyne varsinaisesti ajax-puoleen. Sen sijaan esim. tuplavälilyönti näytti jäävän mukavasti kiinni, samoin kirjainkokovirheet.</div><div><br></div><div>* Yksi ihmetys tuli vastaan, kun kopioin mallitekstiä osoitteesta <a href="http://www.loremipsum.fi/seitseman_veljesta/">http://www.loremipsum.fi/seitseman_veljesta/</a> ja huomasin, että ympäristö-sana värjäytyi punaiseksi. Syyksi paljastui alkuperäisessä tekstissä esiintynyt (näkymätön) soft hyphen, jonka vuoksi kyselyksi tuli <a href="http://joukahainen.puimula.org/ajaxvoikko/spell?q=ym%C2%ADp%C3%A4rist%C3%B6">http://joukahainen.puimula.org/ajaxvoikko/spell?q=ym%C2%ADp%C3%A4rist%C3%B6</a> (huom %C2%AD tuossa keskellä). Pitäisikö Voikon jättää syötteessä tulevat soft hyphenit huomioimatta (ainakin tavurajalla olevat...)? Vai ainakin vaihtaa ne joksikin muiksi merkeiksi tulosteessaan? No, tämä lienee melko epätavallinen tilanne joka tapauksessa.</div><div><br></div><div>* Kyselyt kannattaisi varmaankin tehdä post-metodilla, ainakin jos teksti (tai siis sen querystring-koodattu versio!) on pitkä.</div><div><br></div><div>* Muuten, "En ollut ollut kotona." huomauttaa toistosta. Tosi pieni ongelma sekin, eikä sekään ajax-seikka. (Sen sijaan "Talo talo." ei huomauta, ilmeisesti koska ensimmäinen sana on isolla.)</div><div><br></div><div>* js-koodissa tämä funktio näyttää ainakin yhtäkkiä vähän oudolta (content- ja html-muuttujat?):</div><div><br></div><div><pre id="line1">function buildGrammarError(outerElement) {
var content = outerElement.html();
outerElement.html("*<span class='gErrorInner'>" + html + "</span>");
}
</pre><div>Muuten js-koodi näyttää siistiltä. Laittaisin kaiken vielä anonyymifunktion sisään, jotta nimiavaruus pysyy siistinä. Näyttäisi olevan mahdollista siirtää koko koodi google-kutsuja lukuunottamatta näin:</div><div><br></div><div><pre id="line1"><font class="Apple-style-span" face="Helvetica"><span class="Apple-style-span" style="white-space: normal;"><font class="Apple-style-span" face="monospace"><span class="Apple-style-span" style="white-space: pre;"><pre id="line1">google.setOnLoadCallback(function() { jQuery(function($) {</pre><pre id="line1"> // tähän kaikki funktiot</pre><pre id="line1"> $("#input").keyup(keyUpInInput);</pre><pre id="line1">});});</pre></span></font></span></font></pre></div></div><div><br></div><div>Vielä yksi js-huomio: errortext-attribuuttia lisätään nyt dokumenttiin html-muotoisena. Vaikka attribuutin sisältö onkin luotettua, suosittelisin xss-ongelmien välttämiseksi muuttamaan attribuutin nimen (jolloin serveripuolella muistetaan että ollaan tuottamassa html:ää) - tai lisäämään sisällön eskapoinnin johonkin tälle kohdille koodia:</div><div><br></div><div><pre id="line1"> var explanation = outerElement.attr("errortext");
var html = "<div>... " + original.html() + " ...<br />"
+ explanation + "</div>";
</pre><div><br></div><div><br></div><div>No joo, tosiaankin satunnaisia huomioita. Joka tapauksessa: hyvältä näyttää!</div><div><br></div></div><div>Jatkossahan tästä saa tosi helposti esim. jQuery-pluginin.</div><div><br></div><div><br></div><div>Tuomas</div><div><br></div><div><br><div><div>On Jan 31, 2010, at 21:00 , Harri Pitkänen wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div>Webvoikon tuleva Ajax-pohjainen korvaaja on nyt julkisesti testattavissa <br>osoitteessa<br><br> <a href="http://joukahainen.puimula.org/ajaxvoikko/">http://joukahainen.puimula.org/ajaxvoikko/</a><br><br>Tämä versio tarjoaa mm. seuraavat ominaisuudet:<br>- oikoluku<br>- väärin kirjoitettujen sanojen korjausehdotukset<br>- kieliopin tarkistus<br>- sanojen morfologinen analyysi<br>- linkit Joukahaisen tietueisiin sanoille, joille sellaiset löytyy.<br><br>Otan vastaan kehitysideoita, jos tulee mieleen tapoja joilla tätä voi <br>kehittää. Käyttöliittymä on toki vielä aika askeettinen, sen kehittäminen ei <br>ole ollut vielä etusijalla.<br><br>Palvelu saattaa hävitä pois käytöstä ilman varoitusta, pyrin kuitenkin <br>pitämään sen saatavilla jossain muodossa jatkossakin. URL tulee muuttumaan, <br>joten älkää turhaan mainostako tätä vielä laajemmin.<br><br>Harri<br>_______________________________________________<br>voikko mailing list<br><a href="mailto:voikko@lists.puimula.org">voikko@lists.puimula.org</a><br>http://lists.puimula.org/listinfo/voikko<br></div></blockquote></div><br></div></body></html>