← Back to team overview

widelands-dev team mailing list archive

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