facundobatista
commented
8 years ago Hello!
Great cleaning up! Thanks for this.
Some issues you should address before merging:
- The "size of the editor area" is broken: now it's small, as it was a long ago! The idea is that it should take as much as the screen as possible (yes, we have the issue about where to locate the node tree, but making the editor small is not a solution: we should maximize the editor area).
- Some tests failed with "ImportError: decorators", see http://linkode.org/xBHnKUYGNuv5D1SO8ejWX
- pylint fails with a bunch of stuff too: http://linkode.org/SIfolNh26Wv5kNB52a7Sh1
matibarriento
This is the first PR on upgrades and organization.
In this PR the main code was divided into API and Webclient.
The API code is the core of Kilink and Webclient code will consume the API. Templates and Statics should go in the Webclient blueprint (module)
There is a decorator called "json_return" to convert API returns in JSON and preserve the original return to the WebClient.
I upgraded the libraries to Jquery 2.2.4, ToolTipster 4.0.4 and CoderMirror 5.16, but I had to remove Rust from options for a bug in CodeMirror.
There is an empty file called Main.js, which plans to have all the js in a module for better use and maintenance, in addition to the use of API Kilink.
There is code that can optimize and remove, is a work in progress.
Also, there are some minor fixes in templates
Hope you like