[pve-devel] applied: [PATCH manager v2 0/3] add notes panel to nodes
Thomas Lamprecht
t.lamprecht at proxmox.com
Fri May 25 11:16:36 CEST 2018
On 5/25/18 10:31 AM, Dominik Csapak wrote:
> as for the commment from thomas:
>
> i wanted to put it in the 'summary' tab, but there is no way to put it there
> without changing the amount of info/graphs you see by default
> and since we do not want to disrupt users view of data, i put it
> as its own tab
>
> in the future, we possibly have more node specific configuration we
> can put there, so we need a new tab anyway
>
not sure if this is the right place but it's the best we came up
for now, so applied series
> changes from v1:
> * refactor the NotesView more and add an 'edit' button for the node
> instead of showing the header to make it more consistent with the
> rest of our gui
>
> Dominik Csapak (3):
> let NotesView load itself
> accept type 'node' for NotesView and refactor it
> fix #427: add notes panel to node
>
> www/manager6/lxc/Summary.js | 1 -
> www/manager6/node/Config.js | 6 +++
> www/manager6/panel/NotesView.js | 89 +++++++++++++++++++++++++----------------
> www/manager6/qemu/Summary.js | 1 -
> 4 files changed, 60 insertions(+), 37 deletions(-)
>
More information about the pve-devel
mailing list