widelands-dev team mailing list archive
-
widelands-dev team
-
Mailing list archive
-
Message #11311
Re: [Merge] lp:~trimardio/widelands-website/scheduling_module into lp:widelands-website
> Kaputtnik, could you merge this once you are happy?
I am not good at code review, but will try.
As i understand trimardio in our last pms, he wants to work further on this?
@trimardio:
> - clone html element in js: more coherent and a bit less work
We should do this i think, so we have one template in html which can be adjusted. This is imho less error prone than modifying js.
> Another question, should I jquerify all the script for easier understanding by future reader?
At least we should have one style in one script file.
--
https://code.launchpad.net/~trimardio/widelands-website/scheduling_module/+merge/331477
Your team Widelands Developers is subscribed to branch lp:widelands-website.
References