[pve-devel] [PATCH] apt updates: show full package description conditionally
Tim Marx
t.marx at proxmox.com
Mon Jun 3 14:32:05 CEST 2019
Looks good.
Maybe instead of "Show Full Description" just "Show details".
> Thomas Lamprecht <t.lamprecht at proxmox.com> hat am 3. Juni 2019 um 13:02 geschrieben:
>
>
> The full package description is often quite long, and for upgrades
> it's not relevant most of the time. But, it can be intresting to
> have, if one wonders what a package is even used for.
> So add a checkbox in the top bar which allows to switch between
> showing and hiding the description, simply done by re-using the
> 'grid-row-body-hidden' ExtJS CSS class (got the idea from the
> RowBody feature source code).
> Default it to off, as it's better to not overwhelm users here with a
> huge pile of text by default, showing the list of package with the
> short one line title is enough most of the times, I'd guess.
>
> Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
> ---
> node/APT.js | 17 ++++++++++++++---
> 1 file changed, 14 insertions(+), 3 deletions(-)
>
> diff --git a/node/APT.js b/node/APT.js
> index 68075b6..4e09962 100644
> --- a/node/APT.js
> +++ b/node/APT.js
> @@ -75,11 +75,11 @@ Ext.define('Proxmox.node.APT', {
> getAdditionalData: function (data, rowIndex, record, orig) {
> var headerCt = this.view.headerCt;
> var colspan = headerCt.getColumnCount();
> - // Usually you would style the my-body-class in CSS file
> return {
> rowBody: '<div style="padding: 1em">' +
> Ext.String.htmlEncode(data.Description) +
> '</div>',
> + rowBodyCls: me.full_description ? '' : Ext.baseCSSPrefix + 'grid-row-body-hidden',
> rowBodyColspan: colspan
> };
> }
> @@ -178,10 +178,21 @@ Ext.define('Proxmox.node.APT', {
> }
> });
>
> + var verbose_desc_checkbox = new Ext.form.field.Checkbox({
> + boxLabel: gettext('Show Full Description'),
> + value: false,
> + listeners: {
> + change: (f, val) => {
> + me.full_description = val;
> + me.getView().refresh();
> + }
> + }
> + });
> +
> if (me.upgradeBtn) {
> - me.tbar = [ update_btn, me.upgradeBtn, changelog_btn ];
> + me.tbar = [ update_btn, me.upgradeBtn, changelog_btn, '->', verbose_desc_checkbox ];
> } else {
> - me.tbar = [ update_btn, changelog_btn ];
> + me.tbar = [ update_btn, changelog_btn, '->', verbose_desc_checkbox ];
> }
>
> Ext.apply(me, {
> --
> 2.20.1
>
>
> _______________________________________________
> pve-devel mailing list
> pve-devel at pve.proxmox.com
> https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
More information about the pve-devel
mailing list