[pve-devel] [PATCH manager 1/3] CreateWizard: remove title from confirm tabpanel

Emmanuel Kasper e.kasper at proxmox.com
Thu Aug 24 14:27:23 CEST 2017


makes sense, looks good to me

On 07/26/2017 03:49 PM, Thomas Lamprecht wrote:
> It isn't needed as its obvious that the table below show the
> configuration. It just takes away space.
> 
> Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
> ---
>  www/manager6/lxc/CreateWizard.js  | 1 -
>  www/manager6/qemu/CreateWizard.js | 1 -
>  2 files changed, 2 deletions(-)
> 
> diff --git a/www/manager6/lxc/CreateWizard.js b/www/manager6/lxc/CreateWizard.js
> index c2f16a1a..8515877a 100644
> --- a/www/manager6/lxc/CreateWizard.js
> +++ b/www/manager6/lxc/CreateWizard.js
> @@ -258,7 +258,6 @@ Ext.define('PVE.lxc.CreateWizard', {
>  		    layout: 'fit',
>  		    items: [
>  			{
> -			    title: gettext('Settings'),
>  			    xtype: 'grid',
>  			    store: summarystore,
>  			    columns: [
> diff --git a/www/manager6/qemu/CreateWizard.js b/www/manager6/qemu/CreateWizard.js
> index aab78479..29374b3a 100644
> --- a/www/manager6/qemu/CreateWizard.js
> +++ b/www/manager6/qemu/CreateWizard.js
> @@ -127,7 +127,6 @@ Ext.define('PVE.qemu.CreateWizard', {
>  		    layout: 'fit',
>  		    items: [
>  			{
> -			    title: gettext('Settings'),
>  			    xtype: 'grid',
>  			    store: summarystore,
>  			    columns: [
> 




More information about the pve-devel mailing list