[pve-devel] applied: [PATCH widget-toolkit] input panel: Add drawing of layout
Thomas Lamprecht
t.lamprecht at proxmox.com
Tue Jan 26 18:55:20 CET 2021
On 27.11.20 12:08, Dominic Jäger wrote:
> The drawing makes clear in a few seconds:
> - what columnT and columnB stand for
> - what additional containers and panels are created
> - to which of those the elements of column1, column2... go to
>
> When you're in the JS debugger and lost overview of where in this
> element hierarchy you are, you can quickly check xtype + layout. Then
> consulting this drawing solves the mistery.
>
> Signed-off-by: Dominic Jäger <d.jaeger at proxmox.com>
> ---
> IMO it's not that easy/quick to see where everything goes to.
> initComponent is not that short. I actually drew the layout on paper
> to keep an overview while reading it.
>
> There was a drawing already in the commit message of
> 455f5fe input panel: support convenience top column
> so I assumed I'm not the only one who finds such a drawing useful and
> it is easier to find here than in the commit message.
>
> src/panel/InputPanel.js | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
>
applied, thanks!
More information about the pve-devel
mailing list