Chromium Code Reviews| Index: appengine/swarming/elements/res/imp/botlist/DESIGN.md |
| diff --git a/appengine/swarming/elements/res/imp/botlist/DESIGN.md b/appengine/swarming/elements/res/imp/botlist/DESIGN.md |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..06cb123160e2f5bf5d8c256fe5c279cabf50bcb5 |
| --- /dev/null |
| +++ b/appengine/swarming/elements/res/imp/botlist/DESIGN.md |
| @@ -0,0 +1,29 @@ |
| +This is a highlevel overview of the pieces that make up botlist. |
| + |
| +``` |
| + +--------------+ |
| + +------------->+CommonBehavior+<-----------------+ |
|
jcgregorio
2016/08/23 17:58:59
nit: Can you add a single spacing in the boxes? Fo
kjlubick
2016/08/23 19:13:56
Done.
|
| + | +------+-------+ | |
| + | ^ | |
| + | | | |
| + | | | |
| + | | | |
| ++--------+--------+ +------+--------+ +---------+----------+ |
| +|QueryColumnFilter| |BotListBehavior| |DynamicTableBehavior| |
| ++-----------------+ +---------------+ +--------------------+ |
| + ^ +-------------^ ^ ^ ^-------------------+ ^ |
| + | | | | | | |
| + | | | +---+ | | |
| + | | | | | | |
| + +----+--+-----+ | +-----+------------+ +-+---+----+ |
| + |<bot-filters>| | |<bot-list-summary>| |<bot-list>| |
|
jcgregorio
2016/08/23 17:58:59
This shows inheritance of behaviors, which is grea
kjlubick
2016/08/23 19:13:56
I don't think I need a diagram for this. See line
|
| + +-------------+ | +------------------+ +----------+ |
| + | +---------------+ |
| + +-----+<bot-list-data>| |
| + +---------------+ |
| +``` |
| + |
| +Although methods defined in the various behaviors could be overwritten by each |
| +other, this is not done. |
| + |
| +`<bot-list>` is the main element and composes the others. |