IntenseWebs/freeipa
3
0
Fork 0
mirror of https://salsa.debian.org/freeipa-team/freeipa.git synced 2025-02-25 18:55:28 -06:00
Code Issues Packages Projects Releases Wiki Activity

webui: disable batch action buttons by default

Browse Source 
action buttons associated with batch actions were enabled by default, but
they were disabled right after facet creation and a load of data. It caused
a visual flicker.

UX is enhanced by making them disabled by default.

Reviewed-By: Endi Sukma Dewata <edewata@redhat.com>
This commit is contained in:
Petr Vobornik 2014-07-31 13:14:57 +02:00
parent a8a799822c
commit dd45278e5a
3 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
install/ui/src/freeipa/association.js
View File

@ -868,6 +868,7 @@ exp.association_facet_pre_op = function(spec, context) {
hide_cond: ['read-only'], hide_cond: ['read-only'],
show_cond: ['direct'], show_cond: ['direct'],
enable_cond: ['item-selected'], enable_cond: ['item-selected'],
enabled: false,
handler: function(facet) { handler: function(facet) {
facet.show_remove_dialog(); facet.show_remove_dialog();
} }

2
install/ui/src/freeipa/otptoken.js
View File

@ -78,6 +78,7 @@ return {
options: { ipatokendisabled: false }, options: { ipatokendisabled: false },
needs_confirm: true, needs_confirm: true,
enable_cond: ['item-selected'], enable_cond: ['item-selected'],
enabled: false,
success_msg: '@i18n:search.enabled', success_msg: '@i18n:search.enabled',
confirm_msg: '@i18n:search.enable_confirm' confirm_msg: '@i18n:search.enable_confirm'
}, },
@ -88,6 +89,7 @@ return {
options: { ipatokendisabled: true }, options: { ipatokendisabled: true },
needs_confirm: true, needs_confirm: true,
enable_cond: ['item-selected'], enable_cond: ['item-selected'],
enabled: false,
success_msg: '@i18n:search.disabled', success_msg: '@i18n:search.disabled',
confirm_msg: '@i18n:search.disable_confirm' confirm_msg: '@i18n:search.disable_confirm'
}, },

3
install/ui/src/freeipa/search.js
View File

@ -430,6 +430,7 @@ IPA.batch_remove_action = function(spec) {
spec.name = spec.name || 'remove'; spec.name = spec.name || 'remove';
spec.label = spec.label || '@i18n:buttons.remove'; spec.label = spec.label || '@i18n:buttons.remove';
spec.enable_cond = spec.enable_cond || ['item-selected']; spec.enable_cond = spec.enable_cond || ['item-selected'];
spec.enabled = spec.enabled === undefined ? false : spec.enabled;
spec.hide_cond = spec.hide_cond || ['self-service']; spec.hide_cond = spec.hide_cond || ['self-service'];
var that = IPA.action(spec); var that = IPA.action(spec);
@ -535,6 +536,7 @@ IPA.batch_disable_action = function(spec) {
spec.method = spec.method || 'disable'; spec.method = spec.method || 'disable';
spec.needs_confirm = spec.needs_confirm === undefined ? true : spec.needs_confirm; spec.needs_confirm = spec.needs_confirm === undefined ? true : spec.needs_confirm;
spec.enable_cond = spec.enable_cond || ['item-selected']; spec.enable_cond = spec.enable_cond || ['item-selected'];
spec.enabled = spec.enabled === undefined ? false : spec.enabled;
spec.success_msg = spec.success_msg || '@i18n:search.disabled'; spec.success_msg = spec.success_msg || '@i18n:search.disabled';
spec.confirm_msg = spec.confirm_msg || '@i18n:search.disable_confirm'; spec.confirm_msg = spec.confirm_msg || '@i18n:search.disable_confirm';
@ -548,6 +550,7 @@ IPA.batch_enable_action = function(spec) {
spec.name = spec.name || 'enable'; spec.name = spec.name || 'enable';
spec.method = spec.method || 'enable'; spec.method = spec.method || 'enable';
spec.needs_confirm = spec.needs_confirm === undefined ? true : spec.needs_confirm; spec.needs_confirm = spec.needs_confirm === undefined ? true : spec.needs_confirm;
spec.enabled = spec.enabled === undefined ? false : spec.enabled;
spec.enable_cond = spec.enable_cond || ['item-selected']; spec.enable_cond = spec.enable_cond || ['item-selected'];
spec.success_msg = spec.success_msg || '@i18n:search.enabled'; spec.success_msg = spec.success_msg || '@i18n:search.enabled';
spec.confirm_msg = spec.confirm_msg || '@i18n:search.enable_confirm'; spec.confirm_msg = spec.confirm_msg || '@i18n:search.enable_confirm';