From 250f82b9cafdc335f916fd279fb5f050bba09db0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?G=C3=A9ry=20Debongnie?= Date: Mon, 28 Jan 2019 10:59:46 +0100 Subject: [PATCH] update readme (last section on to do/discussions) --- README.md | 30 ++++++++++++++++++------------ 1 file changed, 18 insertions(+), 12 deletions(-) diff --git a/README.md b/README.md index d544fd0a..eafdbac6 100644 --- a/README.md +++ b/README.md @@ -159,16 +159,22 @@ We have 3 main folders and 3 main files: ## Random Notes -Before even thinking about using this in a real scenario: +- Change of behaviour: The compilation of a template should have a unique root + node (but sub templates can have multiple roots) -- check qweb tests and see if it is reasonable (escaping? safety/security?) -- Note: the compilation of a template should have a unique node (but sub templates - can have multiple roots) -- remove the "if (${exprID} || ${exprID} === 0) {" -- style is props? difference between props and attrs -- text node -- t-extend??? -- improve qweb generated code: do not assign object/array if no props/attrs/children -- improve qweb gen code: when building a vnode, propagate a structure with - children/attrs/hooks, fill it properly by each directive, then and only - then create node with minimal code +## TO THINK ABOUT/TO DO + +- **Qweb** + + - check qweb tests and see if it is reasonable (escaping? safety/security?) + - remove the "if (${exprID} || ${exprID} === 0) {" + - need to implement t-extend (?) + - improve qweb generated code: do not assign object/array if no props/attrs/children + - improve qweb gen code: when building a vnode, propagate a structure with + children/attrs/hooks, fill it properly by each directive, then and only + then create node with minimal code + +- **Widget** + + - check if it is possible to remove parent reference in widget + - default implementation of propsUpdated?