Skip to content

Commit

Permalink
[ADD] doc: new documentation, with training tutorials, and new scaffo…
Browse files Browse the repository at this point in the history
…lding
  • Loading branch information
rco-odoo committed Aug 22, 2014
1 parent 97256fa commit 2ad092b
Show file tree
Hide file tree
Showing 402 changed files with 27,207 additions and 25,257 deletions.
2 changes: 1 addition & 1 deletion addons/web/static/src/js/view_list.js
Expand Up @@ -167,7 +167,7 @@ instance.web.ListView = instance.web.View.extend( /** @lends instance.web.ListVi
* @returns {String} CSS style declaration
*/
style_for: function (record) {
var style= '';
var len, style= '';

var context = _.extend({}, record.attributes, {
uid: this.session.uid,
Expand Down
4 changes: 2 additions & 2 deletions addons/web/static/src/js/views.js
Expand Up @@ -1447,12 +1447,12 @@ instance.web.View = instance.web.Widget.extend({
if (action_data.special === 'cancel') {
return handler({"type":"ir.actions.act_window_close"});
} else if (action_data.type=="object") {
var args = [[record_id]], additional_args = [];
var args = [[record_id]];
if (action_data.args) {
try {
// Warning: quotes and double quotes problem due to json and xml clash
// Maybe we should force escaping in xml or do a better parse of the args array
additional_args = JSON.parse(action_data.args.replace(/'/g, '"'));
var additional_args = JSON.parse(action_data.args.replace(/'/g, '"'));
args = args.concat(additional_args);
} catch(e) {
console.error("Could not JSON.parse arguments", action_data.args);
Expand Down
27 changes: 12 additions & 15 deletions addons/web_graph/static/src/js/graph_view.js
Expand Up @@ -52,24 +52,21 @@ instance.web_graph.GraphView = instance.web.View.extend({
if (_.has(field.attrs, 'interval')) {
field_name = field.attrs.name + ':' + field.attrs.interval;
}
if (_.has(field.attrs, 'type')) {
switch (field.attrs.type) {
case 'row':
self.widget_config.row_groupby.push(field_name);
break;
case 'col':
self.widget_config.col_groupby.push(field_name);
break;
case 'measure':
self.widget_config.measures.push(field_name);
break;
}
} else { // old style, kept for backward compatibility
//noinspection FallThroughInSwitchStatementJS
switch (field.attrs.type) {
case 'measure':
self.widget_config.measures.push(field_name);
break;
case 'col':
self.widget_config.col_groupby.push(field_name);
break;
default:
if ('operator' in field.attrs) {
self.widget_config.measures.push(field_name);
} else {
self.widget_config.row_groupby.push(field_name);
break;
}
case 'row':
self.widget_config.row_groupby.push(field_name);
}
});
if (self.widget_config.measures.length === 0) {
Expand Down
211 changes: 0 additions & 211 deletions doc/01_getting_started.rst

This file was deleted.

1 comment on commit 2ad092b

@blaggacao
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rco-odoo and team
congrats to the new layout. Somone already pulled it onto rtfd: http://odoo-80.readthedocs.org/en/latest/guides/snippets.html

My comments:
I would suggest some little tweaks on the headline css, spaces before and after headlines, etc. to give it even more visual structure. Play arround a bit with the baseline and probably make the paragraph size narrower ( maybe cutting it at 3/4 would look nice and harmoneous ).
Probably you could think about using an alternate/contrasting font for headlines, this is usually a good idea, IMO. (some suggestions: Arvo, Bree Serif, Patua One !, Comfortaa, Coustard)

on the other hand:

  • I don't see too much syntactical distinction between guides and tutorials, so maybe can be unfied
  • Module objects might be too low level. probably think of "apps docu", see here http://odoo.readthedocs.org/en/latest/apps/

Please sign in to comment.