proxmox-mirrors/pmg-gui
1
0
Fork 0
mirror of https://git.proxmox.com/git/pmg-gui synced 2025-08-24 09:29:33 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

quarantines: color code deliver and delete buttons

Browse Source 
In some languages, e.g. widely used English, "Delete" and "Deliver"
are looking rather similar without too much eye squinting required.

Add an accent on their icons using for, at least in the western
hemisphere, the respective associated color, like red for trash can.
To 1) avoid to much messing with those cultures that got it switched
and 2) some more common color blindness use blue for the delivery
button, just to make it stand out.

Keep the white/black list as is, as such accents work best if not
everything uses them, i.e., so that they more common options actually
stand out.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
This commit is contained in:
Thomas Lamprecht 2022-10-22 16:16:39 +02:00
parent 2cc13fbc8a
commit 365f84c19b
5 changed files with 10 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
js/AttachmentQuarantine.js
View File

@ -166,13 +166,13 @@ Ext.define('PMG.AttachmentQuarantine', {
{
reference: 'deliver',
text: gettext('Deliver'),
iconCls: 'fa fa-paper-plane-o',
iconCls: 'fa fa-paper-plane-o info-blue',
handler: 'btnHandler',
},
{
reference: 'delete',
text: gettext('Delete'),
iconCls: 'fa fa-trash-o',
iconCls: 'fa fa-trash-o critical',
handler: 'btnHandler',
},
],

4
js/QuarantineContextMenu.js
View File

@ -17,13 +17,13 @@ Ext.define('PMG.menu.QuarantineContextMenu', {
items: [
{
text: gettext('Deliver'),
iconCls: 'fa fa-fw fa-paper-plane-o',
iconCls: 'fa fa-fw fa-paper-plane-o good',
action: 'deliver',
handler: 'callCallback',
},
{
text: gettext('Delete'),
iconCls: 'fa fa-fw fa-trash-o',
iconCls: 'fa fa-fw fa-trash-o critical',
action: 'delete',
handler: 'callCallback',
},

4
js/SpamContextMenu.js
View File

@ -4,13 +4,13 @@ Ext.define('PMG.menu.SpamContextMenu', {
items: [
{
text: gettext('Deliver'),
iconCls: 'fa fa-fw fa-paper-plane-o',
iconCls: 'fa fa-fw fa-paper-plane-o info-blue',
action: 'deliver',
handler: 'callCallback',
},
{
text: gettext('Delete'),
iconCls: 'fa fa-fw fa-trash-o',
iconCls: 'fa fa-fw fa-trash-o critical',
action: 'delete',
handler: 'callCallback',
},

4
js/SpamQuarantine.js
View File

@ -364,13 +364,13 @@ Ext.define('PMG.SpamQuarantine', {
{
reference: 'deliver',
text: gettext('Deliver'),
iconCls: 'fa fa-paper-plane-o',
iconCls: 'fa fa-paper-plane-o info-blue',
handler: 'btnHandler',
},
{
reference: 'delete',
text: gettext('Delete'),
iconCls: 'fa fa-trash-o',
iconCls: 'fa fa-trash-o critical',
handler: 'btnHandler',
},
],

4
js/VirusQuarantine.js
View File

@ -175,13 +175,13 @@ Ext.define('PMG.VirusQuarantine', {
{
reference: 'deliver',
text: gettext('Deliver'),
iconCls: 'fa fa-paper-plane-o',
iconCls: 'fa fa-paper-plane-o info-blue',
handler: 'btnHandler',
},
{
reference: 'delete',
text: gettext('Delete'),
iconCls: 'fa fa-trash-o',
iconCls: 'fa fa-trash-o critical',
handler: 'btnHandler',
},
],