You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hmm... kaikki vaikuttaa päällyspuolin mainiolta. Idea on hyvä, dokumentaatio on selkeä. Sivut ovat yksinkertaiset ja helposti ymmärrettävät. Toteutus vaikuttaa hyvältä. Vaikea löytää valittamista, kun kaikki toimii ja koodi on yksinkertaista ilman ylimääräisyyksiä. Jännää oli, että ainoastaan Pelaajan "password" muuttuja oli englanniksi, kun kaikki muu on Suomeksi, mutta onhan tämä jo pilkunviilausta.
Parannusehdotuksiksi (vaikka nää onkin jo mitä luultavimmin käynyt mielessäsi), voisin mainita, että yläpalkki voisi näyttää muutakin kuin etusivun, ja kirjautumissivun, mikäli tulee olemaan jatkossakin etusivu, olisi ihan jees sisältää lista radoista, jotta niitä voi selailla helposti kirjautumatta.
Jatka samaan malliin, hyvähän tästä näyttäisi tulevan. T: Koodikatselmoija.
The text was updated successfully, but these errors were encountered:
Klo 10:52 Ma. 08.02.2016
Hmm... kaikki vaikuttaa päällyspuolin mainiolta. Idea on hyvä, dokumentaatio on selkeä. Sivut ovat yksinkertaiset ja helposti ymmärrettävät. Toteutus vaikuttaa hyvältä. Vaikea löytää valittamista, kun kaikki toimii ja koodi on yksinkertaista ilman ylimääräisyyksiä. Jännää oli, että ainoastaan Pelaajan "password" muuttuja oli englanniksi, kun kaikki muu on Suomeksi, mutta onhan tämä jo pilkunviilausta.
Parannusehdotuksiksi (vaikka nää onkin jo mitä luultavimmin käynyt mielessäsi), voisin mainita, että yläpalkki voisi näyttää muutakin kuin etusivun, ja kirjautumissivun, mikäli tulee olemaan jatkossakin etusivu, olisi ihan jees sisältää lista radoista, jotta niitä voi selailla helposti kirjautumatta.
Jatka samaan malliin, hyvähän tästä näyttäisi tulevan. T: Koodikatselmoija.
The text was updated successfully, but these errors were encountered: